commit 818b59b2292f87c3781a6f7a288d10e2453b0d13
author Suren Baghdasaryan <surenb@google.com> Fri Apr 13 11:49:54 2018 -0700
committer Suren Baghdasaryan <surenb@google.com> Fri Apr 13 23:24:24 2018 +0000
tree fc2432f425aac19b15c3f3956389e2345ea46b9e
parent ff61afb002d7683b309b2b174fc7b18d30e5dbd1

lmkd: Fix usage of ro.lmk.kill_heaviest_task property

lmkd should use ro.lmk.kill_heaviest_task property to select between
algorithms for victim selection. Set ro.lmk.kill_heaviest_task default
value to false in order to keep it compatible with previous versions
of lmkd (killing the heaviest task is a new mechanism).

Change-Id: I78d2dc79d9c54e636c26665605518d9c87b535b3
Signed-off-by: Suren Baghdasaryan <surenb@google.com>

lmkd/lmkd.c

diff --git a/lmkd/lmkd.c b/lmkd/lmkd.c
index f19b43e..4c98a35 100644
--- a/lmkd/lmkd.c
+++ b/lmkd/lmkd.c
@@ -788,10 +788,8 @@
         struct proc *procp;
 
         while (true) {
-            if (low_ram_device)
-                procp = proc_adj_lru(i);
-            else
-                procp = proc_get_heaviest(i);
+            procp = kill_heaviest_task ?
+                proc_get_heaviest(i) : proc_adj_lru(i);
 
             if (!procp)
                 break;
@@ -1202,7 +1200,7 @@
     downgrade_pressure =
         (int64_t)property_get_int32("ro.lmk.downgrade_pressure", 100);
     kill_heaviest_task =
-        property_get_bool("ro.lmk.kill_heaviest_task", true);
+        property_get_bool("ro.lmk.kill_heaviest_task", false);
     low_ram_device = property_get_bool("ro.config.low_ram", false);
     kill_timeout_ms =
         (unsigned long)property_get_int32("ro.lmk.kill_timeout_ms", 0);