diff options
| author | Jan Kiszka <jan.kiszka@web.de> | 2007-05-02 17:27:21 (GMT) |
|---|---|---|
| committer | Andi Kleen <andi@basil.nowhere.org> | 2007-05-02 17:27:21 (GMT) |
| commit | c41bf8fa5e777b6a8a19cf2484937a7167eac77f (patch) | |
| tree | 982494445301d399078fe6abdedc724843561f5d | |
| parent | 02b64dab5675bc08048c7f70cbb0d8a417d20dbe (diff) | |
| download | linux-c41bf8fa5e777b6a8a19cf2484937a7167eac77f.tar.xz | |
[PATCH] i386: avoid redundant preempt_disable in __unlazy_fpu
There are two callers of __unlazy_fpu, unlazy_fpu and __switch_to, and
none of them appear to require additional preempt_disable/enable here.
Let's open-code save_init_fpu in __unlazy_fpu to save a few ops.
Signed-off-by: Jan Kiszka <jan.kiszka@web.de>
Signed-off-by: Andi Kleen <ak@suse.de>
| -rw-r--r-- | include/asm-i386/i387.h | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/include/asm-i386/i387.h b/include/asm-i386/i387.h index 49dc8e1..cdd1e24 100644 --- a/include/asm-i386/i387.h +++ b/include/asm-i386/i387.h @@ -74,11 +74,12 @@ static inline void __save_init_fpu( struct task_struct *tsk ) task_thread_info(tsk)->status &= ~TS_USEDFPU; } -#define __unlazy_fpu( tsk ) do { \ - if (task_thread_info(tsk)->status & TS_USEDFPU) \ - save_init_fpu( tsk ); \ - else \ - tsk->fpu_counter = 0; \ +#define __unlazy_fpu( tsk ) do { \ + if (task_thread_info(tsk)->status & TS_USEDFPU) { \ + __save_init_fpu(tsk); \ + stts(); \ + } else \ + tsk->fpu_counter = 0; \ } while (0) #define __clear_fpu( tsk ) \ |