On Mon, 15 Apr 2013 17:46:46 +0800 Tang Chen <tangchen@xxxxxxxxxxxxxx> wrote: > kernel_physical_mapping_remove() is only called by arch_remove_memory() in > init_64.c, which is enclosed in CONFIG_MEMORY_HOTREMOVE. So when we don't > configure CONFIG_MEMORY_HOTREMOVE, the compiler will give a warning: > > warning: ___kernel_physical_mapping_remove___ defined but not used > > So put kernel_physical_mapping_remove() in CONFIG_MEMORY_HOTREMOVE. > > Signed-off-by: Tang Chen <tangchen@xxxxxxxxxxxxxx> > --- > arch/x86/mm/init_64.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c > index 474e28f..dafdeb2 100644 > --- a/arch/x86/mm/init_64.c > +++ b/arch/x86/mm/init_64.c > @@ -1019,6 +1019,7 @@ void __ref vmemmap_free(struct page *memmap, unsigned long nr_pages) > remove_pagetable(start, end, false); > } > > +#ifdef CONFIG_MEMORY_HOTREMOVE > static void __meminit > kernel_physical_mapping_remove(unsigned long start, unsigned long end) > { > @@ -1028,7 +1029,6 @@ kernel_physical_mapping_remove(unsigned long start, unsigned long end) > remove_pagetable(start, end, true); > } > > -#ifdef CONFIG_MEMORY_HOTREMOVE > int __ref arch_remove_memory(u64 start, u64 size) > { > unsigned long start_pfn = start >> PAGE_SHIFT; Thanks. I already have this one queued in http://ozlabs.org/~akpm/mmots/broken-out/arch-x86-mm-init_64c-fix-build-warning-when-config_memory_hotremove=n.patch, within my "send it to the x86 maintainers" section. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>