*: 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
This commit is contained in:
Denis Vlasenko 2008-07-01 10:40:41 +00:00
parent 148f67af9d
commit 3da5572bfa
12 changed files with 18 additions and 42 deletions

View File

@ -25,9 +25,7 @@ int FAST_FUNC open_transformer(int src_fd,
if (pid == -1) if (pid == -1)
bb_perror_msg_and_die("can't fork"); bb_perror_msg_and_die("can't fork");
#else #else
pid = vfork(); pid = xvfork();
if (pid == -1)
bb_perror_msg_and_die("can't vfork");
#endif #endif
if (pid == 0) { if (pid == 0) {

View File

@ -536,9 +536,7 @@ static void NOINLINE vfork_compressor(int tar_fd, int gzip)
(void) &zip_exec; (void) &zip_exec;
#endif #endif
gzipPid = vfork(); gzipPid = xvfork();
if (gzipPid < 0)
bb_perror_msg_and_die("can't vfork");
if (gzipPid == 0) { if (gzipPid == 0) {
/* child */ /* child */

View File

@ -404,9 +404,7 @@ int start_stop_daemon_main(int argc ATTRIBUTE_UNUSED, char **argv)
/* DAEMON_DEVNULL_STDIO is superfluous - /* DAEMON_DEVNULL_STDIO is superfluous -
* it's always done by bb_daemonize() */ * it's always done by bb_daemonize() */
#else #else
pid_t pid = vfork(); pid_t pid = xvfork();
if (pid < 0) /* error */
bb_perror_msg_and_die("vfork");
if (pid != 0) { if (pid != 0) {
/* parent */ /* parent */
/* why _exit? the child may have changed the stack, /* why _exit? the child may have changed the stack,

View File

@ -719,6 +719,8 @@ int bb_execvp(const char *file, char *const argv[]) FAST_FUNC;
#define BB_EXECLP(prog,cmd,...) execlp(prog,cmd, __VA_ARGS__) #define BB_EXECLP(prog,cmd,...) execlp(prog,cmd, __VA_ARGS__)
#endif #endif
pid_t xvfork(void) FAST_FUNC;
/* NOMMU friendy fork+exec */ /* NOMMU friendy fork+exec */
pid_t spawn(char **argv) FAST_FUNC; pid_t spawn(char **argv) FAST_FUNC;
pid_t xspawn(char **argv) FAST_FUNC; pid_t xspawn(char **argv) FAST_FUNC;

View File

@ -109,6 +109,7 @@ lib-y += xfunc_die.o
lib-y += xgetcwd.o lib-y += xgetcwd.o
lib-y += xgethostbyname.o lib-y += xgethostbyname.o
lib-y += xreadlink.o lib-y += xreadlink.o
lib-y += xvfork.o
# conditionally compiled objects: # conditionally compiled objects:
lib-$(CONFIG_FEATURE_MOUNT_LOOP) += loop.o lib-$(CONFIG_FEATURE_MOUNT_LOOP) += loop.o

View File

@ -226,9 +226,7 @@ void FAST_FUNC forkexit_or_rexec(char **argv)
if (re_execed) if (re_execed)
return; return;
pid = vfork(); pid = xvfork();
if (pid < 0) /* wtf? */
bb_perror_msg_and_die("vfork");
if (pid) /* parent */ if (pid) /* parent */
exit(EXIT_SUCCESS); exit(EXIT_SUCCESS);
/* child - re-exec ourself */ /* child - re-exec ourself */

View File

@ -38,10 +38,8 @@ static void change_user(const struct passwd *pas)
static void edit_file(const struct passwd *pas, const char *file) static void edit_file(const struct passwd *pas, const char *file)
{ {
const char *ptr; const char *ptr;
int pid = vfork(); int pid = xvfork();
if (pid < 0) /* failure */
bb_perror_msg_and_die("vfork");
if (pid) { /* parent */ if (pid) { /* parent */
wait4pid(pid); wait4pid(pid);
return; return;
@ -65,9 +63,7 @@ static int open_as_user(const struct passwd *pas, const char *file)
pid_t pid; pid_t pid;
char c; char c;
pid = vfork(); pid = xvfork();
if (pid < 0) /* ERROR */
bb_perror_msg_and_die("vfork");
if (pid) { /* PARENT */ if (pid) { /* PARENT */
if (wait4pid(pid) == 0) { if (wait4pid(pid) == 0) {
/* exitcode 0: child says it can read */ /* exitcode 0: child says it can read */

View File

@ -372,9 +372,7 @@ static void run_command(char *const *cmd, resource_t *resp)
void (*quit_signal)(int); void (*quit_signal)(int);
resp->elapsed_ms = monotonic_us() / 1000; resp->elapsed_ms = monotonic_us() / 1000;
pid = vfork(); /* Run CMD as child process. */ pid = xvfork(); /* Run CMD as child process. */
if (pid < 0)
bb_error_msg_and_die("cannot fork");
if (pid == 0) { /* If child. */ if (pid == 0) { /* If child. */
/* Don't cast execvp arguments; that causes errors on some systems, /* Don't cast execvp arguments; that causes errors on some systems,
versus merely warnings if the cast is left off. */ versus merely warnings if the cast is left off. */

View File

@ -1008,12 +1008,9 @@ static int popen2(FILE **in, FILE **out, char *command, char *param)
xpiped_pair(outfd); xpiped_pair(outfd);
fflush(NULL); fflush(NULL);
pid = vfork(); pid = xvfork();
switch (pid) { if (pid == 0) { /* child */
case -1: /* failure */
bb_perror_msg_and_die("vfork");
case 0: /* child */
/* NB: close _first_, then move fds! */ /* NB: close _first_, then move fds! */
close(infd.wr); close(infd.wr);
close(outfd.rd); close(outfd.rd);

View File

@ -120,15 +120,6 @@ static void signal_handler(int signo)
#undef err #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) static void launch_helper(const char **argv)
{ {
// setup vanilla unidirectional pipes interchange // setup vanilla unidirectional pipes interchange
@ -137,7 +128,7 @@ static void launch_helper(const char **argv)
xpipe(pipes); xpipe(pipes);
xpipe(pipes+2); xpipe(pipes+2);
helper_pid = vfork_or_die(); helper_pid = xvfork();
idx = (!helper_pid) * 2; idx = (!helper_pid) * 2;
xdup2(pipes[idx], STDIN_FILENO); xdup2(pipes[idx], STDIN_FILENO);
xdup2(pipes[3-idx], STDOUT_FILENO); xdup2(pipes[3-idx], STDOUT_FILENO);

View File

@ -3096,9 +3096,11 @@ static FILE *generate_stream_from_list(struct pipe *head)
* huge=`cat TESTFILE` # will block here forever * huge=`cat TESTFILE` # will block here forever
* echo OK * echo OK
*/ */
pid = BB_MMU ? fork() : vfork(); pid = BB_MMU ? fork() : xvfork();
#if BB_MMU
if (pid < 0) if (pid < 0)
bb_perror_msg_and_die(BB_MMU ? "fork" : "vfork"); bb_perror_msg_and_die("fork");
#endif
if (pid == 0) { /* child */ if (pid == 0) { /* child */
if (ENABLE_HUSH_JOB) if (ENABLE_HUSH_JOB)
die_sleep = 0; /* let nofork's xfuncs die */ die_sleep = 0; /* let nofork's xfuncs die */

View File

@ -87,10 +87,7 @@ int script_main(int argc ATTRIBUTE_UNUSED, char **argv)
/* TODO: SIGWINCH? pass window size changes down to slave? */ /* TODO: SIGWINCH? pass window size changes down to slave? */
child_pid = vfork(); child_pid = xvfork();
if (child_pid < 0) {
bb_perror_msg_and_die("vfork");
}
if (child_pid) { if (child_pid) {
/* parent */ /* parent */