Re: [PATCH for review] [140/145] i386: mark cpu_dev structures as __cpuinitdata
From: Chuck Ebbert
Date: Fri Aug 11 2006 - 11:26:43 EST
On Thu, 10 Aug 2006 21:37:40 +0200, Andi Kleen wrote:
> From: Magnus Damm <magnus@xxxxxxxxxxxxx>
> The different cpu_dev structures are all used from __cpuinit callers what
> I can tell. So mark them as __cpuinitdata instead of __initdata. I am a
> little bit unsure about arch/i386/common.c:default_cpu, especially when it
> comes to the purpose of this_cpu.
But none of these CPUs supports hotplug and only one (AMD) does SMP.
So this is just wasting space in the kernel at runtime.
If anything I would only do this for AMD.
Same for the other patch that does more of this kind of change.
(IIRC I tried to do this a while ago and was told not to.)
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/