summaryrefslogtreecommitdiffstats
path: root/arch/sparc/include/uapi/asm/unistd.h
diff options
context:
space:
mode:
authorHelge Deller <deller@gmx.de>2015-12-21 10:03:30 +0100
committerHelge Deller <deller@gmx.de>2015-12-21 10:16:18 +0100
commit71a71fb5374a23be36a91981b5614590b9e722c3 (patch)
tree7caabcb26d339b8b7ecde1de88f133743dae74b9 /arch/sparc/include/uapi/asm/unistd.h
parent4ef7675344d687a0ef5b0d7c0cee12da005870c0 (diff)
downloadop-kernel-dev-71a71fb5374a23be36a91981b5614590b9e722c3.zip
op-kernel-dev-71a71fb5374a23be36a91981b5614590b9e722c3.tar.gz
parisc: Fix syscall restarts
On parisc syscalls which are interrupted by signals sometimes failed to restart and instead returned -ENOSYS which in the worst case lead to userspace crashes. A similiar problem existed on MIPS and was fixed by commit e967ef02 ("MIPS: Fix restart of indirect syscalls"). On parisc the current syscall restart code assumes that all syscall callers load the syscall number in the delay slot of the ble instruction. That's how it is e.g. done in the unistd.h header file: ble 0x100(%sr2, %r0) ldi #syscall_nr, %r20 Because of that assumption the current code never restored %r20 before returning to userspace. This assumption is at least not true for code which uses the glibc syscall() function, which instead uses this syntax: ble 0x100(%sr2, %r0) copy regX, %r20 where regX depend on how the compiler optimizes the code and register usage. This patch fixes this problem by adding code to analyze how the syscall number is loaded in the delay branch and - if needed - copy the syscall number to regX prior returning to userspace for the syscall restart. Signed-off-by: Helge Deller <deller@gmx.de> Cc: stable@vger.kernel.org Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Diffstat (limited to 'arch/sparc/include/uapi/asm/unistd.h')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud