search for: c2ac11ef

...= -1 || fcntl (h->fds[1], F_SETFL, f | O_NONBLOCK) == -1) { + nbdkit_error ("fcntl: %m"); + close (h->fds[0]); + close (h->fds[1]); + free (h); + return NULL; + } + } +#endif retry: h->fd = -1; diff --git a/server/quit.c b/server/quit.c index c2ac11ef..ffd3fecb 100644 --- a/server/quit.c +++ b/server/quit.c @@ -32,6 +32,7 @@ #include <config.h> +#include <fcntl.h> #include <stdio.h> #include <stdlib.h> #include <stdarg.h> @@ -39,6 +40,7 @@ #include <unistd.h> #include "internal.h" +#include...

There's enough here to need a review; some of it probably needs backporting to stable-1.12. This probably breaks tests on Haiku or other platforms that have not been as on-the-ball about atomic CLOEXEC; feel free to report issues that arise, and I'll help come up with workarounds (even if we end up leaving a rare fd leak on less-capable systems). Meanwhile, I'm still working on my

This is a major rewrite compared to my v1 series, where I've tried a lot harder to ensure that we still accommodate building on Haiku (although I have not actually yet fired up a Haiku VM to try it for myself). I also managed to make the sh plugin fully parallel, on capable platforms. See also my question on patch 10 on whether I've picked the best naming convention. Eric Blake (17):