Re: [patch -mm] mm, thp: fix mlock statistics fix
From: Hugh Dickins <hughd@google.com>
Date: 2012-10-03 21:29:59
Also in:
lkml
On Wed, 3 Oct 2012, David Rientjes wrote:
On Wed, 3 Oct 2012, Andrew Morton wrote:quoted
The free_page_mlock() hunk gets dropped because free_page_mlock() is removed. And clear_page_mlock() doesn't need this treatment. But please check my handiwork.I reviewed what was merged into -mm and clear_page_mlock() does need this fix as well. It's an easy fix, there's no need to pass "anon" into clear_page_mlock() since PageHuge() is already checked in its only caller. mm, thp: fix mlock statistics fix Signed-off-by: David Rientjes <rientjes@google.com>
Acked-by: Hugh Dickins <hughd@google.com> Thanks for providing that, David, I was just on the point of building and working out a test, suspecting that what you've added is necessary. Probably some equivalent always was missing, but between the THP Mlock counting issue that you're fixing, and the Mlock counting issue that I'm fixing by moving the clear_page_mlock, it's hard to say just where. While clear_page_mlock was being called from truncate.c, we knew that it couldn't happen on a THP. But now that it's from page_remove_rmap, yes, we do want to add in this additional fix. Hugh
quoted hunk ↗ jump to hunk
--- mm/mlock.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)diff --git a/mm/mlock.c b/mm/mlock.c --- a/mm/mlock.c +++ b/mm/mlock.c@@ -56,7 +56,8 @@ void clear_page_mlock(struct page *page) if (!TestClearPageMlocked(page)) return; - dec_zone_page_state(page, NR_MLOCK); + mod_zone_page_state(page_zone(page), NR_MLOCK, + -hpage_nr_pages(page)); count_vm_event(UNEVICTABLE_PGCLEARED); if (!isolate_lru_page(page)) { putback_lru_page(page);
-- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>