davidlt/kernel-ark
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

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

Browse Source 
On exec(), extended register states saved in the buffer is cleared. With
dynamic features, each task carries variables besides the register states.
The struct fpu has permission information and struct fpstate contains
buffer size and feature masks. They are all dynamically updated with
dynamic features.

Reset the current task's entire FPU data before an exec() so that the new
task starts with default permission and fpstate.

Rename the register state reset function because the old naming confuses as
it does not reset struct fpstate.

Signed-off-by: Chang S. Bae <chang.seok.bae@intel.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lkml.kernel.org/r/20211021225527.10184-12-chang.seok.bae@intel.com
This commit is contained in:
Chang S. Bae 2021-10-21 15:55:15 -07:00 committed by Borislav Petkov
parent 9e798e9aa1
commit e61d6310a0
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
arch/x86/kernel/fpu/core.c
View File

@ -544,7 +544,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_fpstate(void)
static void fpu_reset_fpregs(void)
{
struct fpu *fpu = &current->thread.fpu;
@ -579,7 +579,7 @@ void fpu__clear_user_states(struct fpu *fpu)
fpregs_lock();
if (!cpu_feature_enabled(X86_FEATURE_FPU)) {
fpu_reset_fpstate();
fpu_reset_fpregs();
fpregs_unlock();
return;
}
@ -609,7 +609,8 @@ void fpu__clear_user_states(struct fpu *fpu)
void fpu_flush_thread(void)
{
fpu_reset_fpstate();
fpstate_reset(&current->thread.fpu);
fpu_reset_fpregs();
}
/*
* Load FPU context before returning to userspace.