We want to limit the use of page_mapcount() to the places where it is
absolutely necessary. We can only unmap full folios; page_mapped(),
which we check here, is translated to folio_mapped() -- based on
folio_mapcount(). So let's print the folio mapcount instead.

Signed-off-by: David Hildenbrand <da...@redhat.com>
---
 mm/memory-failure.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mm/memory-failure.c b/mm/memory-failure.c
index 88359a185c5f..ee2f4b8905ef 100644
--- a/mm/memory-failure.c
+++ b/mm/memory-failure.c
@@ -1628,8 +1628,8 @@ static bool hwpoison_user_mappings(struct page *p, 
unsigned long pfn,
 
        unmap_success = !page_mapped(p);
        if (!unmap_success)
-               pr_err("%#lx: failed to unmap page (mapcount=%d)\n",
-                      pfn, page_mapcount(p));
+               pr_err("%#lx: failed to unmap page (folio mapcount=%d)\n",
+                      pfn, folio_mapcount(page_folio(p)));
 
        /*
         * try_to_unmap() might put mlocked page in lru cache, so call
-- 
2.44.0


Reply via email to