On Mon, Feb 28, 2011 at 11:23 AM, Mike Travis <travis@xxxxxxx> wrote:
Yinghai Lu wrote:On 02/27/2011 04:15 AM, Ingo Molnar wrote:I had to debug a weird ACPI -> Node mapping last week and the* Ingo Molnar <mingo@xxxxxxx> wrote:Mike,
You could avoid all this ugly workaround of bootmem limitations bymemblock_alloc() could return -ENOSYS on architectures that do not
moving the allocation to memblock_alloc() and desupporting the log_buf_len=
boot parameter on non-memblock architectures.
implement it - thus enabling such optional features without ugly #ifdef
CONFIG_HAVE_MEMBLOCK conditionals.
please check updated patch...
with the memblock change, you don't need to change acpi SRAT handling etc
any more.
"improved" SRAT messages helped that considerably. It was
far easier to spot which Node didn't have the correct assignments.
I'd submit that patch even without needing fewer (like 512 lines
max instead of 4096 lines max) bytes in the log buffer.
Your current change to ACPI srat is not complete yet.
you only handle x2apic entries.
According to ACPI 4.0 spec, We should have mixed entries with apic
entries and x2apic entries.
apic entries are for apic id < 255.
x2apic entries are for apic id > 255.
Yinghai