mm: remove SWAP_MLOCK check for SWAP_SUCCESS in ttu
authorMinchan Kim <[email protected]>
Wed, 3 May 2017 21:54:07 +0000 (14:54 -0700)
committerLinus Torvalds <[email protected]>
Wed, 3 May 2017 22:52:10 +0000 (15:52 -0700)
If the page is mapped and rescue in try_to_unmap_one, the
page_mapcount() of a page cannot be zero, so the page_mapcount check in
try_to_unmap is enough to return SWAP_SUCCESS.  IOW, SWAP_MLOCK check is
redundant so remove it.

Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Minchan Kim <[email protected]>
Cc: Anshuman Khandual <[email protected]>
Cc: Hillf Danton <[email protected]>
Cc: Johannes Weiner <[email protected]>
Cc: Kirill A. Shutemov <[email protected]>
Cc: Michal Hocko <[email protected]>
Cc: Naoya Horiguchi <[email protected]>
Cc: Vlastimil Babka <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
mm/rmap.c

index f6aa18d8a420d492b11a0ded1b0e695dfec0e3ec..dfe40557ea29f55c00999e6c5bd0a252f43a3d73 100644 (file)
--- a/mm/rmap.c
+++ b/mm/rmap.c
@@ -1535,7 +1535,7 @@ int try_to_unmap(struct page *page, enum ttu_flags flags)
        else
                ret = rmap_walk(page, &rwc);
 
-       if (ret != SWAP_MLOCK && !page_mapcount(page))
+       if (!page_mapcount(page))
                ret = SWAP_SUCCESS;
        return ret;
 }