diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2009-08-13 07:49:07 -0400 |
|---|---|---|
| committer | Austin Foxley <austinf@cetoncorp.com> | 2009-09-18 11:45:00 -0700 |
| commit | 0d5b9c86de553d3e620d0594268f3978f31220b6 (patch) | |
| tree | e6a3e50b896a1cad1a3773f3536cf476ac2e73b1 /libc | |
| parent | c72a5ae6e97aeffdb07cacde7d3c972ad8c60bd5 (diff) | |
| download | uClibc-alpine-0d5b9c86de553d3e620d0594268f3978f31220b6.tar.bz2 uClibc-alpine-0d5b9c86de553d3e620d0594268f3978f31220b6.tar.xz | |
daemon: fix up INTERNAL_SYSCALL() usage
Make sure we declare the error properly in case a port uses it, and fix the
invocation of exit(). Since clone() will be returning a pid, assume that
the value will not have the MSB set (negative) to simplify the error test a
bit more. If gcc supports it, force this function to always be heavily
optimized in a bid to avoid stack usage as much as possible.
Signed-off-by: James Coleman <james.coleman@ubicom.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Austin Foxley <austinf@cetoncorp.com>
Diffstat (limited to 'libc')
| -rw-r--r-- | libc/unistd/daemon.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/libc/unistd/daemon.c b/libc/unistd/daemon.c index e5417227b..c270603f7 100644 --- a/libc/unistd/daemon.c +++ b/libc/unistd/daemon.c @@ -60,12 +60,14 @@ /* use clone() to get fork() like behavior here -- we just want to disassociate * from the controlling terminal */ -static inline pid_t _fork_parent(void) +static inline attribute_optimize("O3") +pid_t _fork_parent(void) { - register unsigned long ret = INTERNAL_SYSCALL(clone, wtf, 2, CLONE_VM, 0); - if (ret != -1 && ret != 0) + INTERNAL_SYSCALL_DECL(err); + register long ret = INTERNAL_SYSCALL(clone, err, 2, CLONE_VM, 0); + if (ret > 0) /* parent needs to die now w/out touching stack */ - INTERNAL_SYSCALL(exit, wtf, 0); + INTERNAL_SYSCALL(exit, err, 1, 0); return ret; } static inline pid_t fork_parent(void) |