ARC: entry: fix potential EFA clobber when TIF_SYSCALL_TRACE
authorVineet Gupta <vgupta@synopsys.com>
Wed, 20 May 2020 05:28:32 +0000 (22:28 -0700)
committerVineet Gupta <vgupta@synopsys.com>
Tue, 16 Jun 2020 19:56:37 +0000 (12:56 -0700)
commit00fdec98d9881bf5173af09aebd353ab3b9ac729
tree041616ad6c5cb039d0f236afdc32f965992689cf
parentb3a9e3b9622ae10064826dccb4f7a52bd88c7407
ARC: entry: fix potential EFA clobber when TIF_SYSCALL_TRACE

Trap handler for syscall tracing reads EFA (Exception Fault Address),
in case strace wants PC of trap instruction (EFA is not part of pt_regs
as of current code).

However this EFA read is racy as it happens after dropping to pure
kernel mode (re-enabling interrupts). A taken interrupt could
context-switch, trigger a different task's trap, clobbering EFA for this
execution context.

Fix this by reading EFA early, before re-enabling interrupts. A slight
side benefit is de-duplication of FAKE_RET_FROM_EXCPN in trap handler.
The trap handler is common to both ARCompact and ARCv2 builds too.

This just came out of code rework/review and no real problem was reported
but is clearly a potential problem specially for strace.

Cc: <stable@vger.kernel.org>
Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
arch/arc/kernel/entry.S