Skip to content

Commit

Permalink
Use __libc_ptf_call in _longjmp_unwind
Browse files Browse the repository at this point in the history
Use __libc_ptf_call in _longjmp_unwind to avoid duplicated logic.  On
x86-64, I got

jmp-unwind.os:
_longjmp_unwind:
	movl	__libc_pthread_functions_init(%rip), %eax
	testl	%eax, %eax
	jne	.L4
	rep ret
.L4:
	movq	304+__libc_pthread_functions(%rip), %rax
	movq	%rsp, %rsi
	ror $2*8+1, %rax
	xor %fs:48, %rax
	jmp	*%rax

jmp-unwind.o:

_longjmp_unwind:
	movl	$__pthread_cleanup_upto, %eax
	testq	%rax, %rax
	je	.L1
	movq	%rsp, %rsi
	jmp	__pthread_cleanup_upto
.L1:
	rep ret

	* sysdeps/nptl/jmp-unwind.c: Include <libc-lock.h> instead of
	<nptl/pthreadP.h>.
	(_longjmp_unwind): Use __libc_ptf_call.
	* sysdeps/unix/sysv/linux/s390/jmp-unwind.c: Likewise.
  • Loading branch information
H.J. Lu committed Sep 18, 2015
1 parent 3ee5786 commit 94afa0a
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 17 deletions.
7 changes: 7 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,10 @@
2015-09-18 H.J. Lu <hongjiu.lu@intel.com>

* sysdeps/nptl/jmp-unwind.c: Include <libc-lock.h> instead of
<nptl/pthreadP.h>.
(_longjmp_unwind): Use __libc_ptf_call.
* sysdeps/unix/sysv/linux/s390/jmp-unwind.c: Likewise.

2015-09-18 David Lamparter <equinox-gentoo@diac24.net>

* sysdeps/arm/setjmp.S: Change PIC to SHARED.
Expand Down
12 changes: 3 additions & 9 deletions sysdeps/nptl/jmp-unwind.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

#include <setjmp.h>
#include <stddef.h>
#include <nptl/pthreadP.h>
#include <libc-lock.h>

extern void __pthread_cleanup_upto (__jmp_buf env, char *targetframe);
#pragma weak __pthread_cleanup_upto
Expand All @@ -27,12 +27,6 @@ extern void __pthread_cleanup_upto (__jmp_buf env, char *targetframe);
void
_longjmp_unwind (jmp_buf env, int val)
{
#ifdef SHARED
if (__libc_pthread_functions_init)
PTHFCT_CALL (ptr___pthread_cleanup_upto, (env->__jmpbuf,
CURRENT_STACK_FRAME));
#else
if (__pthread_cleanup_upto != NULL)
__pthread_cleanup_upto (env->__jmpbuf, CURRENT_STACK_FRAME);
#endif
__libc_ptf_call (__pthread_cleanup_upto, (env->__jmpbuf,
CURRENT_STACK_FRAME), 0);
}
10 changes: 2 additions & 8 deletions sysdeps/unix/sysv/linux/s390/jmp-unwind.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

#include <setjmp.h>
#include <stddef.h>
#include <nptl/pthreadP.h>
#include <libc-lock.h>

extern void __pthread_cleanup_upto (__jmp_buf env, char *targetframe);
#pragma weak __pthread_cleanup_upto
Expand All @@ -29,11 +29,5 @@ _longjmp_unwind (jmp_buf env, int val)
{
char local_var;

#ifdef SHARED
if (__libc_pthread_functions_init)
PTHFCT_CALL (ptr___pthread_cleanup_upto, (env->__jmpbuf, &local_var));
#else
if (__pthread_cleanup_upto != NULL)
__pthread_cleanup_upto (env->__jmpbuf, &local_var);
#endif
__libc_ptf_call (__pthread_cleanup_upto, (env->__jmpbuf, &local_var), 0);
}

0 comments on commit 94afa0a

Please sign in to comment.