From patchwork Tue Feb 5 02:37:16 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: li guang X-Patchwork-Id: 2095911 Return-Path: X-Original-To: patchwork-linux-acpi@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 7EE813FC23 for ; Tue, 5 Feb 2013 02:37:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754780Ab3BEChj (ORCPT ); Mon, 4 Feb 2013 21:37:39 -0500 Received: from cn.fujitsu.com ([222.73.24.84]:19171 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1755106Ab3BEChi (ORCPT ); Mon, 4 Feb 2013 21:37:38 -0500 X-IronPort-AV: E=Sophos;i="4.84,603,1355068800"; d="scan'208";a="6689141" Received: from unknown (HELO tang.cn.fujitsu.com) ([10.167.250.3]) by song.cn.fujitsu.com with ESMTP; 05 Feb 2013 10:35:23 +0800 Received: from fnstmail02.fnst.cn.fujitsu.com (tang.cn.fujitsu.com [127.0.0.1]) by tang.cn.fujitsu.com (8.14.3/8.13.1) with ESMTP id r152bZfQ003449; Tue, 5 Feb 2013 10:37:35 +0800 Received: from liguang.fnst.cn.fujitsu.com ([10.167.225.128]) by fnstmail02.fnst.cn.fujitsu.com (Lotus Domino Release 8.5.3) with ESMTP id 2013020510361982-295381 ; Tue, 5 Feb 2013 10:36:19 +0800 From: liguang To: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, x86@kernel.org Cc: rientjes@google.com, liguang Subject: [PATCH v2 2/4] numa: avoid export acpi_numa variable Date: Tue, 5 Feb 2013 10:37:16 +0800 Message-Id: <1360031838-26898-3-git-send-email-lig.fnst@cn.fujitsu.com> X-Mailer: git-send-email 1.7.2.5 In-Reply-To: <1360031838-26898-1-git-send-email-lig.fnst@cn.fujitsu.com> References: <1360031838-26898-1-git-send-email-lig.fnst@cn.fujitsu.com> X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/02/05 10:36:19, Serialize by Router on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/02/05 10:36:20, Serialize complete at 2013/02/05 10:36:20 Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org acpi_numa is used to prevent srat table being parsed, seems a little miss-named, if 'noacpi' was specified by cmdline and CONFIG_ACPI_NUMA was enabled, acpi_numa will be operated directly from everywhere it needed to disable/enable numa in acpi mode which was a bad thing, so, try to export a fuction to get srat table enable/disable info. Reviewed-by: David Rientjes Signed-off-by: liguang --- arch/x86/include/asm/acpi.h | 2 +- arch/x86/kernel/acpi/srat.c | 15 ++++++++++----- arch/x86/mm/numa.c | 2 +- arch/x86/xen/enlighten.c | 2 +- 4 files changed, 13 insertions(+), 8 deletions(-) diff --git a/arch/x86/include/asm/acpi.h b/arch/x86/include/asm/acpi.h index 0c44630..6b9c861 100644 --- a/arch/x86/include/asm/acpi.h +++ b/arch/x86/include/asm/acpi.h @@ -181,7 +181,7 @@ static inline void disable_acpi(void) { } #define ARCH_HAS_POWER_INIT 1 #ifdef CONFIG_ACPI_NUMA -extern int acpi_numa; +extern void disable_acpi_numa(void); extern int x86_acpi_numa_init(void); #endif /* CONFIG_ACPI_NUMA */ diff --git a/arch/x86/kernel/acpi/srat.c b/arch/x86/kernel/acpi/srat.c index 4ddf497..0a4d7ee 100644 --- a/arch/x86/kernel/acpi/srat.c +++ b/arch/x86/kernel/acpi/srat.c @@ -24,22 +24,27 @@ #include #include -int acpi_numa __initdata; +static bool acpi_numa __initdata; static __init int setup_node(int pxm) { return acpi_map_pxm_to_node(pxm); } -static __init void bad_srat(void) +void __init disable_acpi_numa(void) +{ + acpi_numa = false; +} + +static void __init bad_srat(void) { - printk(KERN_ERR "SRAT: SRAT not used.\n"); acpi_numa = -1; + printk(KERN_ERR "SRAT: SRAT will not be used.\n"); } -static __init inline int srat_disabled(void) +static bool __init srat_disabled(void) { - return acpi_numa < 0; + return acpi_numa; } /* Callback for SLIT parsing */ diff --git a/arch/x86/mm/numa.c b/arch/x86/mm/numa.c index 2d125be..870ca6b 100644 --- a/arch/x86/mm/numa.c +++ b/arch/x86/mm/numa.c @@ -47,7 +47,7 @@ static __init int numa_setup(char *opt) #endif #ifdef CONFIG_ACPI_NUMA if (!strncmp(opt, "noacpi", 6)) - acpi_numa = -1; + disable_acpi_numa(); #endif return 0; } diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c index 138e566..a5f6353 100644 --- a/arch/x86/xen/enlighten.c +++ b/arch/x86/xen/enlighten.c @@ -1415,7 +1415,7 @@ asmlinkage void __init xen_start_kernel(void) * any NUMA information the kernel tries to get from ACPI will * be meaningless. Prevent it from trying. */ - acpi_numa = -1; + disable_acpi_numa(); #endif /* Don't do the full vcpu_info placement stuff until we have a