diff options
author | Ingo Molnar <mingo@kernel.org> | 2015-04-28 11:17:55 +0200 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2015-05-19 15:48:00 +0200 |
commit | 1ac91a767f1d2ac049dc11e5b7e4342c63c21538 (patch) | |
tree | 5a0397d21e5d47972a01adfe1f35014b5670c51f /arch/x86/ia32/ia32_signal.c | |
parent | d0903193124132c6bb59a895eeb0656f86013da1 (diff) | |
download | op-kernel-dev-1ac91a767f1d2ac049dc11e5b7e4342c63c21538.zip op-kernel-dev-1ac91a767f1d2ac049dc11e5b7e4342c63c21538.tar.gz |
x86/fpu: Simplify fpstate_sanitize_xstate() calls
Remove the extra layer of __fpstate_sanitize_xstate():
if (!use_xsaveopt())
return;
__fpstate_sanitize_xstate(tsk);
and move the check for use_xsaveopt() into fpstate_sanitize_xstate().
In general we optimize for the presence of CPU features, not for
the absence of them. Furthermore there's little point in this inlining,
as the call sites are not super hot code paths.
Doing this uninlining shrinks the code a bit:
text data bss dec hex filename
14108751 2573624 1634304 18316679 1177d87 vmlinux.before
14108627 2573624 1634304 18316555 1177d0b vmlinux.after
Also remove a pointless '!fx' check from fpstate_sanitize_xstate().
Cc: Andy Lutomirski <luto@amacapital.net>
Cc: Borislav Petkov <bp@alien8.de>
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: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'arch/x86/ia32/ia32_signal.c')
0 files changed, 0 insertions, 0 deletions