]> git.baikalelectronics.ru Git - kernel.git/commit
powerpc/tm: Avoid possible userspace r1 corruption on reclaim
authorMichael Neuling <mikey@neuling.org>
Tue, 25 Sep 2018 09:36:47 +0000 (19:36 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 25 Sep 2018 12:51:32 +0000 (22:51 +1000)
commit12d20a791faa2dcde25976f106a866464a155fd0
treef0654ed863e42de110a65900fdfaf2e82c10fb3e
parent9a99de6b28acde2c0e50b9e49e7bca2a921f83d1
powerpc/tm: Avoid possible userspace r1 corruption on reclaim

Current we store the userspace r1 to PACATMSCRATCH before finally
saving it to the thread struct.

In theory an exception could be taken here (like a machine check or
SLB miss) that could write PACATMSCRATCH and hence corrupt the
userspace r1. The SLB fault currently doesn't touch PACATMSCRATCH, but
others do.

We've never actually seen this happen but it's theoretically
possible. Either way, the code is fragile as it is.

This patch saves r1 to the kernel stack (which can't fault) before we
turn MSR[RI] back on. PACATMSCRATCH is still used but only with
MSR[RI] off. We then copy r1 from the kernel stack to the thread
struct once we have MSR[RI] back on.

Suggested-by: Breno Leitao <leitao@debian.org>
Signed-off-by: Michael Neuling <mikey@neuling.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kernel/tm.S