phew, yay.  because i didn't see anything obviously wrong with the #16962 commit.

also, regarding checking for EINTR on some of the close() calls.  It isn't a big deal.  This _posixsubprocess.c code that contains those checks is always executing single-threaded post-fork so there is no chance of a race condition even though the EINTR check and retry are pointless as Linus mentions.  They will never cause errant behavior but I'll "fix" them in a future commit for consistency.