diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2024-05-08 22:53:47 +0300 |
---|---|---|
committer | Borislav Petkov (AMD) <bp@alien8.de> | 2024-05-10 18:42:50 +0300 |
commit | 5754ace3c3199c162dcee1f3f87a538c46d1c832 (patch) | |
tree | 8d1df87ac7012a4c5ab0a09c46ee5350d79af6f7 /Documentation/devicetree/bindings/net/mediatek,net.yaml | |
parent | 0e640f0a47d8426eab1fb9c03f0af898dfe810b8 (diff) | |
download | linux-5754ace3c3199c162dcee1f3f87a538c46d1c832.tar.xz |
x86/topology/amd: Ensure that LLC ID is initialized
The original topology evaluation code initialized cpu_data::topo::llc_id
with the die ID initialy and then eventually overwrite it with information
gathered from a CPUID leaf.
The conversion analysis failed to spot that particular detail and omitted
this initial assignment under the assumption that each topology evaluation
path will set it up. That assumption is mostly correct, but turns out to be
wrong in case that the CPUID leaf 0x80000006 does not provide a LLC ID.
In that case, LLC ID is invalid and as a consequence the setup of the
scheduling domain CPU masks is incorrect which subsequently causes the
scheduler core to complain about it during CPU hotplug:
BUG: arch topology borken
the CLS domain not a subset of the MC domain
Cure it by reusing legacy_set_llc() and assigning the die ID if the LLC ID
is invalid after all possible parsers have been tried.
Fixes: f7fb3b2dd92c ("x86/cpu: Provide an AMD/HYGON specific topology parser")
Reported-by: Yuezhang Mo <Yuezhang.Mo@sony.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Tested-by: Yuezhang Mo <Yuezhang.Mo@sony.com>
Link: https://lore.kernel.org/r/PUZPR04MB63168AC442C12627E827368581292@PUZPR04MB6316.apcprd04.prod.outlook.com
Diffstat (limited to 'Documentation/devicetree/bindings/net/mediatek,net.yaml')
0 files changed, 0 insertions, 0 deletions