Revert "Only log the lmkd state change when the killing really happened." am: 0051684905

am: 971f23bf66

Change-Id: Ia57c8d7e63c538442780fa20809c886a3af31941
This commit is contained in:
Dan Willemsen 2018-05-14 20:35:38 -07:00 committed by android-build-merger
commit 6e24015a96
1 changed files with 5 additions and 10 deletions

15
lmkd.c
View File

@ -1062,7 +1062,9 @@ static int find_and_kill_processes(enum vmpressure_level level,
int pages_freed = 0; int pages_freed = 0;
#ifdef LMKD_LOG_STATS #ifdef LMKD_LOG_STATS
bool lmk_state_change_start = false; if (enable_stats_log) {
stats_write_lmk_state_changed(log_ctx, LMK_STATE_CHANGED, LMK_STATE_CHANGE_START);
}
#endif #endif
for (i = OOM_SCORE_ADJ_MAX; i >= min_score_adj; i--) { for (i = OOM_SCORE_ADJ_MAX; i >= min_score_adj; i--) {
@ -1077,18 +1079,11 @@ static int find_and_kill_processes(enum vmpressure_level level,
killed_size = kill_one_process(procp, min_score_adj, level); killed_size = kill_one_process(procp, min_score_adj, level);
if (killed_size >= 0) { if (killed_size >= 0) {
#ifdef LMKD_LOG_STATS
if (enable_stats_log && !lmk_state_change_start) {
lmk_state_change_start = true;
stats_write_lmk_state_changed(log_ctx, LMK_STATE_CHANGED, FIRST_KILLING_DONE);
}
#endif
pages_freed += killed_size; pages_freed += killed_size;
if (pages_freed >= pages_to_free) { if (pages_freed >= pages_to_free) {
#ifdef LMKD_LOG_STATS #ifdef LMKD_LOG_STATS
if (enable_stats_log && lmk_state_change_start) { if (enable_stats_log) {
stats_write_lmk_state_changed(log_ctx, LMK_STATE_CHANGED, stats_write_lmk_state_changed(log_ctx, LMK_STATE_CHANGED,
LMK_STATE_CHANGE_STOP); LMK_STATE_CHANGE_STOP);
} }
@ -1100,7 +1095,7 @@ static int find_and_kill_processes(enum vmpressure_level level,
} }
#ifdef LMKD_LOG_STATS #ifdef LMKD_LOG_STATS
if (enable_stats_log && lmk_state_change_start) { if (enable_stats_log) {
stats_write_lmk_state_changed(log_ctx, LMK_STATE_CHANGED, LMK_STATE_CHANGE_STOP); stats_write_lmk_state_changed(log_ctx, LMK_STATE_CHANGED, LMK_STATE_CHANGE_STOP);
} }
#endif #endif