diff options
| author | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2017-11-06 09:31:37 -0200 |
|---|---|---|
| committer | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2017-11-15 15:45:39 -0200 |
| commit | 1ba1d8a44d244110c1f8d768f098ac44351acb76 (patch) | |
| tree | cb0d5bb38299dd3755f7496d111bdd00e450e8a4 | |
| parent | ad4f43a2344864ae2304060dcc722a7a63bad1b4 (diff) | |
| download | glibc-1ba1d8a44d244110c1f8d768f098ac44351acb76.zip glibc-1ba1d8a44d244110c1f8d768f098ac44351acb76.tar.gz glibc-1ba1d8a44d244110c1f8d768f098ac44351acb76.tar.bz2 | |
Optimize sigrelse implementation
This patch simplifies sighold a bit by removing an extra sigprocmask
and using SIG_BLOCK (which union of the current set and the set argument).
Checked on x86_64-linux-gnu.
* signal/sighold.c (sighold): Optimize implementation.
Signed-off-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
| -rw-r--r-- | ChangeLog | 2 | ||||
| -rw-r--r-- | signal/sigrelse.c | 12 |
2 files changed, 5 insertions, 9 deletions
@@ -1,5 +1,7 @@ 2017-11-15 Adhemerval Zanella <adhemerval.zanella@linaro.org> + * signal/sigrelse.c (sigrelse): Optimize implementation. + * sysdeps/posix/sigpause.c (do_sigpause): Remove. (__sigpause): Rely on __sigsuspend to implement single thread optimization. Add LIBC_CANCEL_HANDLED for cancellation marking. diff --git a/signal/sigrelse.c b/signal/sigrelse.c index e41c2bd..61f240b 100644 --- a/signal/sigrelse.c +++ b/signal/sigrelse.c @@ -26,14 +26,8 @@ sigrelse (int sig) { sigset_t set; - /* Retrieve current signal set. */ - if (__sigprocmask (SIG_SETMASK, NULL, &set) < 0) + sigemptyset (&set); + if (sigaddset (&set, sig) < 0) return -1; - - /* Remove the specified signal. */ - if (sigdelset (&set, sig) < 0) - return -1; - - /* Set the new mask. */ - return __sigprocmask (SIG_SETMASK, &set, NULL); + return __sigprocmask (SIG_UNBLOCK, &set, NULL); } |