diff options
| author |   Jan Kiszka <jan.kiszka@web.de> | 2007-05-02 19:27:21 +0200 |
|---|---|---|
| committer |   Andi Kleen <andi@basil.nowhere.org> | 2007-05-02 19:27:21 +0200 |
| commit | c41bf8fa5e777b6a8a19cf2484937a7167eac77f (patch) | |
| tree | 982494445301d399078fe6abdedc724843561f5d /include/asm-i386 | |
| parent | [PATCH] i386: white space fixes in i387.h (diff) | |
| download | linux-dev-c41bf8fa5e777b6a8a19cf2484937a7167eac77f.tar.xz linux-dev-c41bf8fa5e777b6a8a19cf2484937a7167eac77f.zip | |
[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>
Diffstat (limited to 'include/asm-i386')
| -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 49dc8e141310..cdd1e248e3b4 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 ) \ |