Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932858Ab3HGLOe (ORCPT ); Wed, 7 Aug 2013 07:14:34 -0400 Received: from cn.fujitsu.com ([222.73.24.84]:1839 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S932559Ab3HGLNm (ORCPT ); Wed, 7 Aug 2013 07:13:42 -0400 X-IronPort-AV: E=Sophos;i="4.89,832,1367942400"; d="scan'208";a="8144304" From: Tang Chen To: robert.moore@intel.com, lv.zheng@intel.com, rjw@sisk.pl, lenb@kernel.org, tglx@linutronix.de, mingo@elte.hu, hpa@zytor.com, akpm@linux-foundation.org, tj@kernel.org, trenn@suse.de, yinghai@kernel.org, jiang.liu@huawei.com, wency@cn.fujitsu.com, laijs@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, izumi.taku@jp.fujitsu.com, mgorman@suse.de, minchan@kernel.org, mina86@mina86.com, gong.chen@linux.intel.com, vasilis.liaskovitis@profitbricks.com, lwoodman@redhat.com, riel@redhat.com, jweiner@redhat.com, prarit@redhat.com, zhangyanfei@cn.fujitsu.com, yanghy@cn.fujitsu.com Cc: x86@kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-acpi@vger.kernel.org Subject: [PATCH v3 14/25] x86, acpi: Initialize acpi golbal root table list earlier. Date: Wed, 7 Aug 2013 18:52:05 +0800 Message-Id: <1375872736-4822-15-git-send-email-tangchen@cn.fujitsu.com> X-Mailer: git-send-email 1.7.11.7 In-Reply-To: <1375872736-4822-1-git-send-email-tangchen@cn.fujitsu.com> References: <1375872736-4822-1-git-send-email-tangchen@cn.fujitsu.com> X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/08/07 18:52:16, Serialize by Router on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/08/07 18:52:18, Serialize complete at 2013/08/07 18:52:18 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3563 Lines: 122 As the previous patches split the acpi_gbl_root_table_list initialization procedure into two steps: install and override, this patch does the "install" steps earlier, right after memblock is ready. In this way, we are able to find SRAT in firmware earlier. And then, we can prevent memblock from allocating hotpluggable memory for the kernel. Signed-off-by: Tang Chen Reviewed-by: Zhang Yanfei --- arch/x86/kernel/acpi/boot.c | 30 +++++++++++++++++------------- arch/x86/kernel/setup.c | 8 +++++++- include/linux/acpi.h | 1 + 3 files changed, 25 insertions(+), 14 deletions(-) diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c index ddb0bc1..30daefd 100644 --- a/arch/x86/kernel/acpi/boot.c +++ b/arch/x86/kernel/acpi/boot.c @@ -1497,6 +1497,23 @@ static struct dmi_system_id __initdata acpi_dmi_table_late[] = { {} }; +void __init early_acpi_boot_table_init(void) +{ + dmi_check_system(acpi_dmi_table); + + /* + * If acpi_disabled, bail out + */ + if (acpi_disabled) + return; + + /* + * Initialize the ACPI boot-time table parser. + */ + if (acpi_table_init_firmware()) + disable_acpi(); +} + /* * acpi_boot_table_init() and acpi_boot_init() * called from setup_arch(), always. @@ -1504,9 +1521,6 @@ static struct dmi_system_id __initdata acpi_dmi_table_late[] = { * 2. enumerates lapics * 3. enumerates io-apics * - * acpi_table_init() is separate to allow reading SRAT without - * other side effects. - * * side effects of acpi_boot_init: * acpi_lapic = 1 if LAPIC found * acpi_ioapic = 1 if IOAPIC found @@ -1518,22 +1532,12 @@ static struct dmi_system_id __initdata acpi_dmi_table_late[] = { void __init acpi_boot_table_init(void) { - dmi_check_system(acpi_dmi_table); - /* * If acpi_disabled, bail out */ if (acpi_disabled) return; - /* - * Initialize the ACPI boot-time table parser. - */ - if (acpi_table_init_firmware()) { - disable_acpi(); - return; - } - acpi_table_init_override(); acpi_check_multiple_madt(); diff --git a/arch/x86/kernel/setup.c b/arch/x86/kernel/setup.c index f8ec578..fdb5a26 100644 --- a/arch/x86/kernel/setup.c +++ b/arch/x86/kernel/setup.c @@ -1074,6 +1074,12 @@ void __init setup_arch(char **cmdline_p) memblock_x86_fill(); /* + * Parse the ACPI tables from firmware for possible boot-time SMP + * configuration. + */ + early_acpi_boot_table_init(); + + /* * The EFI specification says that boot service code won't be called * after ExitBootServices(). This is, in fact, a lie. */ @@ -1130,7 +1136,7 @@ void __init setup_arch(char **cmdline_p) io_delay_init(); /* - * Parse the ACPI tables for possible boot-time SMP configuration. + * Finish parsing the ACPI tables. */ acpi_boot_table_init(); diff --git a/include/linux/acpi.h b/include/linux/acpi.h index 44a3e5f..c5e7b2a 100644 --- a/include/linux/acpi.h +++ b/include/linux/acpi.h @@ -91,6 +91,7 @@ char * __acpi_map_table (unsigned long phys_addr, unsigned long size); void __acpi_unmap_table(char *map, unsigned long size); int early_acpi_boot_init(void); int acpi_boot_init (void); +void early_acpi_boot_table_init (void); void acpi_boot_table_init (void); int acpi_mps_check (void); int acpi_numa_init (void); -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/