s390: fix fpu restore in entry.S
authorSven Schnelle <svens@linux.ibm.com>
Fri, 20 Nov 2020 13:17:52 +0000 (14:17 +0100)
committerHeiko Carstens <hca@linux.ibm.com>
Mon, 23 Nov 2020 10:52:13 +0000 (11:52 +0100)
commit1179f170b6f0af7bb0b3b7628136eaac450ddf31
tree19e8acb64e0968b41de4899cc1315c41b002839e
parent78d732e1f326f74f240d416af9484928303d9951
s390: fix fpu restore in entry.S

We need to disable interrupts in load_fpu_regs(). Otherwise an
interrupt might come in after the registers are loaded, but before
CIF_FPU is cleared in load_fpu_regs(). When the interrupt returns,
CIF_FPU will be cleared and the registers will never be restored.

The entry.S code usually saves the interrupt state in __SF_EMPTY on the
stack when disabling/restoring interrupts. sie64a however saves the pointer
to the sie control block in __SF_SIE_CONTROL, which references the same
location.  This is non-obvious to the reader. To avoid thrashing the sie
control block pointer in load_fpu_regs(), move the __SIE_* offsets eight
bytes after __SF_EMPTY on the stack.

Cc: <stable@vger.kernel.org> # 5.8
Fixes: 0b0ed657fe00 ("s390: remove critical section cleanup from entry.S")
Reported-by: Pierre Morel <pmorel@linux.ibm.com>
Signed-off-by: Sven Schnelle <svens@linux.ibm.com>
Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
Reviewed-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
arch/s390/kernel/asm-offsets.c
arch/s390/kernel/entry.S