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.15 by greg, Sat Dec 28 18:05:14 2019 UTC vs.
Revision 3.18 by greg, Tue May 16 20:58:07 2023 UTC

# Line 17 | Line 17 | static const char      RCSid[] = "$Id$";
17   #include "rtio.h"
18  
19  
20 + SUBPROC         sp_inactive = SP_INACTIVE;
21 +
22 +
23   int
24   open_process(           /* open communication to separate process */
25 < SUBPROC *pd,
26 < char    *av[]
25 >        SUBPROC *pd,
26 >        char    *av[]
27   )
28   {
29          char    *compath;
30          int     p0[2], p1[2];
31  
32 <        pd->pid = -1;
33 <        pd->running = 0;                /* not going yet */
32 >        if (pd->pid > 0)
33 >                return(-1);             /* need to close, first */
34          
35 +        if ((pd->flags&(PF_FILT_INP|PF_FILT_OUT)) == (PF_FILT_INP|PF_FILT_OUT))
36 +                return(-1);             /* circular process not supported */
37 +
38 +        pd->flags &= ~PF_RUNNING;       /* not running for sure */
39 +        pd->pid = -1;
40 +
41          if (av == NULL)                 /* cloning operation? */
42                  compath = NULL;
43          else if ((compath = getpath(av[0], getenv("PATH"), X_OK)) == NULL)
44                  return(0);
45 <        if (pipe(p0) < 0 || pipe(p1) < 0)
45 >
46 >        if (pd->flags & PF_FILT_INP) {  /* filterning input stream? */
47 >                if ((pd->r < 0) | (pd->r == 1))
48 >                        return(-1);
49 >                p0[0] = pd->r;
50 >                p0[1] = -1;
51 >        } else if (pipe(p0) < 0)
52                  return(-1);
53 < #ifdef BSD
54 <        if (compath != NULL)
55 <                pd->pid = vfork();      /* more efficient with exec() */
56 <        else
57 < #endif
53 >
54 >        if (pd->flags & PF_FILT_OUT) {  /* filtering output stream? */
55 >                if (pd->w < 1)
56 >                        return(-1);
57 >                p1[0] = -1;
58 >                p1[1] = pd->w;
59 >        } else if (pipe(p1) < 0)
60 >                return(-1);
61          pd->pid = fork();
62          if (pd->pid == 0) {             /* if child... */
63                  close(p0[1]);
64                  close(p1[0]);
65                  if (p0[0] != 0) {       /* connect p0 to stdin */
66 <                        dup2(p0[0], 0);
66 >                        if (dup2(p0[0], 0) < 0)
67 >                                return(-1);
68                          close(p0[0]);
69                  }
70                  if (p1[1] != 1) {       /* connect p1 to stdout */
71 <                        dup2(p1[1], 1);
71 >                        if (dup2(p1[1], 1) < 0)
72 >                                return(-1);
73                          close(p1[1]);
74                  }
75                  if (compath == NULL)    /* just cloning? */
76                          return(0);
77 <                execv(compath, av);     /* else exec command */
77 >                                        /* else clear streams' FD_CLOEXEC */
78 >                if (p0[0] == 0)
79 >                        fcntl(0, F_SETFD, 0);
80 >                if (p1[1] == 1)
81 >                        fcntl(1, F_SETFD, 0);
82 >                execv(compath, av);     /* exec command */
83                  perror(compath);
84                  _exit(127);
85          }
86          if (pd->pid == -1)
87                  return(-1);
88 <        close(p0[0]);
89 <        close(p1[1]);
90 <        pd->r = p1[0];
91 <        pd->w = p0[1];
88 >                                        /* connect parent's streams */
89 >        if (!(pd->flags & PF_FILT_INP)) {
90 >                close(p0[0]);
91 >                pd->r = p1[0];
92 >        } else if (p1[0] != pd->r) {
93 >                if (dup2(p1[0], pd->r) < 0)
94 >                        return(-1);
95 >                close(p1[0]);
96 >        }
97 >        if (!(pd->flags & PF_FILT_OUT)) {
98 >                close(p1[1]);
99 >                pd->w = p0[1];
100 >        } else if (p0[1] != pd->w) {
101 >                if (dup2(p0[1], pd->w) < 0)
102 >                        return(-1);
103 >                close(p0[1]);
104 >        }
105          /*
106           * Close write stream on exec to avoid multiprocessing deadlock.
107           * No use in read stream without it, so set flag there as well.
108           * GW: This bug took me two days to figure out!!
109           */
110 <        fcntl(pd->r, F_SETFD, FD_CLOEXEC);
111 <        fcntl(pd->w, F_SETFD, FD_CLOEXEC);
112 <        pd->running = 1;
110 >        if (pd->r > 0)
111 >                fcntl(pd->r, F_SETFD, FD_CLOEXEC);
112 >        if (pd->w > 1)
113 >                fcntl(pd->w, F_SETFD, FD_CLOEXEC);
114 >        pd->flags |= PF_RUNNING;
115          return(PIPE_BUF);
116   }
117  
118  
119   int
120   close_processes(        /* close pipes and wait for processes to finish */
121 < SUBPROC pd[],
122 < int nproc
121 >        SUBPROC pd[],
122 >        int nproc
123   )
124   {
125          int     togo = nproc;
# Line 88 | Line 128 | int nproc
128          int     i;
129  
130          for (i = 0; i < nproc; i++)             /* close pipes, first */
131 <                if (pd[i].running) {
131 >                if (pd[i].flags & PF_RUNNING) {
132                          close(pd[i].w);
133                          close(pd[i].r);
134 <                        pd[i].running = 0;
134 >                        pd[i].flags &= ~PF_RUNNING;
135                  } else
136                          togo -= (pd[i].pid < 0);
137          if (nproc == 1) {                       /* await specific process? */
138 +                status = 0;
139                  if (waitpid(pd->pid, &status, 0) != pd->pid)
140                          return(-1);
141 <                pd->pid = -1;
141 >                *pd = sp_inactive;
142                  return(status>>8 & 0xff);
143          }
144                                                  /* else unordered wait */
145          while (togo > 0 && (pid = wait(&status)) >= 0) {
146                  for (i = nproc; i-- > 0; )
147                          if (pd[i].pid == pid) {
148 <                                pd[i].pid = -1;
148 >                                pd[i] = sp_inactive;
149                                  --togo;
150                                  break;
151                          }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines