diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2021-06-23 14:02:11 +0200 |
---|---|---|
committer | Borislav Petkov <bp@suse.de> | 2021-06-23 19:15:16 +0200 |
commit | 371071131cd1032c1e9172c51234a2a324841cab (patch) | |
tree | 0ca7ca1065a411917f11a58ad791d6c94c2e635b /include/linux/pkeys.h | |
parent | fa8c84b77a54bf3cf351c8b4b26a5aca27a14013 (diff) | |
download | lwn-371071131cd1032c1e9172c51234a2a324841cab.tar.gz lwn-371071131cd1032c1e9172c51234a2a324841cab.zip |
x86/fpu: Use pkru_write_default() in copy_init_fpstate_to_fpregs()
There is no point in using copy_init_pkru_to_fpregs() which in turn calls
write_pkru(). write_pkru() tries to fiddle with the task's xstate buffer
for nothing because the XRSTOR[S](init_fpstate) just cleared the xfeature
flag in the xstate header which makes get_xsave_addr() fail.
It's a useless exercise anyway because the reinitialization activates the
FPU so before the task's xstate buffer can be used again a XRSTOR[S] must
happen which in turn dumps the PKRU value.
Get rid of the now unused copy_init_pkru_to_fpregs().
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lkml.kernel.org/r/20210623121455.732508792@linutronix.de
Diffstat (limited to 'include/linux/pkeys.h')
-rw-r--r-- | include/linux/pkeys.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/include/linux/pkeys.h b/include/linux/pkeys.h index 2955ba976048..6beb26b7151d 100644 --- a/include/linux/pkeys.h +++ b/include/linux/pkeys.h @@ -44,10 +44,6 @@ static inline bool arch_pkeys_enabled(void) return false; } -static inline void copy_init_pkru_to_fpregs(void) -{ -} - #endif /* ! CONFIG_ARCH_HAS_PKEYS */ #endif /* _LINUX_PKEYS_H */ |