]> git.baikalelectronics.ru Git - kernel.git/commit
parisc: make interrupt and interruption stack allocation reentrant
authorJohn David Anglin <dave.anglin@bell.net>
Mon, 20 May 2013 16:42:53 +0000 (16:42 +0000)
committerHelge Deller <deller@gmx.de>
Fri, 24 May 2013 20:35:36 +0000 (22:35 +0200)
commit979cbd68c9ef94b78c00c05563eae348f9390d4e
tree28dfb7722a7c974f9872a566905b441cf7f18190
parent7ab3b01b218e07ab223f8725a5d41dcbf2bfd913
parisc: make interrupt and interruption stack allocation reentrant

The get_stack_use_cr30 and get_stack_use_r30 macros allocate a stack
frame for external interrupts and interruptions requiring a stack frame.
They are currently not reentrant in that they save register context
before the stack is set or adjusted.

I have observed a number of system crashes where there was clear
evidence of stack corruption during interrupt processing, and as a
result register corruption. Some interruptions can still occur during
interruption processing, however external interrupts are disabled and
data TLB misses don't occur for absolute accesses. So, it's not entirely
clear what triggers this issue. Also, if an interruption occurs when
Q=0, it is generally not possible to recover as the shadowed registers
are not copied.

The attached patch reworks the get_stack_use_cr30 and get_stack_use_r30
macros to allocate stack before doing register saves. The new code is a
couple of instructions shorter than the old implementation. Thus, it's
an improvement even if it doesn't fully resolve the stack corruption
issue. Based on limited testing, it improves SMP system stability.

Signed-off-by: John David Anglin <dave.anglin@bell.net>
Signed-off-by: Helge Deller <deller@gmx.de>
arch/parisc/include/asm/assembly.h
arch/parisc/kernel/entry.S