diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2015-02-06 15:01:58 -0500 |
|---|---|---|
| committer | Borislav Petkov <bp@suse.de> | 2015-02-19 11:12:40 +0100 |
| commit | a9241ea5fd709fc935dade130f4e3b2612bbe9e3 (patch) | |
| tree | 73b7221bea33cb1caed9c79fb30018aec5274ed4 /arch/x86/kernel/i387.c | |
| parent | bfa76d49576599a4b9f9b7a71f23d73d6dcff735 (diff) | |
| download | op-kernel-dev-a9241ea5fd709fc935dade130f4e3b2612bbe9e3.zip op-kernel-dev-a9241ea5fd709fc935dade130f4e3b2612bbe9e3.tar.gz | |
x86/fpu: Don't reset thread.fpu_counter
The "else" branch clears ->fpu_counter as a remnant of the lazy FPU
usage counting:
e07e23e1fd30 ("[PATCH] non lazy "sleazy" fpu implementation")
However, switch_fpu_prepare() does this now so that else branch is
superfluous.
If we do use_eager_fpu(), then this has no effect. Otherwise, if we
actually wanted to prevent fpu preload after the context switch we would
need to reset it unconditionally, even if __thread_has_fpu().
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Rik van Riel <riel@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Link: http://lkml.kernel.org/r/1423252925-14451-2-git-send-email-riel@redhat.com
Signed-off-by: Borislav Petkov <bp@suse.de>
Diffstat (limited to 'arch/x86/kernel/i387.c')
| -rw-r--r-- | arch/x86/kernel/i387.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/x86/kernel/i387.c b/arch/x86/kernel/i387.c index a9a4229..4d0db9e 100644 --- a/arch/x86/kernel/i387.c +++ b/arch/x86/kernel/i387.c @@ -108,8 +108,7 @@ void unlazy_fpu(struct task_struct *tsk) if (__thread_has_fpu(tsk)) { __save_init_fpu(tsk); __thread_fpu_end(tsk); - } else - tsk->thread.fpu_counter = 0; + } preempt_enable(); } EXPORT_SYMBOL(unlazy_fpu); |
