From f78fd1fa2a5f4ef9d1eac48dc772d79154cd8bf7 Mon Sep 17 00:00:00 2001
From: Robin Peng <robinpeng@google.com>
Date: Tue, 17 Dec 2019 15:10:24 +0800
Subject: [PATCH] persist all logs to disk on userdebug and eng builds

Having long traces really helps to debug nasty power bugs.

Bug: 114766334
Change-Id: I9a5cffc039649d59afdb00480746d56d7b849b3f
Signed-off-by: Thierry Strudel <tstrudel@google.com>
Signed-off-by: Robin Peng <robinpeng@google.com>
---
 device.mk | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/device.mk b/device.mk
index 7b95f836..806a79ee 100755
--- a/device.mk
+++ b/device.mk
@@ -637,10 +637,14 @@ PRODUCT_COPY_FILES += \
 PRODUCT_PACKAGES += \
     charger_res_images
 
-# b/36703476
-# Set default log size on userdebug/eng build to 1M
 ifneq (,$(filter userdebug eng, $(TARGET_BUILD_VARIANT)))
-PRODUCT_PROPERTY_OVERRIDES += ro.logd.size=1M
+# b/36703476: Set default log size to 1M
+PRODUCT_PROPERTY_OVERRIDES += \
+  ro.logd.size=1M
+# b/114766334: persist all logs by default rotating on 30 files of 1MiB
+PRODUCT_PROPERTY_OVERRIDES += \
+  logd.logpersistd=logcatd \
+  logd.logpersistd.size=30
 endif
 
 # Dumpstate HAL