On Fri, Jul 25, 2014 at 11:39:08AM +0100, Hanjun Guo wrote:

On 2014-7-24 23:21, Mark Rutland wrote:

quoted

On Thu, Jul 24, 2014 at 02:00:15PM +0100, Hanjun Guo wrote:

quoted

Move the initialization of cpu_logical_map(0) before acpi_boot_init()
to remove the duplicated initialization of cpu_logical_map(0).

It always make sense to initialise CPU0's logical map entry from the
hardware values, so you could do this earlier in the series, before you
introduce any ACPI code. Then you don't have the churn in acpi.c

ok, I will prepare a separate patch to do this before ACPI code, does it
make sense to you?

Yes. Just put that at the start of the series.

Thanks,
Mark.