Re: [PATCH] remove empty node at boot time

[KAMEZAWA Hiroyuki]

On Sun, 9 Jul 2006 20:38:40 -0600
Bjorn Helgaas <bjorn.helgaas@xxxxxx> wrote:
> Yes.  Here's the relevant part of the call tree:
> 
> setup_arch
>   acpi_numa_init
>     acpi_numa_arch_fixup
>       acpi_online_node_fixup (test available_cpus)
>   ... 
>   acpi_boot_init
>     acpi_table_parse_madt(..., acpi_parse_lsapic, ...)
>       acpi_parse_lsapic (increment available_cpus)
> 
> Note that we test available_cpus in acpi_online_node_fixup()
> before we increment it in acpi_parse_lsapic(), so the inner
> loop is never executed.

Hmm...okay, I misunderstood the boot path..
To work with my remove-empty-node patch, parsing lsapic should be done
before SRAT. 
I'd like to fix this. BTW, can we move parsing MADT before SRAT ?

-Kame


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/