Re: [PATCH v4 06/34] x86/cpuid: Introduce <asm/cpuid/leaf_types.h>

[Borislav Petkov]

On Fri, Aug 15, 2025 at 09:01:59AM +0200, Ahmed S. Darwish wrote:
> +struct leaf_0x1_0 {
> +	// eax
> +	u32	stepping			:  4, // Stepping ID

All those bit names in all those leafs: are they taken from the official
documentation?

They should be.

...

> +/*
> + * Leaf 0x5
> + * MONITOR/MWAIT instructions enumeration
				 ^^^^^^^^^^^

Let's drop all those tautologies - it is absolutely clear that it is an
enumeration so no need for it. Just keep the minimum text that is needed to
describe the bit. People can always use that to find the official
documentation of they need more info.

...

> +/*
> + * Leaf 0x18
> + * Intel determenestic address translation (TLB) parameters

+ * Intel determenestic address translation (TLB) parameters
Unknown word [determenestic] in comment.
Suggestions: ['deterministic',...

spellchecker please.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette