• Hugh Dickins's avatar
    mm,thp,rmap: clean up the end of __split_huge_pmd_locked() · 96d82deb
    Hugh Dickins authored
    It's hard to add a page_add_anon_rmap() into __split_huge_pmd_locked()'s
    HPAGE_PMD_NR set_pte_at() loop, without wincing at the "freeze" case's
    HPAGE_PMD_NR page_remove_rmap() loop below it.
    
    It's just a mistake to add rmaps in the "freeze" (insert migration entries
    prior to splitting huge page) case: the pmd_migration case already avoids
    doing that, so just follow its lead.  page_add_ref() versus put_page()
    likewise.  But why is one more put_page() needed in the "freeze" case? 
    Because it's removing the pmd rmap, already removed when pmd_migration
    (and freeze and pmd_migration are mutually exclusive cases).
    
    Link: https://lkml.kernel.org/r/d43748aa-fece-e0b9-c4ab-f23c9ebc9011@google.comSigned-off-by: default avatarHugh Dickins <hughd@google.com>
    Acked-by: default avatarKirill A. Shutemov <kirill.shutemov@linux.intel.com>
    Cc: Dan Carpenter <error27@gmail.com>
    Cc: David Hildenbrand <david@redhat.com>
    Cc: James Houghton <jthoughton@google.com>
    Cc: Johannes Weiner <hannes@cmpxchg.org>
    Cc: John Hubbard <jhubbard@nvidia.com>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Matthew Wilcox <willy@infradead.org>
    Cc: Miaohe Lin <linmiaohe@huawei.com>
    Cc: Mike Kravetz <mike.kravetz@oracle.com>
    Cc: Mina Almasry <almasrymina@google.com>
    Cc: Muchun Song <songmuchun@bytedance.com>
    Cc: Naoya Horiguchi <naoya.horiguchi@linux.dev>
    Cc: Peter Xu <peterx@redhat.com>
    Cc: Sidhartha Kumar <sidhartha.kumar@oracle.com>
    Cc: Vlastimil Babka <vbabka@suse.cz>
    Cc: Yang Shi <shy828301@gmail.com>
    Cc: Yu Zhao <yuzhao@google.com>
    Cc: Zach O'Keefe <zokeefe@google.com>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    96d82deb
huge_memory.c 87.3 KB