summary refs log tree commit diff
path: root/sysdeps/unix
diff options
context:
space:
mode:
authorMatheus Castanho <msc@linux.ibm.com>2021-12-01 11:14:40 -0300
committerMatheus Castanho <msc@linux.ibm.com>2021-12-17 15:40:53 -0300
commitae91d3df24a4a1b1f264d101a71a298bff310d14 (patch)
tree9ecb3750b4552a8064c33c54269da420216aa192 /sysdeps/unix
parentc16dc431c81b8cd0605d61d24642bd3c5dcefacd (diff)
downloadglibc-ae91d3df24a4a1b1f264d101a71a298bff310d14.tar.gz
glibc-ae91d3df24a4a1b1f264d101a71a298bff310d14.tar.xz
glibc-ae91d3df24a4a1b1f264d101a71a298bff310d14.zip
powerpc64[le]: Allocate extra stack frame on syscall.S
The syscall function does not allocate the extra stack frame for scv like other
assembly syscalls using DO_CALL_SCV. So after commit d120fb9941 changed the
offset that is used to save LR, syscall ended up using an invalid offset,
causing regressions on powerpc64. So make sure the extra stack frame is
allocated in syscall.S as well to make it consistent with other uses of
DO_CALL_SCV and avoid similar issues in the future.

Tested on powerpc, powerpc64, and powerpc64le (with and without scv)

Reviewed-by: Raphael M Zinsly <rzinsly@linux.ibm.com>
Diffstat (limited to 'sysdeps/unix')
-rw-r--r--sysdeps/unix/sysv/linux/powerpc/syscall.S4
1 files changed, 4 insertions, 0 deletions
diff --git a/sysdeps/unix/sysv/linux/powerpc/syscall.S b/sysdeps/unix/sysv/linux/powerpc/syscall.S
index a29652feaf..a5497c8370 100644
--- a/sysdeps/unix/sysv/linux/powerpc/syscall.S
+++ b/sysdeps/unix/sysv/linux/powerpc/syscall.S
@@ -27,7 +27,11 @@ ENTRY (syscall)
 	mr   r8,r9
 #if defined(USE_PPC_SCV) && !IS_IN(rtld) && (defined(__PPC64__) || defined(__powerpc64__))
 	CHECK_SCV_SUPPORT r9 0f
+	stdu r1,-SCV_FRAME_SIZE(r1)
+	cfi_adjust_cfa_offset(SCV_FRAME_SIZE)
 	DO_CALL_SCV
+	addi r1,r1,SCV_FRAME_SIZE
+	cfi_adjust_cfa_offset(-SCV_FRAME_SIZE)
 	RET_SCV
 	b 1f
 #endif