[PATCH] POWERPC: Fix subtle FP state corruption bug in signal return on SMP
authorPaul Mackerras <paulus@samba.org>
Tue, 26 Jun 2007 10:10:12 +0000 (20:10 +1000)
committerWilly Tarreau <w@1wt.eu>
Wed, 15 Aug 2007 08:02:32 +0000 (10:02 +0200)
commit0c53130a7d5da7597548f885e7b3238af18945d3
tree7a9ab3b0aa9f413b1f8e43db1d2e1c44f6012ce3
parent78977b665b8f03c56bbb0f7549200e058b7e067f
[PATCH] POWERPC: Fix subtle FP state corruption bug in signal return on SMP

This fixes a bug which can cause corruption of the floating-point state
on return from a signal handler.  If we have a signal handler that has
used the floating-point registers, and it happens to context-switch to
another task while copying the interrupted floating-point state from the
user stack into the thread struct (e.g. because of a page fault, or
because it gets preempted), the context switch code will think that the
FP registers contain valid FP state that needs to be copied into the
thread_struct, and will thus overwrite the values that the signal return
code has put into the thread_struct.

This can occur because we clear the MSR bits that indicate the presence
of valid FP state after copying the state into the thread_struct.  To fix
this we just move the clearing of the MSR bits to before the copy.  A
similar potential problem also occurs with the Altivec state, and this
fixes that in the same way.

Signed-off-by: Paul Mackerras <paulus@samba.org>
Signed-off-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
arch/powerpc/kernel/signal_64.c