From patchwork Mon Feb 4 08:43:42 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: li guang X-Patchwork-Id: 2089691 Return-Path: X-Original-To: patchwork-linux-acpi@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 05B75DFE82 for ; Mon, 4 Feb 2013 08:45:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754175Ab3BDIon (ORCPT ); Mon, 4 Feb 2013 03:44:43 -0500 Received: from cn.fujitsu.com ([222.73.24.84]:30215 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1753969Ab3BDIoL (ORCPT ); Mon, 4 Feb 2013 03:44:11 -0500 X-IronPort-AV: E=Sophos;i="4.84,598,1355068800"; d="scan'208";a="6686576" Received: from unknown (HELO tang.cn.fujitsu.com) ([10.167.250.3]) by song.cn.fujitsu.com with ESMTP; 04 Feb 2013 16:41:55 +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 r148i6ij008271; Mon, 4 Feb 2013 16:44:07 +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 2013020416425222-272211 ; Mon, 4 Feb 2013 16:42:52 +0800 From: liguang To: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, x86@kernel.org Cc: liguang Subject: [PATCH 1/5] numa: avoid export acpi_numa variable Date: Mon, 4 Feb 2013 16:43:42 +0800 Message-Id: <1359967426-32629-2-git-send-email-lig.fnst@cn.fujitsu.com> X-Mailer: git-send-email 1.7.2.5 In-Reply-To: <1359967426-32629-1-git-send-email-lig.fnst@cn.fujitsu.com> References: <1359967426-32629-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/04 16:42:52, Serialize by Router on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/02/04 16:42:52, Serialize complete at 2013/02/04 16:42:52 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. Signed-off-by: liguang --- arch/x86/include/asm/acpi.h | 3 ++- arch/x86/mm/numa.c | 2 +- arch/x86/mm/srat.c | 10 +++++----- arch/x86/xen/enlighten.c | 2 +- 4 files changed, 9 insertions(+), 8 deletions(-) diff --git a/arch/x86/include/asm/acpi.h b/arch/x86/include/asm/acpi.h index 0c44630..c4e0875 100644 --- a/arch/x86/include/asm/acpi.h +++ b/arch/x86/include/asm/acpi.h @@ -181,7 +181,8 @@ static inline void disable_acpi(void) { } #define ARCH_HAS_POWER_INIT 1 #ifdef CONFIG_ACPI_NUMA -extern int acpi_numa; +extern void bad_srat(void); +extern bool srat_disabled(void); extern int x86_acpi_numa_init(void); #endif /* CONFIG_ACPI_NUMA */ diff --git a/arch/x86/mm/numa.c b/arch/x86/mm/numa.c index 2d125be..92986d8 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; + bad_srat(); #endif return 0; } diff --git a/arch/x86/mm/srat.c b/arch/x86/mm/srat.c index 4ddf497..a837c95 100644 --- a/arch/x86/mm/srat.c +++ b/arch/x86/mm/srat.c @@ -24,22 +24,22 @@ #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 bad_srat(void) { printk(KERN_ERR "SRAT: SRAT not used.\n"); - acpi_numa = -1; + acpi_numa = false; } -static __init inline int srat_disabled(void) +bool __init srat_disabled(void) { - return acpi_numa < 0; + return acpi_numa; } /* Callback for SLIT parsing */ diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c index 138e566..94c8c70 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; + bad_srat(); #endif /* Don't do the full vcpu_info placement stuff until we have a