• Zach O'Keefe's avatar
    mm/khugepaged: dedup and simplify hugepage alloc and charging · 9710a78a
    Zach O'Keefe authored
    The following code is duplicated in collapse_huge_page() and
    collapse_file():
    
            gfp = alloc_hugepage_khugepaged_gfpmask() | __GFP_THISNODE;
    
    	new_page = khugepaged_alloc_page(hpage, gfp, node);
            if (!new_page) {
                    result = SCAN_ALLOC_HUGE_PAGE_FAIL;
                    goto out;
            }
    
            if (unlikely(mem_cgroup_charge(page_folio(new_page), mm, gfp))) {
                    result = SCAN_CGROUP_CHARGE_FAIL;
                    goto out;
            }
            count_memcg_page_event(new_page, THP_COLLAPSE_ALLOC);
    
    Also, "node" is passed as an argument to both collapse_huge_page() and
    collapse_file() and obtained the same way, via
    khugepaged_find_target_node().
    
    Move all this into a new helper, alloc_charge_hpage(), and remove the
    duplicate code from collapse_huge_page() and collapse_file().  Also,
    simplify khugepaged_alloc_page() by returning a bool indicating allocation
    success instead of a copy of the allocated struct page *.
    
    Link: https://lkml.kernel.org/r/20220706235936.2197195-5-zokeefe@google.comSigned-off-by: default avatarZach O'Keefe <zokeefe@google.com>
    Suggested-by: default avatarPeter Xu <peterx@redhat.com>
    Acked-by: default avatarDavid Rientjes <rientjes@google.com>
    Reviewed-by: default avatarYang Shi <shy828301@gmail.com>
    Cc: Alex Shi <alex.shi@linux.alibaba.com>
    Cc: Andrea Arcangeli <aarcange@redhat.com>
    Cc: Arnd Bergmann <arnd@arndb.de>
    Cc: Axel Rasmussen <axelrasmussen@google.com>
    Cc: Chris Kennelly <ckennelly@google.com>
    Cc: Chris Zankel <chris@zankel.net>
    Cc: David Hildenbrand <david@redhat.com>
    Cc: Helge Deller <deller@gmx.de>
    Cc: Hugh Dickins <hughd@google.com>
    Cc: Ivan Kokshaysky <ink@jurassic.park.msu.ru>
    Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
    Cc: Jens Axboe <axboe@kernel.dk>
    Cc: "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>
    Cc: Matthew Wilcox <willy@infradead.org>
    Cc: Matt Turner <mattst88@gmail.com>
    Cc: Max Filippov <jcmvbkbc@gmail.com>
    Cc: Miaohe Lin <linmiaohe@huawei.com>
    Cc: Michal Hocko <mhocko@suse.com>
    Cc: Minchan Kim <minchan@kernel.org>
    Cc: Pasha Tatashin <pasha.tatashin@soleen.com>
    Cc: Pavel Begunkov <asml.silence@gmail.com>
    Cc: Rongwei Wang <rongwei.wang@linux.alibaba.com>
    Cc: SeongJae Park <sj@kernel.org>
    Cc: Song Liu <songliubraving@fb.com>
    Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
    Cc: Vlastimil Babka <vbabka@suse.cz>
    Cc: Zi Yan <ziy@nvidia.com>
    Cc: Dan Carpenter <dan.carpenter@oracle.com>
    Cc: "Souptick Joarder (HPE)" <jrdr.linux@gmail.com>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    9710a78a
khugepaged.c 57.8 KB