From c3f90e26a5160c3fe287c4283e7a2bc1ff54b1a9 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Fri, 6 Jun 2003 17:34:54 +0000 Subject: Update. * sysdeps/unix/sysv/linux/sleep.c (__sleep): Use nanosleep syscall directly to avoid cancellation in the regular nanosleep implementation. --- ChangeLog | 4 ++++ linuxthreads/ChangeLog | 6 ++++++ linuxthreads/sysdeps/unix/sysv/linux/arm/sysdep-cancel.h | 14 ++++++++------ sysdeps/unix/sysv/linux/sleep.c | 16 ++++++++++------ 4 files changed, 28 insertions(+), 12 deletions(-) diff --git a/ChangeLog b/ChangeLog index 342a2a15ee..2295357527 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ 2003-06-06 Ulrich Drepper + * sysdeps/unix/sysv/linux/sleep.c (__sleep): Use nanosleep syscall + directly to avoid cancellation in the regular nanosleep + implementation. + * sysdeps/unix/sysv/linux/i386/sysdep.h (ASMFMT_2): Only allow %edx for first parameter. This means no pushl and therefore the unwind info isn't screwed up. diff --git a/linuxthreads/ChangeLog b/linuxthreads/ChangeLog index 9dee60ff23..a7d422e837 100644 --- a/linuxthreads/ChangeLog +++ b/linuxthreads/ChangeLog @@ -1,3 +1,9 @@ +2003-06-06 Ulrich Drepper + + * sysdeps/unix/sysv/linux/arm/sysdep-cancel.h: New sequences for + 5+ arg syscalls only needed for PIC. + Patch by Ralph Siemsen . + 2003-06-05 Richard Henderson * sysdeps/unix/sysv/linux/alpha/sysdep-cancel.h (PSEUDO): Use diff --git a/linuxthreads/sysdeps/unix/sysv/linux/arm/sysdep-cancel.h b/linuxthreads/sysdeps/unix/sysv/linux/arm/sysdep-cancel.h index d2043aad91..92d8460e26 100644 --- a/linuxthreads/sysdeps/unix/sysv/linux/arm/sysdep-cancel.h +++ b/linuxthreads/sysdeps/unix/sysv/linux/arm/sysdep-cancel.h @@ -26,12 +26,14 @@ /* We push lr onto the stack, so we have to use ldmib instead of ldmia to find the saved arguments. */ -#undef DOARGS_5 -#undef DOARGS_6 -#undef DOARGS_7 -#define DOARGS_5 str r4, [sp, $-4]!; ldr r4, [sp, $8]; -#define DOARGS_6 mov ip, sp; stmfd sp!, {r4, r5}; ldmib ip, {r4, r5}; -#define DOARGS_7 mov ip, sp; stmfd sp!, {r4, r5, r6}; ldmib ip, {r4, r5, r6}; +# ifdef PIC +# undef DOARGS_5 +# undef DOARGS_6 +# undef DOARGS_7 +# define DOARGS_5 str r4, [sp, $-4]!; ldr r4, [sp, $8]; +# define DOARGS_6 mov ip, sp; stmfd sp!, {r4, r5}; ldmib ip, {r4, r5}; +# define DOARGS_7 mov ip, sp; stmfd sp!, {r4, r5, r6}; ldmib ip, {r4, r5, r6}; +# endif # undef PSEUDO_RET # define PSEUDO_RET \ diff --git a/sysdeps/unix/sysv/linux/sleep.c b/sysdeps/unix/sysv/linux/sleep.c index 0cf6be285d..38c197358f 100644 --- a/sysdeps/unix/sysv/linux/sleep.c +++ b/sysdeps/unix/sysv/linux/sleep.c @@ -1,5 +1,5 @@ /* Implementation of the POSIX sleep function using nanosleep. - Copyright (C) 1996, 1997, 1998, 1999 Free Software Foundation, Inc. + Copyright (C) 1996, 1997, 1998, 1999, 2003 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper , 1996. @@ -24,8 +24,12 @@ #include /* We are going to use the `nanosleep' syscall of the kernel. But the - kernel does not implement the sstupid SysV SIGCHLD vs. SIG_IGN - behaviour for this syscall. Therefore we have to emulate it here. */ + kernel does not implement the stupid SysV SIGCHLD vs. SIG_IGN + behaviour for this syscall. Therefore we have to emulate it here. + + Also note that we use the syscall directly instead of using the + __nanosleep function. The reason is that nanosleep() is a cancellation + point while sleep() isn't. */ unsigned int __sleep (unsigned int seconds) { @@ -67,7 +71,7 @@ __sleep (unsigned int seconds) if (oact.sa_handler == SIG_IGN) { /* We should leave SIGCHLD blocked. */ - result = __nanosleep (&ts, &ts); + result = INLINE_SYSCALL (nanosleep, 2, &ts, &ts); saved_errno = errno; /* Restore the original signal mask. */ @@ -78,11 +82,11 @@ __sleep (unsigned int seconds) { /* We should unblock SIGCHLD. Restore the original signal mask. */ (void) __sigprocmask (SIG_SETMASK, &oset, (sigset_t *) NULL); - result = __nanosleep (&ts, &ts); + result = INLINE_SYSCALL (nanosleep, 2, &ts, &ts); } } else - result = __nanosleep (&ts, &ts); + result = INLINE_SYSCALL (nanosleep, 2, &ts, &ts); if (result != 0) /* Round remaining time. */ -- cgit v1.2.3