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