Re: [PATCH v3 3/5] x86/fpu: Clean up the fpu__clear() variants

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6/1/21 11:14 AM, Andy Lutomirski wrote:
>> Nit: Could we move the detailed comments about TIF_NEED_FPU_LOAD
>> right next to the fpu__initialize() call?  It would make it
>> painfully obvious which call is responsible.  The naming isn't
>> super helpful here.
>> 
> Hmm. I was actually thinking of open-coding it all.
> fpu__initialize() and fpu__drop() have very few callers, and I’m not
> even convinced that the other callers are calling them for a valid
> reason.

Fine by me.  We all know what the TIF flag does, but its connection to
this code is totally opaque.  It's a place where removing the
abstraction might actually make sense.



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux