Re: [patch V4 00/41] x86/cpu: Rework the topology evaluation
From: Zhang, Rui
Date: Wed Aug 16 2023 - 07:38:09 EST
On Mon, 2023-08-14 at 10:53 +0200, Thomas Gleixner wrote:
> Hi!
>
> This is the follow up to V3:
>
> https://lore.kernel.org/lkml/20230802101635.459108805@xxxxxxxxxxxxx
>
> which addresses the review feedback.
>
> TLDR:
>
> This reworks the way how topology information is evaluated via CPUID
> in preparation for a larger topology management overhaul to address
> shortcomings of the current code vs. hybrid systems and systems which
> make
> use of the extended topology domains in leaf 0x1f. Aside of that it's
> an
> overdue spring cleaning to get rid of accumulated layers of duct tape
> and
> haywire.
>
> What changed vs. V3:
>
> - Added the Hygon fix from Pu Wen and adjusted the new code
> accordingly
>
> - Fixed the off by one in the AMD parser for real - Michael
>
> - Reworked the unknown domain type handling in the 0xb/01f parser -
> Rui
>
> - Made core ID package relative - Rui
>
> - Folded the missing u32 conversions - Qiuxu
>
> - Folded the fake_topology() fixup from the full topology series
>
> - Small cleanups and enhancements
>
> - Picked up Tested-by tags
>
> The series is based on the the APIC cleanup series in tip:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/apic
>
> and also available on top of that from git:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tglx/devel.git topo-
> cpuid-v4
>
>
Although there is still discussion ongoing, I tested this patch series
on two different platforms with Module Type enumerated, and the lscpu
output as well as the topology sysfs look good, so
Tested-by: Zhang Rui <rui.zhang@xxxxxxxxx>
thanks,
rui