Commit 0afc4a94 authored by Ingo Molnar's avatar Ingo Molnar
Browse files

x86/fpu: Remove fpu_xsave() 



It's a pointless wrapper now - use xsave_state().

Reviewed-by: default avatarBorislav Petkov <bp@alien8.de>
Cc: Andy Lutomirski <luto@amacapital.net>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent 3e261c14

arch/x86/include/asm/fpu-internal.h

+1 −1
Original line number Diff line number Diff line
@@ -265,7 +265,7 @@ static inline void fpu_fxsave(struct fpu *fpu) 
static inline int fpu_save_init(struct fpu *fpu)

{

	if (use_xsave()) {

		fpu_xsave(fpu);

		xsave_state(&fpu->state->xsave);



		/*

		 * xsave header may indicate the init state of the FP.

arch/x86/include/asm/xsave.h

+0 −8
Original line number Diff line number Diff line
@@ -186,14 +186,6 @@ static inline int xrstor_state(struct xsave_struct *fx, u64 mask) 
	return err;

}



/*

 * Save xstate context for old process during context switch.

 */

static inline void fpu_xsave(struct fpu *fpu)

{

	xsave_state(&fpu->state->xsave);

}



/*

 * Restore xstate context for new process during context switch.

 */

arch/x86/mm/mpx.c

+1 −1
Original line number Diff line number Diff line
@@ -389,7 +389,7 @@ int mpx_enable_management(struct task_struct *tsk) 
	 * directory into XSAVE/XRSTOR Save Area and enable MPX through

	 * XRSTOR instruction.

	 *

	 * fpu_xsave() is expected to be very expensive. Storing the bounds

	 * xsave_state() is expected to be very expensive. Storing the bounds

	 * directory here means that we do not have to do xsave in the unmap

	 * path; we can just use mm->bd_addr instead.

	 */

CRA Git | Maintained and supported by SUSTech CRA and CCSE