Re: [PATCH 05/17] x86-64: Move cpu number from PDA to per-cpu and consolidate with 32-bit.

From: Tejun Heo
Date: Sun Jan 18 2009 - 00:59:46 EST


Brian Gerst wrote:
> On Sun, Jan 18, 2009 at 12:05 AM, Tejun Heo <tj@xxxxxxxxxx> wrote:
>> Hello,
>>
>>> --- a/arch/x86/kernel/setup_percpu.c
>>> +++ b/arch/x86/kernel/setup_percpu.c
>>> @@ -147,6 +147,9 @@ unsigned long __per_cpu_offset[NR_CPUS] __read_mostly;
>>> #endif
>>> EXPORT_SYMBOL(__per_cpu_offset);
>>>
>>> +DEFINE_PER_CPU(int, cpu_number);
>>> +EXPORT_PER_CPU_SYMBOL(cpu_number);
>> This is inside CONFIG_HAVE_SETUP_PER_CPU_AREA. I think voyage would
>> be unhappy with this change.
>
> Is there any specific reason Voyager doesn't use the x86
> setup_per_cpu_areas() function? I don't see anything on a quick
> glance that would not work. The x86 code is pretty much a superset of
> the default code in init/main.c.

I have no idea at all. Given that not many people can test it, I
figured just leaving it alone would be the best course but if it can
be merged, all the better.

Thanks.

--
tejun
--
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/