PATCH: Intel Xeon E5440 identified as p3

[Tristan Gingold]

On Apr 19, 2008, at 8:29 AM, Torbjorn Granlund wrote:
>
>   !   family = ((fms >> 8) & 15) | ((fms >> 16) & 0xf0);
>   !   model = ((fms >> 4) & 15) | ((fms >> 12) & 0xf0);
>        stepping = fms & 15;
>
> I don't think that is correct, the "extended family" field is 8 bits
> wide and should be added,to contatenated to the family field:
>
>       family = ((fms >> 8) & 15) | ((fms >> 20) & 0xff);
>       model = ((fms >> 4) & 15) | ((fms >> 12) & 0xf0);
>
> Yes, this is silly, but it is how cpuid is defined.

Right, so this should be a '+' instead of '|'.

Tristan.