From 94cd15bf2267b998a2aea96e3dd1121b9dbfc423 Mon Sep 17 00:00:00 2001 From: Juju Sung Date: Wed, 20 Feb 2019 00:41:28 +0800 Subject: [PATCH] lmkd: Remove redundant free swap and free memory check lmkd can't kill processes because it has compare the size between free swap and free memory. Free swap is often larger than the free memory when system is under low memory with less swap-backed or swappable pages and finally leads to I/O thrashing. Test: TreeHugger Bug: 124727769 Change-Id: Ia2848859aa97a24bd13c704acee4b86cd2d3f647 --- lmkd.c | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/lmkd.c b/lmkd.c index 562e578..98b3aa1 100644 --- a/lmkd.c +++ b/lmkd.c @@ -1694,16 +1694,6 @@ do_kill: static unsigned long report_skip_count = 0; if (!use_minfree_levels) { - /* If pressure level is less than critical and enough free swap then ignore */ - if (level < VMPRESS_LEVEL_CRITICAL && - mi.field.free_swap > low_pressure_mem.max_nr_free_pages) { - if (debug_process_killing) { - ALOGI("Ignoring pressure since %" PRId64 - " swap pages are available ", - mi.field.free_swap); - } - return; - } /* Free up enough memory to downgrate the memory pressure to low level */ if (mi.field.nr_free_pages >= low_pressure_mem.max_nr_free_pages) { if (debug_process_killing) {