diff options
| author | Ingo Molnar | 2015-04-03 12:28:15 +0200 |
|---|---|---|
| committer | Ingo Molnar | 2015-05-19 15:47:11 +0200 |
| commit | 8694c3e793810de3f7ff612d0fc23b70dc5a7e4d (patch) | |
| tree | f01bdbcf89771d3ac398166d746ce9ed88c95885 | |
| parent | x86/fpu: Rename init_fpu() to fpu__unlazy_stopped() and add debugging check (diff) | |
| download | kernel-qcow2-linux-8694c3e793810de3f7ff612d0fc23b70dc5a7e4d.tar.gz kernel-qcow2-linux-8694c3e793810de3f7ff612d0fc23b70dc5a7e4d.tar.xz kernel-qcow2-linux-8694c3e793810de3f7ff612d0fc23b70dc5a7e4d.zip | |
x86/fpu: Optimize fpu__unlazy_stopped()
This function is only called for stopped child tasks, so the
fpu__save() branch will never get called - remove it.
Reviewed-by: Borislav 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: Ingo Molnar <mingo@kernel.org>
| -rw-r--r-- | arch/x86/kernel/i387.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/x86/kernel/i387.c b/arch/x86/kernel/i387.c index 909d10d2fb70..76006a701dbb 100644 --- a/arch/x86/kernel/i387.c +++ b/arch/x86/kernel/i387.c @@ -291,8 +291,6 @@ static int fpu__unlazy_stopped(struct task_struct *child) return -EINVAL; if (tsk_used_math(child)) { - if (cpu_has_fpu && child == current) - fpu__save(child); task_disable_lazy_fpu_restore(child); return 0; } |