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