Re: [PATCH v2 RESEND 05/18] x86, ACPICA: Split acpi_boot_table_init() into two parts.

2013-08-05 Thread Tang Chen
Hi Rafael, On 08/05/2013 09:26 PM, Rafael J. Wysocki wrote: .. I think I understand what you're trying to achieve and I don't have objections agaist the goal, but the matter is *how* to do that. Why don't you do something like this: (1) Introduce two new functions that will each do part of

Re: [PATCH v2 RESEND 05/18] x86, ACPICA: Split acpi_boot_table_init() into two parts.

2013-08-05 Thread Rafael J. Wysocki
On Monday, August 05, 2013 11:21:51 AM Tang Chen wrote: > Hi Rafael, > > On 08/02/2013 09:00 PM, Rafael J. Wysocki wrote: > .. > >> This patch splits acpi_boot_table_init() into two steps: > >> 1. Parse RSDT, which cannot be overrided, and initialize > >> acpi_gbl_root_table_list. (step 1

Re: [PATCH v2 RESEND 05/18] x86, ACPICA: Split acpi_boot_table_init() into two parts.

2013-08-04 Thread Tang Chen
Hi Rafael, On 08/02/2013 09:00 PM, Rafael J. Wysocki wrote: .. This patch splits acpi_boot_table_init() into two steps: 1. Parse RSDT, which cannot be overrided, and initialize acpi_gbl_root_table_list. (step 1 + 2 above) 2. Install all ACPI tables into acpi_gbl_root_table_list. (ste

Re: [PATCH v2 RESEND 05/18] x86, ACPICA: Split acpi_boot_table_init() into two parts.

2013-08-02 Thread Rafael J. Wysocki
On Friday, August 02, 2013 05:14:24 PM Tang Chen wrote: > In ACPI, SRAT(System Resource Affinity Table) contains NUMA info. > The memory affinities in SRAT record every memory range in the > system, and also, flags specifying if the memory range is > hotpluggable. > (Please refer to ACPI spec 5.0 5

[PATCH v2 RESEND 05/18] x86, ACPICA: Split acpi_boot_table_init() into two parts.

2013-08-02 Thread Tang Chen
In ACPI, SRAT(System Resource Affinity Table) contains NUMA info. The memory affinities in SRAT record every memory range in the system, and also, flags specifying if the memory range is hotpluggable. (Please refer to ACPI spec 5.0 5.2.16) memblock starts to work at very early time, and SRAT has n