From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=none Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7FA1D10DB; Thu, 30 Nov 2023 08:48:24 -0800 (PST) Received: from mail.maildlp.com (unknown [172.18.186.216]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4Sh2CS2XSVz6K8vh; Fri, 1 Dec 2023 00:46:44 +0800 (CST) Received: from lhrpeml500005.china.huawei.com (unknown [7.191.163.240]) by mail.maildlp.com (Postfix) with ESMTPS id 045821402CD; Fri, 1 Dec 2023 00:48:21 +0800 (CST) Received: from localhost (10.202.227.76) by lhrpeml500005.china.huawei.com (7.191.163.240) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Thu, 30 Nov 2023 16:48:20 +0000 Date: Thu, 30 Nov 2023 16:48:19 +0000 From: Jonathan Cameron To: "Russell King (Oracle)" CC: , , , , , , , , , , , , , Salil Mehta , Jean-Philippe Brucker , , , James Morse , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , "H. Peter Anvin" Subject: Re: [PATCH 03/21] x86/topology: remove arch_*register_cpu() exports Message-ID: <20231130164819.00002f75@Huawei.com> In-Reply-To: References: Organization: Huawei Technologies Research and Development (UK) Ltd. X-Mailer: Claws Mail 4.1.0 (GTK 3.24.33; x86_64-w64-mingw32) Precedence: bulk X-Mailing-List: linux-arch@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-ClientProxiedBy: lhrpeml100005.china.huawei.com (7.191.160.25) To lhrpeml500005.china.huawei.com (7.191.163.240) On Tue, 21 Nov 2023 13:44:05 +0000 "Russell King (Oracle)" wrote: > arch_register_cpu() and arch_unregister_cpu() are not used by anything > that can be a module - they are used by drivers/base/cpu.c and > drivers/acpi/acpi_processor.c, neither of which can be a module. > > Remove the exports. > > Reviewed-by: Gavin Shan > Signed-off-by: Russell King (Oracle) I somehow ended up replying to the RFC even though you'd posted this. On basis this might get picked up directly from this posting. Reviewed-by: Jonathan Cameron > --- > arch/x86/kernel/topology.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/arch/x86/kernel/topology.c b/arch/x86/kernel/topology.c > index 0bab03130033..fcb62cfdf946 100644 > --- a/arch/x86/kernel/topology.c > +++ b/arch/x86/kernel/topology.c > @@ -45,13 +45,11 @@ int arch_register_cpu(int cpu) > xc->cpu.hotpluggable = cpu > 0; > return register_cpu(&xc->cpu, cpu); > } > -EXPORT_SYMBOL(arch_register_cpu); > > void arch_unregister_cpu(int num) > { > unregister_cpu(&per_cpu(cpu_devices, num).cpu); > } > -EXPORT_SYMBOL(arch_unregister_cpu); > #else /* CONFIG_HOTPLUG_CPU */ > > int __init arch_register_cpu(int num)