arm64: Abstract syscallno manipulation
authorDave Martin <Dave.Martin@arm.com>
Tue, 1 Aug 2017 14:35:54 +0000 (15:35 +0100)
committerCatalin Marinas <catalin.marinas@arm.com>
Mon, 7 Aug 2017 08:58:33 +0000 (09:58 +0100)
commit17c28958600928109049a3bcc814b0d5bfb1ff3a
tree901508d772cc2fe2b33013b01b59ca990bd8f6c5
parent35d0e6fb4d219d64ab3b7cffef7a11a0662140f5
arm64: Abstract syscallno manipulation

The -1 "no syscall" value is written in various ways, shared with
the user ABI in some places, and generally obscure.

This patch attempts to make things a little more consistent and
readable by replacing all these uses with a single #define.  A
couple of symbolic helpers are provided to clarify the intent
further.

Because the in-syscall check in do_signal() is changed from >= 0 to
!= NO_SYSCALL by this patch, different behaviour may be observable
if syscallno is set to values less than -1 by a tracer.  However,
this is not different from the behaviour that is already observable
if a tracer sets syscallno to a value >= __NR_(compat_)syscalls.

It appears that this can cause spurious syscall restarting, but
that is not a new behaviour either, and does not appear harmful.

Signed-off-by: Dave Martin <Dave.Martin@arm.com>
Acked-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
arch/arm64/include/asm/processor.h
arch/arm64/include/asm/ptrace.h
arch/arm64/kernel/entry.S
arch/arm64/kernel/ptrace.c
arch/arm64/kernel/signal.c
arch/arm64/kernel/signal32.c