[PATCH 05/10] mm: make page_alloc.c explicitly non-modular
The Makefile currently controlling compilation of this code is obj-y meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modularity so that when reading the driver there is no doubt it is builtin-only. We have to leave the module.h include though, as the file uses print_modules(). Since module_init translates to device_initcall in the non-modular case, the init ordering remains unchanged with this commit. However one could argue that subsys_initcall() would make more sense. Cc: Andrew Morton Cc: Mel Gorman Cc: Vlastimil Babka Cc: David Rientjes Cc: Michal Hocko Cc: Joonsoo Kim Cc: Johannes Weiner Cc: Sasha Levin Cc: linux...@kvack.org Signed-off-by: Paul Gortmaker --- mm/page_alloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/page_alloc.c b/mm/page_alloc.c index c1024db4ac5f..b02c511320fa 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -6327,7 +6327,7 @@ int __meminit init_per_zone_wmark_min(void) setup_per_zone_inactive_ratio(); return 0; } -module_init(init_per_zone_wmark_min) +device_initcall(init_per_zone_wmark_min) /* * min_free_kbytes_sysctl_handler - just a wrapper around proc_dointvec() so -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 05/10] mm: make page_alloc.c explicitly non-modular
The Makefile currently controlling compilation of this code is obj-y meaning that it currently is not being built as a module by anyone. Lets remove the couple traces of modularity so that when reading the driver there is no doubt it is builtin-only. We have to leave the module.h include though, as the file uses print_modules(). Since module_init translates to device_initcall in the non-modular case, the init ordering remains unchanged with this commit. However one could argue that subsys_initcall() would make more sense. Cc: Andrew Morton a...@linux-foundation.org Cc: Mel Gorman mgor...@suse.de Cc: Vlastimil Babka vba...@suse.cz Cc: David Rientjes rient...@google.com Cc: Michal Hocko mho...@suse.cz Cc: Joonsoo Kim iamjoonsoo@lge.com Cc: Johannes Weiner han...@cmpxchg.org Cc: Sasha Levin sasha.le...@oracle.com Cc: linux...@kvack.org Signed-off-by: Paul Gortmaker paul.gortma...@windriver.com --- mm/page_alloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/page_alloc.c b/mm/page_alloc.c index c1024db4ac5f..b02c511320fa 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -6327,7 +6327,7 @@ int __meminit init_per_zone_wmark_min(void) setup_per_zone_inactive_ratio(); return 0; } -module_init(init_per_zone_wmark_min) +device_initcall(init_per_zone_wmark_min) /* * min_free_kbytes_sysctl_handler - just a wrapper around proc_dointvec() so -- 2.5.0 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/