Re: [PATCH 09/10] x86/fpu: Remove unnecessary CPUID level check

From: Dave Hansen
Date: Mon Dec 16 2024 - 16:23:39 EST


On 12/16/24 11:05, Chang S. Bae wrote:
...> Then, the sanity check introduced in this code path appears redundant:
>
> xsave_init()
> -> xstate_enable_boot_cpu()
>     -> setup_xstate_init()
>         -> setup_xstate_features()
>
> Reviewed-by: Chang S. Bae <chang.seok.bae@xxxxxxxxx>

Yeah, good point. Thanks for the review!

BTW, I do still want to make this into a _generic_ warning so that all
cpuid*() users get warned if they go reading leafs that are too high.
But I'll do that in another series.