Drop the maximum recommended log size from 16MiB to 8MiB.
Now we compress logs, 16MiB is a *lot* of logging, often over 100MiB when decompressed. This can cause trouble for tools (and their corresponding timeouts) if they're expecting 16MiB to mean "less than 16MiB" rather than "about 10 times 16MiB". This patch doesn't address the issue of devices that are already configured too high, since it's not obvious that we reasonably can. A user can set the system property themselves, and we have no way to distinguish the two cases. Bug: https://issuetracker.google.com/176959217 Test: treehugger Change-Id: Iba92061b278c487c3dc9a98aa54f014dfb2ea1c9
This commit is contained in:
@@ -291,7 +291,7 @@
|
||||
<item>256K</item>
|
||||
<item>1M</item>
|
||||
<item>4M</item>
|
||||
<item>16M</item>
|
||||
<item>8M</item>
|
||||
</string-array>
|
||||
|
||||
<!-- Titles for logd limit size lowram selection preference. [CHAR LIMIT=14] -->
|
||||
@@ -309,7 +309,7 @@
|
||||
<item>262144</item>
|
||||
<item>1048576</item>
|
||||
<item>4194304</item>
|
||||
<item>16777216</item>
|
||||
<item>8388608</item>
|
||||
</string-array>
|
||||
|
||||
<!-- Summaries for logd limit size selection preference. [CHAR LIMIT=50]-->
|
||||
@@ -319,7 +319,7 @@
|
||||
<item>256K per log buffer</item>
|
||||
<item>1M per log buffer</item>
|
||||
<item>4M per log buffer</item>
|
||||
<item>16M per log buffer</item>
|
||||
<item>8M per log buffer</item>
|
||||
</string-array>
|
||||
|
||||
<!-- Values for logpersist state selection preference. -->
|
||||
|
||||
Reference in New Issue
Block a user