Even is_file_lru() return bool type, we leverage it as an index for anon or file. With this in hand, after reorder scan_balance definition, we can compare directly between scan_balance and file to get the exclusive scan type. After this, the code looks more readable and save some instructions. Signed-off-by: Wei Yang <richard.weiyang@xxxxxxxxx> CC: Johannes Weiner <hannes@xxxxxxxxxxx> CC: Mel Gorman <mgorman@xxxxxxx> --- mm/vmscan.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mm/vmscan.c b/mm/vmscan.c index 89745cf34386..0b7cc9309e70 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2638,10 +2638,10 @@ static bool inactive_is_low(struct lruvec *lruvec, enum lru_list inactive_lru) } enum scan_balance { - SCAN_EQUAL, - SCAN_FRACT, SCAN_ANON, SCAN_FILE, + SCAN_EQUAL, + SCAN_FRACT, }; /* @@ -2838,7 +2838,7 @@ static void get_scan_count(struct lruvec *lruvec, struct scan_control *sc, case SCAN_FILE: case SCAN_ANON: /* Scan one type exclusively */ - if ((scan_balance == SCAN_FILE) != file) + if (scan_balance != file) scan = 0; break; default: -- 2.33.1