Re: [lm-sensors] [PATCH 1/1] Bumps limit of maximum core ID from 32 to 128.

Guenter Roeck
2015-10-13T21:21:55Z

On 10/13/2015 02:05 PM, Phil Pokorny wrote:
> On Tue, Oct 13, 2015 at 1:31 PM, Guenter Roeck <linux@roeck-us.net> wrote:
>>
>> On Mon, Oct 12, 2015 at 01:53:32PM +0200, Lukasz Odzioba wrote:
>>> A new limit selected arbitrarily as power of two greater than
>>> required minimum for Xeon Phi processor.
>
>> Why 128 instead of a more reasonable 64 ? What is the required minimum
>> for Xeon Phi ?
>
> Not meaning to be snarky, but this was answered in the first sentence.
> 64 is less than the required minimum for Xeon Phi processor. So it
> must be 65 or greater...
>

That is an assumption, not an answer, sorry.

Guenter

> I wouldn't expect Intel to give you any more detail than that. And it
> might be that 64 is actually enough for now but would soon (months or
> less than a year) be overrun by a newer processor. So rather than
> submit multiple minor patches, just submit one now that should be
> "enough"
>
> If you think this is a waste of RAM, we could make it a kernel
> configuration option and let it be configured by the distro or user.
> But if most distros select 128 to be able to support Xeon Phi, then
> there might not be a reason for the additional complexity.
>
> Phil P.
>