]> git.baikalelectronics.ru Git - kernel.git/commit
powerpc/64s: Mask SRR0 before checking against the masked NIP
authorNicholas Piggin <npiggin@gmail.com>
Mon, 17 Jan 2022 13:44:03 +0000 (23:44 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 17 Jan 2022 23:25:18 +0000 (10:25 +1100)
commit3a94afe0be5a312f9c0a0537efd89b6136d8b8b9
tree9fbfca2825448b1585bafa5bdc357db78fd14b62
parent069cc9673f719d87fa4830a30f4afc54e607fb24
powerpc/64s: Mask SRR0 before checking against the masked NIP

Commit 7fb75664deb6 ("powerpc/64s: Mask NIP before checking against
SRR0") masked off the low 2 bits of the NIP value in the interrupt
stack frame in case they are non-zero and mis-compare against a SRR0
register value of a CPU which always reads back 0 from the 2 low bits
which are reserved.

This now causes the opposite problem that an implementation which does
implement those bits in SRR0 will mis-compare against the masked NIP
value in which they have been cleared. QEMU is one such implementation,
and this is allowed by the architecture.

This can be triggered by sigfuz by setting low bits of PT_NIP in the
signal context.

Fix this for now by masking the SRR0 bits as well. Cleaner is probably
to sanitise these values before putting them in registers or stack, but
this is the quick and backportable fix.

Fixes: 7fb75664deb6 ("powerpc/64s: Mask NIP before checking against SRR0")
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20220117134403.2995059-1-npiggin@gmail.com
arch/powerpc/kernel/interrupt_64.S