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

From: Guenter Roeck
Date: Tue Oct 13 2015 - 18:26:37 EST


On 10/13/2015 03:02 PM, Odzioba, Lukasz wrote:
On Tuesday, October 12, 2015 at 10:32 PM, Guenter Roeck wrote:
Why 128 instead of a more reasonable 64 ? What is the required minimum
for Xeon Phi ?

It would be fine today, but it will be not enough in 2016 and we would like to
give GNU/Linux distributions some time to propagate this patch.
For Knights Corner which is already on the market we need 61.
For Knights Landing we need 72.

Pardon my ignorance ... those are Xeon Phi processors, and support up to
244 threads (for Knights Corner). Programming datasheet isn't easily available,
so I have to guess a bit. Following the processor numbering scheme of "ordinary"
processors, the CPU ID can therefore be up to 244 (at least) already today,
meaning the limit would have to be 256 (assuming that the processor does support
per-core temperature sensors). On the other side, the public datasheet suggests
that there are only three temperature sensors.

What am I missing here ?

Thanks,
Guenter

For future generation it is not yet public information.

Also, please consider using the subject line commonly used in hwmon,
or at least point to the driver you are changing.

Right, I forgot this time, I'll send another patch just tell me whether 128
is ok, or please select any other value greater or equal known minimum
that you think will be more appropriate.

Thanks,
Lukas




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