![Vlastimil Babka](/assets/img/avatar_default.png)
The page_owner mechanism stores gfp_flags of an allocation and stack
trace that lead to it. During page migration, the original information
is practically replaced by the allocation of free page as the migration
target. Arguably this is less useful and might lead to all the
page_owner info for migratable pages gradually converge towards
compaction or numa balancing migrations. It has also lead to
inaccuracies such as one fixed by commit e2cfc91120
("mm/page_owner:
set correct gfp_mask on page_owner").
This patch thus introduces copying the page_owner info during migration.
However, since the fact that the page has been migrated from its
original place might be useful for debugging, the next patch will
introduce a way to track that information as well.
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
Acked-by: Michal Hocko <mhocko@suse.com>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Sasha Levin <sasha.levin@oracle.com>
Cc: "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Hugh Dickins <hughd@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
58 lines
1.6 KiB
C
58 lines
1.6 KiB
C
#ifndef __LINUX_PAGE_OWNER_H
|
|
#define __LINUX_PAGE_OWNER_H
|
|
|
|
#include <linux/jump_label.h>
|
|
|
|
#ifdef CONFIG_PAGE_OWNER
|
|
extern struct static_key_false page_owner_inited;
|
|
extern struct page_ext_operations page_owner_ops;
|
|
|
|
extern void __reset_page_owner(struct page *page, unsigned int order);
|
|
extern void __set_page_owner(struct page *page,
|
|
unsigned int order, gfp_t gfp_mask);
|
|
extern gfp_t __get_page_owner_gfp(struct page *page);
|
|
extern void __copy_page_owner(struct page *oldpage, struct page *newpage);
|
|
|
|
static inline void reset_page_owner(struct page *page, unsigned int order)
|
|
{
|
|
if (static_branch_unlikely(&page_owner_inited))
|
|
__reset_page_owner(page, order);
|
|
}
|
|
|
|
static inline void set_page_owner(struct page *page,
|
|
unsigned int order, gfp_t gfp_mask)
|
|
{
|
|
if (static_branch_unlikely(&page_owner_inited))
|
|
__set_page_owner(page, order, gfp_mask);
|
|
}
|
|
|
|
static inline gfp_t get_page_owner_gfp(struct page *page)
|
|
{
|
|
if (static_branch_unlikely(&page_owner_inited))
|
|
return __get_page_owner_gfp(page);
|
|
else
|
|
return 0;
|
|
}
|
|
static inline void copy_page_owner(struct page *oldpage, struct page *newpage)
|
|
{
|
|
if (static_branch_unlikely(&page_owner_inited))
|
|
__copy_page_owner(oldpage, newpage);
|
|
}
|
|
#else
|
|
static inline void reset_page_owner(struct page *page, unsigned int order)
|
|
{
|
|
}
|
|
static inline void set_page_owner(struct page *page,
|
|
unsigned int order, gfp_t gfp_mask)
|
|
{
|
|
}
|
|
static inline gfp_t get_page_owner_gfp(struct page *page)
|
|
{
|
|
return 0;
|
|
}
|
|
static inline void copy_page_owner(struct page *oldpage, struct page *newpage)
|
|
{
|
|
}
|
|
#endif /* CONFIG_PAGE_OWNER */
|
|
#endif /* __LINUX_PAGE_OWNER_H */
|