ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/radiance/ray/src/common/unix_process.c
(Generate patch)

Comparing ray/src/common/unix_process.c (file contents):
Revision 3.3 by greg, Mon Nov 10 16:41:52 2003 UTC vs.
Revision 3.13 by greg, Fri Mar 24 12:56:12 2017 UTC

# Line 12 | Line 12 | static const char      RCSid[] = "$Id$";
12  
13   #include <sys/types.h>
14   #include <sys/wait.h>
15 + #include <fcntl.h>
16 + #include <stdlib.h>
17  
18   #include "rtprocess.h"
19 + #include "rtio.h"
20  
21  
22   int
# Line 22 | Line 25 | SUBPROC *pd,
25   char    *av[]
26   )
27   {
25        extern char     *getpath(), *getenv();
28          char    *compath;
29          int     p0[2], p1[2];
30  
31 <        pd->running = 0; /* not yet */
32 <                                        /* find executable */
33 <        compath = getpath(av[0], getenv("PATH"), 1);
34 <        if (compath == 0)
31 >        pd->pid = -1;
32 >        pd->running = 0;                /* not going yet */
33 >        
34 >        if (av == NULL)                 /* cloning operation? */
35 >                compath = NULL;
36 >        else if ((compath = getpath(av[0], getenv("PATH"), X_OK)) == NULL)
37                  return(0);
38          if (pipe(p0) < 0 || pipe(p1) < 0)
39                  return(-1);
40 <        if ((pd->pid = vfork()) == 0) {         /* if child */
40 >        if ((pd->pid = fork()) == 0) {  /* if child... */
41                  close(p0[1]);
42                  close(p1[0]);
43                  if (p0[0] != 0) {       /* connect p0 to stdin */
# Line 44 | Line 48 | char   *av[]
48                          dup2(p1[1], 1);
49                          close(p1[1]);
50                  }
51 <                execv(compath, av);     /* exec command */
51 >                if (compath == NULL)    /* just cloning? */
52 >                        return(0);
53 >                execv(compath, av);     /* else exec command */
54                  perror(compath);
55                  _exit(127);
56          }
# Line 54 | Line 60 | char   *av[]
60          close(p1[1]);
61          pd->r = p1[0];
62          pd->w = p0[1];
63 +        /*
64 +         * Close write stream on exec to avoid multiprocessing deadlock.
65 +         * No use in read stream without it, so set flag there as well.
66 +         * GW: This bug took me two days to figure out!!
67 +         */
68 +        fcntl(pd->r, F_SETFD, FD_CLOEXEC);
69 +        fcntl(pd->w, F_SETFD, FD_CLOEXEC);
70          pd->running = 1;
71          return(PIPE_BUF);
72   }
73  
74  
62
75   int
76 < close_process(          /* close pipes and wait for process */
77 < SUBPROC *pd
76 > close_processes(        /* close pipes and wait for processes to finish */
77 > SUBPROC pd[],
78 > int nproc
79   )
80   {
81 <        int     pid, status;
81 >        int     togo = nproc;
82 >        int     status, rtn_status = 0;
83 >        RT_PID  pid;
84 >        int     i;
85  
86 <        close(pd->r);
87 <        close(pd->w);
88 <        pd->running = 0;
89 <        while ((pid = wait(&status)) != -1)
90 <                if (pid == pd->pid)
91 <                        return(status>>8 & 0xff);
92 <        return(-1);             /* ? unknown status */
86 >        for (i = 0; i < nproc; i++)             /* close pipes, first */
87 >                if (pd[i].running) {
88 >                        close(pd[i].w);
89 >                        close(pd[i].r);
90 >                        pd[i].running = 0;
91 >                } else
92 >                        togo -= (pd[i].pid < 0);
93 >        if (nproc == 1) {                       /* await specific process? */
94 >                if (waitpid(pd->pid, &status, 0) != pd->pid)
95 >                        return(-1);
96 >                pd->pid = -1;
97 >                return(status>>8 & 0xff);
98 >        }
99 >                                                /* else unordered wait */
100 >        while (togo > 0 && (pid = wait(&status)) >= 0) {
101 >                for (i = nproc; i-- > 0; )
102 >                        if (pd[i].pid == pid) {
103 >                                pd[i].pid = -1;
104 >                                --togo;
105 >                                break;
106 >                        }
107 >                if (i < 0)
108 >                        continue;               /* child we don't know? */
109 >                status = status>>8 & 0xff;
110 >                if (status)                     /* record non-zero status */
111 >                        rtn_status = status;
112 >        }
113 >        if (togo)                               /* child went missing? */
114 >                return(-1);
115 >        return(rtn_status);
116   }
78
79

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines