Messages in this thread | | | From | Thomas Gleixner <> | Subject | Re: [patch V3 27/40] x86/cpu: Provide a sane leaf 0xb/0x1f parser | Date | Mon, 14 Aug 2023 14:26:57 +0200 |
| |
> On Sun, 2023-08-13 at 17:04 +0200, Thomas Gleixner wrote: > > With this, we set dom_offset[DIE] to 7 first when parsing TILE, and > then overwrite it to 8 when parsing UBER_TILE, and set > dom_offset[PACKAGE] to 9 when parsinig DIE. > > lossing TILE.eax.shifts is okay, because it is for UBER_TILE id.
No. That's just wrong. TILE is defined and potentially used in the kernel. How can you rightfully assume that UBER TILE is a valid substitution? You can't.
> Currently, die topology information is mandatory in Linux, we cannot > make it right without patching enum topo_types/enum > x86_topology_domains/topo_domain_map (which in fact tells the > relationship between DIE and FOO).
You cannot just nilly willy assume at which domain level FOO sits. Look at your example:
> Say, we have new level FOO, and the CPUID is like this > level type eax.shifts > 0 SMT 1 > 1 CORE 5 > 2 FOO 8
FOO can be anything between CORE and PKG, so you cannot tell what it means.
Simply heuristics _cannot_ be correct by definition. So why trying to come up with them just because?
What's the problem you are trying to solve? Some real world issue or some academic though experiment which might never become a real problem?
Thanks,
tglx
| |