Re: [patch V4 00/41] x86/cpu: Rework the topology evaluation

["Zhang, Rui" <rui.zhang@intel.com>]

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@linutronix.de
> 
> 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@intel.com>

thanks,
rui