On Mon, 15 Apr 2013 17:46:46 +0800 Tang Chen <tangc...@cn.fujitsu.com> 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 <tangc...@cn.fujitsu.com> > --- > 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 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/