svn commit: trunk/busybox: archival archival/libunarchive debianuti etc...
vda at busybox.net
vda at busybox.net
Tue Jul 1 03:40:41 PDT 2008
Author: vda
Date: 2008-07-01 03:40:41 -0700 (Tue, 01 Jul 2008)
New Revision: 22582
Log:
*: introduce and use xvfork()
function old new delta
time_main 1052 1285 +233
crontab_main 623 856 +233
ifupdown_main 2202 2403 +201
xvfork - 20 +20
passwd_main 1049 1053 +4
grave 1068 1066 -2
script_main 935 921 -14
vfork_or_die 20 - -20
vfork_compressor 206 175 -31
open_as_user 109 - -109
popen2 218 - -218
edit_file 910 690 -220
run_command 268 - -268
------------------------------------------------------------------------------
(add/remove: 1/4 grow/shrink: 4/4 up/down: 691/-882) Total: -191 bytes
Modified:
trunk/busybox/archival/libunarchive/open_transformer.c
trunk/busybox/archival/tar.c
trunk/busybox/debianutils/start_stop_daemon.c
trunk/busybox/include/libbb.h
trunk/busybox/libbb/Kbuild
trunk/busybox/libbb/vfork_daemon_rexec.c
trunk/busybox/miscutils/crontab.c
trunk/busybox/miscutils/time.c
trunk/busybox/networking/ifupdown.c
trunk/busybox/networking/sendmail.c
trunk/busybox/shell/hush.c
trunk/busybox/util-linux/script.c
Changeset:
Modified: trunk/busybox/archival/libunarchive/open_transformer.c
===================================================================
--- trunk/busybox/archival/libunarchive/open_transformer.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/archival/libunarchive/open_transformer.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -25,9 +25,7 @@
if (pid == -1)
bb_perror_msg_and_die("can't fork");
#else
- pid = vfork();
- if (pid == -1)
- bb_perror_msg_and_die("can't vfork");
+ pid = xvfork();
#endif
if (pid == 0) {
Modified: trunk/busybox/archival/tar.c
===================================================================
--- trunk/busybox/archival/tar.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/archival/tar.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -536,9 +536,7 @@
(void) &zip_exec;
#endif
- gzipPid = vfork();
- if (gzipPid < 0)
- bb_perror_msg_and_die("can't vfork");
+ gzipPid = xvfork();
if (gzipPid == 0) {
/* child */
Modified: trunk/busybox/debianutils/start_stop_daemon.c
===================================================================
--- trunk/busybox/debianutils/start_stop_daemon.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/debianutils/start_stop_daemon.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -404,9 +404,7 @@
/* DAEMON_DEVNULL_STDIO is superfluous -
* it's always done by bb_daemonize() */
#else
- pid_t pid = vfork();
- if (pid < 0) /* error */
- bb_perror_msg_and_die("vfork");
+ pid_t pid = xvfork();
if (pid != 0) {
/* parent */
/* why _exit? the child may have changed the stack,
Modified: trunk/busybox/include/libbb.h
===================================================================
--- trunk/busybox/include/libbb.h 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/include/libbb.h 2008-07-01 10:40:41 UTC (rev 22582)
@@ -719,6 +719,8 @@
#define BB_EXECLP(prog,cmd,...) execlp(prog,cmd, __VA_ARGS__)
#endif
+pid_t xvfork(void) FAST_FUNC;
+
/* NOMMU friendy fork+exec */
pid_t spawn(char **argv) FAST_FUNC;
pid_t xspawn(char **argv) FAST_FUNC;
Modified: trunk/busybox/libbb/Kbuild
===================================================================
--- trunk/busybox/libbb/Kbuild 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/libbb/Kbuild 2008-07-01 10:40:41 UTC (rev 22582)
@@ -109,6 +109,7 @@
lib-y += xgetcwd.o
lib-y += xgethostbyname.o
lib-y += xreadlink.o
+lib-y += xvfork.o
# conditionally compiled objects:
lib-$(CONFIG_FEATURE_MOUNT_LOOP) += loop.o
Modified: trunk/busybox/libbb/vfork_daemon_rexec.c
===================================================================
--- trunk/busybox/libbb/vfork_daemon_rexec.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/libbb/vfork_daemon_rexec.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -226,9 +226,7 @@
if (re_execed)
return;
- pid = vfork();
- if (pid < 0) /* wtf? */
- bb_perror_msg_and_die("vfork");
+ pid = xvfork();
if (pid) /* parent */
exit(EXIT_SUCCESS);
/* child - re-exec ourself */
Modified: trunk/busybox/miscutils/crontab.c
===================================================================
--- trunk/busybox/miscutils/crontab.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/miscutils/crontab.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -38,10 +38,8 @@
static void edit_file(const struct passwd *pas, const char *file)
{
const char *ptr;
- int pid = vfork();
+ int pid = xvfork();
- if (pid < 0) /* failure */
- bb_perror_msg_and_die("vfork");
if (pid) { /* parent */
wait4pid(pid);
return;
@@ -65,9 +63,7 @@
pid_t pid;
char c;
- pid = vfork();
- if (pid < 0) /* ERROR */
- bb_perror_msg_and_die("vfork");
+ pid = xvfork();
if (pid) { /* PARENT */
if (wait4pid(pid) == 0) {
/* exitcode 0: child says it can read */
Modified: trunk/busybox/miscutils/time.c
===================================================================
--- trunk/busybox/miscutils/time.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/miscutils/time.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -372,9 +372,7 @@
void (*quit_signal)(int);
resp->elapsed_ms = monotonic_us() / 1000;
- pid = vfork(); /* Run CMD as child process. */
- if (pid < 0)
- bb_error_msg_and_die("cannot fork");
+ pid = xvfork(); /* Run CMD as child process. */
if (pid == 0) { /* If child. */
/* Don't cast execvp arguments; that causes errors on some systems,
versus merely warnings if the cast is left off. */
Modified: trunk/busybox/networking/ifupdown.c
===================================================================
--- trunk/busybox/networking/ifupdown.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/networking/ifupdown.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -1008,12 +1008,9 @@
xpiped_pair(outfd);
fflush(NULL);
- pid = vfork();
+ pid = xvfork();
- switch (pid) {
- case -1: /* failure */
- bb_perror_msg_and_die("vfork");
- case 0: /* child */
+ if (pid == 0) { /* child */
/* NB: close _first_, then move fds! */
close(infd.wr);
close(outfd.rd);
Modified: trunk/busybox/networking/sendmail.c
===================================================================
--- trunk/busybox/networking/sendmail.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/networking/sendmail.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -120,15 +120,6 @@
#undef err
}
-/* libbb candidate */
-static pid_t vfork_or_die(void)
-{
- pid_t pid = vfork();
- if (pid < 0)
- bb_perror_msg_and_die("vfork");
- return pid;
-}
-
static void launch_helper(const char **argv)
{
// setup vanilla unidirectional pipes interchange
@@ -137,7 +128,7 @@
xpipe(pipes);
xpipe(pipes+2);
- helper_pid = vfork_or_die();
+ helper_pid = xvfork();
idx = (!helper_pid) * 2;
xdup2(pipes[idx], STDIN_FILENO);
xdup2(pipes[3-idx], STDOUT_FILENO);
Modified: trunk/busybox/shell/hush.c
===================================================================
--- trunk/busybox/shell/hush.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/shell/hush.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -3096,9 +3096,11 @@
* huge=`cat TESTFILE` # will block here forever
* echo OK
*/
- pid = BB_MMU ? fork() : vfork();
+ pid = BB_MMU ? fork() : xvfork();
+#if BB_MMU
if (pid < 0)
- bb_perror_msg_and_die(BB_MMU ? "fork" : "vfork");
+ bb_perror_msg_and_die("fork");
+#endif
if (pid == 0) { /* child */
if (ENABLE_HUSH_JOB)
die_sleep = 0; /* let nofork's xfuncs die */
Modified: trunk/busybox/util-linux/script.c
===================================================================
--- trunk/busybox/util-linux/script.c 2008-07-01 10:05:12 UTC (rev 22581)
+++ trunk/busybox/util-linux/script.c 2008-07-01 10:40:41 UTC (rev 22582)
@@ -87,10 +87,7 @@
/* TODO: SIGWINCH? pass window size changes down to slave? */
- child_pid = vfork();
- if (child_pid < 0) {
- bb_perror_msg_and_die("vfork");
- }
+ child_pid = xvfork();
if (child_pid) {
/* parent */
More information about the busybox-cvs
mailing list