rework mechanism for posix_spawnp calling posix_spawn
authorRich Felker <dalias@aerifal.cx>
Mon, 10 Sep 2018 20:13:29 +0000 (16:13 -0400)
committerRich Felker <dalias@aerifal.cx>
Wed, 12 Sep 2018 18:34:32 +0000 (14:34 -0400)
commitfe61a7aa53e68e8a17b5eb8d502e6fa314139ced
tree76624120cdf978e0d5c3a995d27e09eeba1a1151
parent039f1b3c564667ab6fc9955bd892c2e527eb80b0
rework mechanism for posix_spawnp calling posix_spawn

previously, a common __posix_spawnx backend was used that accepted an
additional argument for the execve variant to call in the child. this
moderately bloated up the posix_spawn function, shuffling arguments
between stack and/or registers to call a 7-argument function from a
6-argument one.

instead, tuck the exec function pointer in an unused part of the
(large) pthread_spawnattr_t structure, and have posix_spawnp duplicate
the attributes and fill in a pointer to __execvpe. the net code size
change is minimal, but the weight is shifted to the "heavier" function
which already pulls in more dependencies.

as a bonus, we get rid of an external symbol (__posix_spawnx) that had
no really good place for a declaration because it shouldn't have
existed to begin with.
include/spawn.h
src/process/posix_spawn.c
src/process/posix_spawnp.c