diff options
author | Suresh Siddha <suresh.b.siddha@intel.com> | 2010-02-22 17:51:34 -0500 |
---|---|---|
committer | H. Peter Anvin <hpa@zytor.com> | 2010-02-23 16:45:27 -0500 |
commit | 6dbbe14f21368a45aedba7eab0221857b8ad8d16 (patch) | |
tree | c6226a4e9e04a7e2bd8aba94c04f543a9dc6cee4 | |
parent | ff7fbc72e0c3ef7e94a27a3a918fd09ec9a30204 (diff) |
x86, ptrace: Remove set_stopped_child_used_math() in [x]fpregs_set
init_fpu() already ensures that the used_math() is set for the stopped child.
Remove the redundant set_stopped_child_used_math() in [x]fpregs_set()
Reported-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>
LKML-Reference: <20100222225240.642169080@sbs-t61.sc.intel.com>
Acked-by: Rolan McGrath <roland@redhat.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
-rw-r--r-- | arch/x86/kernel/i387.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/arch/x86/kernel/i387.c b/arch/x86/kernel/i387.c index 81e23bf12c12..c01a2b846d47 100644 --- a/arch/x86/kernel/i387.c +++ b/arch/x86/kernel/i387.c | |||
@@ -209,8 +209,6 @@ int xfpregs_set(struct task_struct *target, const struct user_regset *regset, | |||
209 | if (ret) | 209 | if (ret) |
210 | return ret; | 210 | return ret; |
211 | 211 | ||
212 | set_stopped_child_used_math(target); | ||
213 | |||
214 | ret = user_regset_copyin(&pos, &count, &kbuf, &ubuf, | 212 | ret = user_regset_copyin(&pos, &count, &kbuf, &ubuf, |
215 | &target->thread.xstate->fxsave, 0, -1); | 213 | &target->thread.xstate->fxsave, 0, -1); |
216 | 214 | ||
@@ -471,8 +469,6 @@ int fpregs_set(struct task_struct *target, const struct user_regset *regset, | |||
471 | if (ret) | 469 | if (ret) |
472 | return ret; | 470 | return ret; |
473 | 471 | ||
474 | set_stopped_child_used_math(target); | ||
475 | |||
476 | if (!HAVE_HWFP) | 472 | if (!HAVE_HWFP) |
477 | return fpregs_soft_set(target, regset, pos, count, kbuf, ubuf); | 473 | return fpregs_soft_set(target, regset, pos, count, kbuf, ubuf); |
478 | 474 | ||