Re: [PATCH] x86/fpu: Fix irq_fpu_usable() to return false during CPU onlining

[Thomas Gleixner <tglx@linutronix.de>]

On Tue, May 20 2025 at 11:33, Ingo Molnar wrote:
> * Eric Biggers <ebiggers@kernel.org> wrote:
>
>> Or we could use DEFINE_PER_CPU() = true in patch 1, then revert that 
>> in patch 2 and replace it with the line in fpu__init_cpu().  But 
>> again I think the split would be more likely to create problems than 
>> solve them.
>
> Well, my request would be for the first patch to simply mimic current 
> (and buggy) behavior as much as reasonably possible (obviously the 
> effects of BSS zeroing shouldn't be mimiced 100%) - and the second 
> patch to fix the initialization-ordering bug.

So the first patch is then incomprehensible buggy and needs a trivial
one-liner to fix up, right?

TBH, that's just bonkers. Eric's patch is trivial enough as is and easy
to review. Artifical patch splitting with buggy intermediate state makes
only sense, when the overall changes are massive and hard to
review. That's absolutely not the case here.

Thanks,

        tglx