diff options
| author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-01-02 19:55:04 +0000 |
|---|---|---|
| committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-01-02 19:55:04 +0000 |
| commit | fb0eba706cccd510d99c4c5339a76dd15bc8a628 (patch) | |
| tree | 45122b93a1a5d628e4aa146251e34397094de817 /shell | |
| parent | 27963980dbe1262fd6c447fc7d06839aea0861bc (diff) | |
| download | busybox-w32-fb0eba706cccd510d99c4c5339a76dd15bc8a628.tar.gz busybox-w32-fb0eba706cccd510d99c4c5339a76dd15bc8a628.tar.bz2 busybox-w32-fb0eba706cccd510d99c4c5339a76dd15bc8a628.zip | |
libbb: introduce and use safe_waitpid (loops in EINTR)
*: use more approproate (shorter) versions of wait()
function old new delta
safe_waitpid - 48 +48
wait_any_nohang - 17 +17
send_tree 365 369 +4
processorstop 432 435 +3
text_yank 110 108 -2
make_human_readable_str 202 200 -2
crond_main 1368 1366 -2
handle_sigchld 49 43 -6
reapchild 166 159 -7
custom 260 250 -10
checkscript 191 177 -14
wait_nohang 17 - -17
wait_pid 43 - -43
------------------------------------------------------------------------------
(add/remove: 2/2 grow/shrink: 2/7 up/down: 72/-103) Total: -31 bytes
Diffstat (limited to 'shell')
| -rw-r--r-- | shell/ash.c | 2 | ||||
| -rw-r--r-- | shell/hush.c | 1 | ||||
| -rw-r--r-- | shell/msh.c | 2 |
3 files changed, 3 insertions, 2 deletions
diff --git a/shell/ash.c b/shell/ash.c index a5b19c863..96563bf06 100644 --- a/shell/ash.c +++ b/shell/ash.c | |||
| @@ -3768,7 +3768,7 @@ waitproc(int block, int *status) | |||
| 3768 | #endif | 3768 | #endif |
| 3769 | if (block == 0) | 3769 | if (block == 0) |
| 3770 | flags |= WNOHANG; | 3770 | flags |= WNOHANG; |
| 3771 | return wait3(status, flags, (struct rusage *)NULL); | 3771 | return waitpid(-1, status, flags); // safe_waitpid? |
| 3772 | } | 3772 | } |
| 3773 | 3773 | ||
| 3774 | /* | 3774 | /* |
diff --git a/shell/hush.c b/shell/hush.c index cb2c3e98e..b08fe10b6 100644 --- a/shell/hush.c +++ b/shell/hush.c | |||
| @@ -1649,6 +1649,7 @@ static int checkjobs(struct pipe* fg_pipe) | |||
| 1649 | // + killall -STOP cat | 1649 | // + killall -STOP cat |
| 1650 | 1650 | ||
| 1651 | wait_more: | 1651 | wait_more: |
| 1652 | // TODO: safe_waitpid? | ||
| 1652 | while ((childpid = waitpid(-1, &status, attributes)) > 0) { | 1653 | while ((childpid = waitpid(-1, &status, attributes)) > 0) { |
| 1653 | const int dead = WIFEXITED(status) || WIFSIGNALED(status); | 1654 | const int dead = WIFEXITED(status) || WIFSIGNALED(status); |
| 1654 | 1655 | ||
diff --git a/shell/msh.c b/shell/msh.c index 9edf793ab..9e9b798a1 100644 --- a/shell/msh.c +++ b/shell/msh.c | |||
| @@ -4162,7 +4162,7 @@ static int grave(int quoted) | |||
| 4162 | return 0; | 4162 | return 0; |
| 4163 | } | 4163 | } |
| 4164 | if (i != 0) { | 4164 | if (i != 0) { |
| 4165 | waitpid(i, NULL, 0); | 4165 | waitpid(i, NULL, 0); // safe_waitpid? |
| 4166 | global_env.iop->argp->aword = ++cp; | 4166 | global_env.iop->argp->aword = ++cp; |
| 4167 | close(pf[1]); | 4167 | close(pf[1]); |
| 4168 | PUSHIO(afile, remap(pf[0]), | 4168 | PUSHIO(afile, remap(pf[0]), |
