]> www.infradead.org Git - users/willy/xarray.git/commitdiff
x86/fpu: Rename fpu_reset_fpregs() to fpu_reset_fpstate_regs()
authorChang S. Bae <chang.seok.bae@intel.com>
Wed, 16 Apr 2025 02:17:00 +0000 (19:17 -0700)
committerIngo Molnar <mingo@kernel.org>
Wed, 16 Apr 2025 08:01:03 +0000 (10:01 +0200)
The original function name came from an overly compressed form of
'fpstate_regs' by commit:

    e61d6310a0f8 ("x86/fpu: Reset permission and fpstate on exec()")

However, the term 'fpregs' typically refers to physical FPU registers. In
contrast, this function copies the init values to fpu->fpstate->regs, not
hardware registers.

Rename the function to better reflect what it actually does.

No functional change.

Signed-off-by: Chang S. Bae <chang.seok.bae@intel.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Cc: Andy Lutomirski <luto@kernel.org>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Oleg Nesterov <oleg@redhat.com>
Link: https://lore.kernel.org/r/20250416021720.12305-11-chang.seok.bae@intel.com
arch/x86/kernel/fpu/core.c

index 3a19877a314e3e7a5ca4884310714ffacae1fe2d..8d674435f1731c5eae4b50eeeacb7d6efb2b1f7d 100644 (file)
@@ -733,7 +733,7 @@ static inline void restore_fpregs_from_init_fpstate(u64 features_mask)
 /*
  * Reset current->fpu memory state to the init values.
  */
-static void fpu_reset_fpregs(void)
+static void fpu_reset_fpstate_regs(void)
 {
        struct fpu *fpu = x86_task_fpu(current);
 
@@ -768,7 +768,7 @@ void fpu__clear_user_states(struct fpu *fpu)
 
        fpregs_lock();
        if (!cpu_feature_enabled(X86_FEATURE_FPU)) {
-               fpu_reset_fpregs();
+               fpu_reset_fpstate_regs();
                fpregs_unlock();
                return;
        }
@@ -798,7 +798,7 @@ void fpu__clear_user_states(struct fpu *fpu)
 void fpu_flush_thread(void)
 {
        fpstate_reset(x86_task_fpu(current));
-       fpu_reset_fpregs();
+       fpu_reset_fpstate_regs();
 }
 /*
  * Load FPU context before returning to userspace.