PATCH: Intel Xeon E5440 identified as p3

[Torbjorn Granlund]

Tristan Gingold <gingold at adacore.com> writes:

  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 '|'.
  
Right.  

-- 
Torbjörn