Commit cab0a7c1 authored by Ting Liu's avatar Ting Liu Committed by Linus Torvalds
Browse files

mm: make some vars and functions static or __init 

"page_idle_ops" as a global var, but its scope of use within this
document.  So it should be static.

"page_ext_ops" is a var used in the kernel initial phase.  And other
functions are aslo used in the kernel initial phase.  So they should be
__init or __initdata to reclaim memory.

Link: https://lkml.kernel.org/r/20211217095023.67293-1-liuting.0x7c00@bytedance.com


Signed-off-by: default avatarTing Liu <liuting.0x7c00@bytedance.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 0b8f0d87

include/linux/page_idle.h

+0 −1
Original line number Diff line number Diff line
@@ -13,7 +13,6 @@ 
 * If there is not enough space to store Idle and Young bits in page flags, use

 * page ext flags instead.

 */

extern struct page_ext_operations page_idle_ops;



static inline bool folio_test_young(struct folio *folio)

{

mm/page_ext.c

+2 −2
Original line number Diff line number Diff line
@@ -64,12 +64,12 @@ static bool need_page_idle(void) 
{

	return true;

}

struct page_ext_operations page_idle_ops = {

static struct page_ext_operations page_idle_ops __initdata = {

	.need = need_page_idle,

};

#endif



static struct page_ext_operations *page_ext_ops[] = {

static struct page_ext_operations *page_ext_ops[] __initdata = {

#ifdef CONFIG_PAGE_OWNER

	&page_owner_ops,

#endif

mm/page_owner.c

+2 −2
Original line number Diff line number Diff line
@@ -46,7 +46,7 @@ static int __init early_page_owner_param(char *buf) 
}

early_param("page_owner", early_page_owner_param);



static bool need_page_owner(void)

static __init bool need_page_owner(void)

{

	return page_owner_enabled;

}
@@ -75,7 +75,7 @@ static noinline void register_early_stack(void) 
	early_handle = create_dummy_stack();

}



static void init_page_owner(void)

static __init void init_page_owner(void)

{

	if (!page_owner_enabled)

		return;