Re: [PATCH -v2 5/5] x86: use dmi check to treat disabled cpus ashotplug cpus.

[H. Peter Anvin]

On 01/13/2010 03:02 PM, Yinghai Lu wrote:
> On 01/13/2010 02:49 PM, Suresh Siddha wrote:
>> On Wed, 2010-01-13 at 14:36 -0800, H. Peter Anvin wrote:
>>> On 01/13/2010 02:29 PM, Suresh Siddha wrote:
>>>>>
>>>>> Well, that *is* working around broken code, in this case the broken code
>>>>> is the percpu allocation strategy.
>>>>
>>>> Andi, Recently percpu folks changed the per-cpu static first chunk to
>>>> PMD SIZE right. I think that is what causing all this issue.
>>>>
>>>
>>> Please don't tell me we're allocating 2 MB per CPU and throwing away
>>> most of it...
>>
>> Looking at the percpu code, they do seem to free the unused memory in
>> that hole.
>>
>> Yinghai, what are the configurations that need MB's of per cpu area?
> 
> please check attached...
> 

I just built an allyesconfig kernel, and it has 1904K of percpu data.
1791K of that is in per_cpu__cpu_lock_stats.  So everything else is
~113K.  Still.  We should not be restricted from doing legitimately
large percpu allocations because of ghost cpus (and Tejun knows my
position on this), but it doesn't sound like the house is burning, either.

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