On Fri, Feb 26, 2021 at 11:12:17AM +0100, Karel Gardas wrote:
> On 2/26/21 7:24 AM, Jonathan Gray wrote:
> > As the ids are used on more than just Skylake-E here is another diff.
> > Though I think these ids are shared with Core X Skylake.  So perhaps
> > giving up on a marketing name is indeed the thing to do.
> 
> Indeed, Intel made quite a mess in this, but I think this your combination
> is probably the best one,
> although we risk to show "Xeon" on user system with high-end Core X cpus.
> The thing
> is those makes minority of all the chips in the family and technically they
> are crippled Xeons anyway,
> so if I may I would vote for this diff.
> 
> Thanks a lot for dealing with this mess.

thanks, committed

> 
> > Index: pcidevs
> > ===================================================================
> > RCS file: /cvs/src/sys/dev/pci/pcidevs,v
> > retrieving revision 1.1956
> > diff -u -p -r1.1956 pcidevs
> > --- pcidevs 22 Feb 2021 01:17:23 -0000      1.1956
> > +++ pcidevs 26 Feb 2021 05:39:05 -0000
> > @@ -4188,6 +4188,61 @@ product INTEL ATOMC2000_PCU_SMB      0x1f3c  A
> >   product INTEL I354_BP_1GBPS       0x1f40  I354
> >   product INTEL I354_SGMII  0x1f41  I354 SGMII
> >   product INTEL I354_BP_2_5GBPS     0x1f45  I354
> > +product INTEL XEONS_UBOX_1 0x2014  Xeon Scalable Ubox
> > +product INTEL XEONS_UBOX_2 0x2015  Xeon Scalable Ubox
> > +product INTEL XEONS_UBOX_3 0x2016  Xeon Scalable Ubox
> > 
> 
> 

Reply via email to