Skip to content

Commit c4ecddf

Browse files
Mateusz Nosektorvalds
authored andcommitted
mm/vmscan.c: do_try_to_free_pages(): clean code by removing unnecessary assignment
sc->memcg_low_skipped resets skipped_deactivate to 0 but this is not needed as this code path is never reachable with skipped_deactivate != 0 due to previous sc->skipped_deactivate branch. [[email protected]: rewrite changelog] Signed-off-by: Mateusz Nosek <[email protected]> Signed-off-by: Andrew Morton <[email protected]> Acked-by: Johannes Weiner <[email protected]> Cc: Michal Hocko <[email protected]> Link: http://lkml.kernel.org/r/[email protected] Signed-off-by: Linus Torvalds <[email protected]>
1 parent 4b79306 commit c4ecddf

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

mm/vmscan.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3093,7 +3093,6 @@ static unsigned long do_try_to_free_pages(struct zonelist *zonelist,
30933093
if (sc->memcg_low_skipped) {
30943094
sc->priority = initial_priority;
30953095
sc->force_deactivate = 0;
3096-
sc->skipped_deactivate = 0;
30973096
sc->memcg_low_reclaim = 1;
30983097
sc->memcg_low_skipped = 0;
30993098
goto retry;

0 commit comments

Comments
 (0)