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.5 by greg, Fri Sep 17 21:43:49 2004 UTC vs.
Revision 3.11 by greg, Fri Mar 4 02:48:14 2016 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 = 0;
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 = fork()) == 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 <        if (!pd->running)
87 <                return(0);
88 <        close(pd->r);
89 <        close(pd->w);
90 <        pd->running = 0;
91 <        if (waitpid(pd->pid, &status, 0) == pd->pid)
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 >                }
92 >        if (nproc == 1) {                       /* await specific process? */
93 >                if (waitpid(pd->pid, &status, 0) != pd->pid)
94 >                        return(-1);
95 >                pd->pid = 0;
96                  return(status>>8 & 0xff);
97 <        return(-1);             /* ? unknown status */
97 >        }
98 >                                                /* else unordered wait */
99 >        while (togo > 0 && (pid = wait(&status)) >= 0) {
100 >                for (i = nproc; i-- > 0; )
101 >                        if (pd[i].pid == pid) {
102 >                                pd[i].pid = 0;
103 >                                --togo;
104 >                                break;
105 >                        }
106 >                if (i < 0)
107 >                        continue;               /* child we don't know? */
108 >                status = status>>8 & 0xff;
109 >                if (status)                     /* record non-zero status */
110 >                        rtn_status = status;
111 >        }
112 >        if (togo)                               /* child went missing? */
113 >                return(-1);
114 >        return(rtn_status);
115   }
79
80

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines