2020-06-15 14:18:57 +00:00
|
|
|
/*-*- mode:c;indent-tabs-mode:nil;c-basic-offset:2;tab-width:8;coding:utf-8 -*-│
|
2023-12-08 03:11:56 +00:00
|
|
|
│ vi: set et ft=c ts=2 sts=2 sw=2 fenc=utf-8 :vi │
|
2020-06-15 14:18:57 +00:00
|
|
|
╞══════════════════════════════════════════════════════════════════════════════╡
|
2021-01-28 03:34:02 +00:00
|
|
|
│ Copyright 2021 Justine Alexandra Roberts Tunney │
|
2020-06-15 14:18:57 +00:00
|
|
|
│ │
|
2020-12-28 01:18:44 +00:00
|
|
|
│ Permission to use, copy, modify, and/or distribute this software for │
|
|
|
|
│ any purpose with or without fee is hereby granted, provided that the │
|
|
|
|
│ above copyright notice and this permission notice appear in all copies. │
|
2020-06-15 14:18:57 +00:00
|
|
|
│ │
|
2020-12-28 01:18:44 +00:00
|
|
|
│ THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL │
|
|
|
|
│ WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED │
|
|
|
|
│ WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE │
|
|
|
|
│ AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL │
|
|
|
|
│ DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR │
|
|
|
|
│ PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER │
|
|
|
|
│ TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR │
|
|
|
|
│ PERFORMANCE OF THIS SOFTWARE. │
|
2020-06-15 14:18:57 +00:00
|
|
|
╚─────────────────────────────────────────────────────────────────────────────*/
|
2021-01-28 03:34:02 +00:00
|
|
|
#include "libc/assert.h"
|
|
|
|
#include "libc/calls/calls.h"
|
2022-06-09 03:01:28 +00:00
|
|
|
#include "libc/calls/struct/rusage.h"
|
2021-01-28 03:34:02 +00:00
|
|
|
#include "libc/errno.h"
|
2023-08-14 03:31:27 +00:00
|
|
|
#include "libc/stdio/internal.h"
|
2021-01-28 03:34:02 +00:00
|
|
|
#include "libc/stdio/stdio.h"
|
|
|
|
#include "libc/sysv/errfuns.h"
|
2020-06-15 14:18:57 +00:00
|
|
|
|
2021-01-28 03:34:02 +00:00
|
|
|
/**
|
|
|
|
* Closes stream created by popen().
|
2022-11-06 02:49:41 +00:00
|
|
|
*
|
|
|
|
* This function may be interrupted or cancelled, however it won't
|
|
|
|
* actually return until the child process has terminated. Thus we
|
|
|
|
* always release the resource, and errors are purely advisory.
|
|
|
|
*
|
2021-01-28 03:34:02 +00:00
|
|
|
* @return termination status of subprocess, or -1 w/ ECHILD
|
2022-11-06 02:49:41 +00:00
|
|
|
* @raise ECANCELED if thread was cancelled in masked mode
|
|
|
|
* @raise ECHILD if child pid didn't exist
|
|
|
|
* @raise EINTR if signal was delivered
|
Make improvements
- We now serialize the file descriptor table when spawning / executing
processes on Windows. This means you can now inherit more stuff than
just standard i/o. It's needed by bash, which duplicates the console
to file descriptor #255. We also now do a better job serializing the
environment variables, so you're less likely to encounter E2BIG when
using your bash shell. We also no longer coerce environ to uppercase
- execve() on Windows now remotely controls its parent process to make
them spawn a replacement for itself. Then it'll be able to terminate
immediately once the spawn succeeds, without having to linger around
for the lifetime as a shell process for proxying the exit code. When
process worker thread running in the parent sees the child die, it's
given a handle to the new child, to replace it in the process table.
- execve() and posix_spawn() on Windows will now provide CreateProcess
an explicit handle list. This allows us to remove handle locks which
enables better fork/spawn concurrency, with seriously correct thread
safety. Other codebases like Go use the same technique. On the other
hand fork() still favors the conventional WIN32 inheritence approach
which can be a little bit messy, but is *controlled* by guaranteeing
perfectly clean slates at both the spawning and execution boundaries
- sigset_t is now 64 bits. Having it be 128 bits was a mistake because
there's no reason to use that and it's only supported by FreeBSD. By
using the system word size, signal mask manipulation on Windows goes
very fast. Furthermore @asyncsignalsafe funcs have been rewritten on
Windows to take advantage of signal masking, now that it's much more
pleasant to use.
- All the overlapped i/o code on Windows has been rewritten for pretty
good signal and cancelation safety. We're now able to ensure overlap
data structures are cleaned up so long as you don't longjmp() out of
out of a signal handler that interrupted an i/o operation. Latencies
are also improved thanks to the removal of lots of "busy wait" code.
Waits should be optimal for everything except poll(), which shall be
the last and final demon we slay in the win32 i/o horror show.
- getrusage() on Windows is now able to report RUSAGE_CHILDREN as well
as RUSAGE_SELF, thanks to aggregation in the process manager thread.
2023-10-08 12:36:18 +00:00
|
|
|
* @cancelationpoint
|
2021-01-28 03:34:02 +00:00
|
|
|
*/
|
|
|
|
int pclose(FILE *f) {
|
2022-11-06 02:49:41 +00:00
|
|
|
int e, rc, ws, pid;
|
|
|
|
bool iscancelled, wasinterrupted;
|
2021-01-28 03:34:02 +00:00
|
|
|
pid = f->pid;
|
|
|
|
fclose(f);
|
Apply clang-format update to repo (#1154)
Commit bc6c183 introduced a bunch of discrepancies between what files
look like in the repo and what clang-format says they should look like.
However, there were already a few discrepancies prior to that. Most of
these discrepancies seemed to be unintentional, but a few of them were
load-bearing (e.g., a #include that violated header ordering needing
something to have been #defined by a 'later' #include.)
I opted to take what I hope is a relatively smooth-brained approach: I
reverted the .clang-format change, ran clang-format on the whole repo,
reapplied the .clang-format change, reran clang-format again, and then
reverted the commit that contained the first run. Thus the full effect
of this PR should only be to apply the changed formatting rules to the
repo, and from skimming the results, this seems to be the case.
My work can be checked by applying the short, manual commits, and then
rerunning the command listed in the autogenerated commits (those whose
messages I have prefixed auto:) and seeing if your results agree.
It might be that the other diffs should be fixed at some point but I'm
leaving that aside for now.
fd '\.c(c|pp)?$' --print0| xargs -0 clang-format -i
2024-04-25 17:38:00 +00:00
|
|
|
if (!pid)
|
|
|
|
return 0;
|
2022-11-06 02:49:41 +00:00
|
|
|
iscancelled = false;
|
|
|
|
wasinterrupted = false;
|
|
|
|
for (e = errno;;) {
|
|
|
|
if (wait4(pid, &ws, 0, 0) != -1) {
|
|
|
|
rc = ws;
|
|
|
|
break;
|
|
|
|
} else if (errno == ECANCELED) {
|
|
|
|
iscancelled = true;
|
|
|
|
errno = e;
|
|
|
|
} else if (errno == EINTR) {
|
|
|
|
wasinterrupted = true;
|
|
|
|
errno = e;
|
|
|
|
} else {
|
|
|
|
rc = echild();
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (iscancelled) {
|
|
|
|
return ecanceled();
|
|
|
|
} else if (wasinterrupted) {
|
|
|
|
return eintr();
|
2021-01-28 03:34:02 +00:00
|
|
|
} else {
|
2022-11-06 02:49:41 +00:00
|
|
|
return rc;
|
2021-01-28 03:34:02 +00:00
|
|
|
}
|
2020-06-15 14:18:57 +00:00
|
|
|
}
|