Messages in this thread | | | From | HORIGUCHI NAOYA(堀口 直也) <> | Subject | Re: [PATCH v2 08/10] mm: rmap: make try_to_unmap() void function | Date | Thu, 10 Jun 2021 07:57:25 +0000 |
| |
On Tue, Jun 08, 2021 at 09:25:22PM -0700, Hugh Dickins wrote: > From: Yang Shi <shy828301@gmail.com> > > Currently try_to_unmap() return bool value by checking page_mapcount(), > however this may return false positive since page_mapcount() doesn't > check all subpages of compound page. The total_mapcount() could be used > instead, but its cost is higher since it traverses all subpages. > > Actually the most callers of try_to_unmap() don't care about the > return value at all. So just need check if page is still mapped by > page_mapped() when necessary. And page_mapped() does bail out early > when it finds mapped subpage. > > Suggested-by: Hugh Dickins <hughd@google.com> > Signed-off-by: Yang Shi <shy828301@gmail.com> > Acked-by: Minchan Kim <minchan@kernel.org> > Reviewed-by: Shakeel Butt <shakeelb@google.com> > Acked-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com> > Signed-off-by: Hugh Dickins <hughd@google.com>
Looks good to me, thank you.
Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com> | |