diff options
author | Ulrich Drepper <drepper@redhat.com> | 2003-06-29 18:01:53 +0000 |
---|---|---|
committer | Ulrich Drepper <drepper@redhat.com> | 2003-06-29 18:01:53 +0000 |
commit | 5245d8a1fe962ee93d81e305e8fa249ee8a6b354 (patch) | |
tree | 37be6c168b736799fa945a64173fd03b939412f6 | |
parent | 5a81730f4150c39a0c0034eaf55a45462dfc78c2 (diff) | |
download | glibc-5245d8a1fe962ee93d81e305e8fa249ee8a6b354.zip glibc-5245d8a1fe962ee93d81e305e8fa249ee8a6b354.tar.gz glibc-5245d8a1fe962ee93d81e305e8fa249ee8a6b354.tar.bz2 |
Update.
2003-06-29 Ulrich Drepper <drepper@redhat.com>
* sysdeps/unix/sysv/linux/sleep.c (__sleep): Handle parameter
values which cannot be handled in one nanosleep call.
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | sysdeps/unix/sysv/linux/sleep.c | 78 |
2 files changed, 67 insertions, 16 deletions
@@ -1,3 +1,8 @@ +2003-06-29 Ulrich Drepper <drepper@redhat.com> + + * sysdeps/unix/sysv/linux/sleep.c (__sleep): Handle parameter + values which cannot be handled in one nanosleep call. + 2003-06-25 Alfred M. Szmidt <ams@kemisten.nu> * sysdeps/generic/bits/in.h (IPV6_HOPOPTS, IPV6_DSTOPTS): New macros. diff --git a/sysdeps/unix/sysv/linux/sleep.c b/sysdeps/unix/sysv/linux/sleep.c index 8df6717..ade9ff411 100644 --- a/sysdeps/unix/sysv/linux/sleep.c +++ b/sysdeps/unix/sysv/linux/sleep.c @@ -22,6 +22,17 @@ #include <time.h> #include <signal.h> #include <unistd.h> +#include <sys/param.h> + + +#if 0 +static void +cl (void *arg) +{ + (void) __sigprocmask (SIG_SETMASK, arg, (sigset_t *) NULL); +} +#endif + /* We are going to use the `nanosleep' syscall of the kernel. But the kernel does not implement the stupid SysV SIGCHLD vs. SIG_IGN @@ -29,7 +40,8 @@ unsigned int __sleep (unsigned int seconds) { - struct timespec ts = { .tv_sec = (long int) seconds, .tv_nsec = 0 }; + const unsigned int max = ((unsigned long int) (~((time_t) 0))) >> 1; + struct timespec ts; sigset_t set, oset; unsigned int result; @@ -42,12 +54,28 @@ __sleep (unsigned int seconds) return 0; } + ts.tv_sec = 0; + ts.tv_nsec = 0; + again: + if (sizeof (ts.tv_sec) <= sizeof (seconds)) + { + /* Since SECONDS is unsigned assigning the value to .tv_sec can + overflow it. In this case we have to wait in steps. */ + ts.tv_sec += MIN (seconds, max); + seconds -= (unsigned int) ts.tv_sec; + } + else + { + ts.tv_sec = (time_t) seconds; + seconds = 0; + } + /* Linux will wake up the system call, nanosleep, when SIGCHLD arrives even if SIGCHLD is ignored. We have to deal with it in libc. We block SIGCHLD first. */ - if (__sigemptyset (&set) < 0 - || __sigaddset (&set, SIGCHLD) < 0 - || __sigprocmask (SIG_BLOCK, &set, &oset)) + __sigemptyset (&set); + __sigaddset (&set, SIGCHLD); + if (__sigprocmask (SIG_BLOCK, &set, &oset)) return -1; /* If SIGCHLD is already blocked, we don't have to do anything. */ @@ -56,8 +84,8 @@ __sleep (unsigned int seconds) int saved_errno; struct sigaction oact; - if (__sigemptyset (&set) < 0 || __sigaddset (&set, SIGCHLD) < 0) - return -1; + __sigemptyset (&set); + __sigaddset (&set, SIGCHLD); /* We get the signal handler for SIGCHLD. */ if (__sigaction (SIGCHLD, (struct sigaction *) NULL, &oact) < 0) @@ -74,27 +102,45 @@ __sleep (unsigned int seconds) have to do anything here. */ if (oact.sa_handler == SIG_IGN) { + //__libc_cleanup_push (cl, &oset); + /* We should leave SIGCHLD blocked. */ - result = __nanosleep (&ts, &ts); + while (1) + { + result = __nanosleep (&ts, &ts); + + if (result != 0 || seconds == 0) + break; + + if (sizeof (ts.tv_sec) <= sizeof (seconds)) + { + ts.tv_sec = MIN (seconds, max); + seconds -= (unsigned int) ts.tv_nsec; + } + } + + //__libc_cleanup_pop (0); saved_errno = errno; /* Restore the original signal mask. */ (void) __sigprocmask (SIG_SETMASK, &oset, (sigset_t *) NULL); __set_errno (saved_errno); + + goto out; } - else - { - /* We should unblock SIGCHLD. Restore the original signal mask. */ - (void) __sigprocmask (SIG_SETMASK, &oset, (sigset_t *) NULL); - result = __nanosleep (&ts, &ts); - } + + /* We should unblock SIGCHLD. Restore the original signal mask. */ + (void) __sigprocmask (SIG_SETMASK, &oset, (sigset_t *) NULL); } - else - result = __nanosleep (&ts, &ts); + result = __nanosleep (&ts, &ts); + if (result == 0 && seconds != 0) + goto again; + + out: if (result != 0) /* Round remaining time. */ - result = (unsigned int) ts.tv_sec + (ts.tv_nsec >= 500000000L); + result = seconds + (unsigned int) ts.tv_sec + (ts.tv_nsec >= 500000000L); return result; } |