diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-02-13 16:47:25 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-02-13 16:47:25 -0500 |
| commit | be98c2cdb15ba26148cd2bd58a857d4f7759ed38 (patch) | |
| tree | 5eb1bdb4e443680c12901add5dc23a9cd3468872 | |
| parent | 3ec1e88b33a3bdd852ce8e014052acec7a9da8b5 (diff) | |
i387: math_state_restore() isn't called from asm
It was marked asmlinkage for some really old and stale legacy reasons.
Fix that and the equally stale comment.
Noticed when debugging the irq_fpu_usable() bugs.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | arch/x86/include/asm/i387.h | 2 | ||||
| -rw-r--r-- | arch/x86/kernel/traps.c | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/arch/x86/include/asm/i387.h b/arch/x86/include/asm/i387.h index 6919e936345b..a5c7ae504176 100644 --- a/arch/x86/include/asm/i387.h +++ b/arch/x86/include/asm/i387.h | |||
| @@ -29,7 +29,7 @@ extern unsigned int sig_xstate_size; | |||
| 29 | extern void fpu_init(void); | 29 | extern void fpu_init(void); |
| 30 | extern void mxcsr_feature_mask_init(void); | 30 | extern void mxcsr_feature_mask_init(void); |
| 31 | extern int init_fpu(struct task_struct *child); | 31 | extern int init_fpu(struct task_struct *child); |
| 32 | extern asmlinkage void math_state_restore(void); | 32 | extern void math_state_restore(void); |
| 33 | extern void __math_state_restore(void); | 33 | extern void __math_state_restore(void); |
| 34 | extern int dump_fpu(struct pt_regs *, struct user_i387_struct *); | 34 | extern int dump_fpu(struct pt_regs *, struct user_i387_struct *); |
| 35 | 35 | ||
diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c index 482ec3af2067..982433b5da30 100644 --- a/arch/x86/kernel/traps.c +++ b/arch/x86/kernel/traps.c | |||
| @@ -599,10 +599,10 @@ void __math_state_restore(void) | |||
| 599 | * Careful.. There are problems with IBM-designed IRQ13 behaviour. | 599 | * Careful.. There are problems with IBM-designed IRQ13 behaviour. |
| 600 | * Don't touch unless you *really* know how it works. | 600 | * Don't touch unless you *really* know how it works. |
| 601 | * | 601 | * |
| 602 | * Must be called with kernel preemption disabled (in this case, | 602 | * Must be called with kernel preemption disabled (eg with local |
| 603 | * local interrupts are disabled at the call-site in entry.S). | 603 | * local interrupts as in the case of do_device_not_available). |
| 604 | */ | 604 | */ |
| 605 | asmlinkage void math_state_restore(void) | 605 | void math_state_restore(void) |
| 606 | { | 606 | { |
| 607 | struct thread_info *thread = current_thread_info(); | 607 | struct thread_info *thread = current_thread_info(); |
| 608 | struct task_struct *tsk = thread->task; | 608 | struct task_struct *tsk = thread->task; |
