Re: [PATCHSET] x86: unify x86_32 and 64 NUMA init paths, take#3

From: Thomas Gleixner
Date: Thu Dec 16 2010 - 11:56:33 EST


On Thu, 16 Dec 2010, Tejun Heo wrote:
> (cc'ing Shaohui)
> Hello, Thoams.
>
> On 12/11/2010 02:30 PM, Tejun Heo wrote:
> > This is the third take of unify-x86_32-and-64-NUMA-init-paths
> > patchset. Most changes from the last take[L] are to reflect comments
> > from tglx.
> >
> > * Rebased on top of tip/x86/apic-cleanup.
> >
> > * "x86: apic: Cleanup and simplify setup_local_APIC()" is already
> > merged into x86/apic-cleanup and dropped from this series.
> >
> > * Instead of repurposing apic->cpu_to_logical_apicid(), replace it
> > with a new 32bit specific callback
> > apic->x86_32_early_logical_apicid().
> >
> > * For consistency, in 0012, apic->apicid_to_node() is renamed to
> > x86_32_numa_cpu_node() and put inside CONFIG_X86_32.
> >
> > * NULL assignments for unimplemented apic ops dropped.
> >
> > * 0013 now explicitly describes why amd srat_detect_node() accesses
> > __apicid_to_node[] directly. Also, add a comment in the function
> > noting the ugliness.
> >
> > * Patch description updated in 0014.
> >
> > * Other misc updates.
>
> Do the above changes agreeable to you? Shaohui is wondering what to
> do with the conflicts between the NUMA hotplug simulator and this
> series, so...

I was looking with hpa into it, review still in progress.
--
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/