streamline old-kernel fallback path of pipe2 to use syscalls directly
authorRich Felker <dalias@aerifal.cx>
Sun, 3 Feb 2013 21:09:05 +0000 (16:09 -0500)
committerRich Felker <dalias@aerifal.cx>
Sun, 3 Feb 2013 21:09:05 +0000 (16:09 -0500)
also, don't waste code/time on F_GETFL since pipes always have blank
flags initially (at least on old kernels, which are all this fallback
code matters for).

src/unistd/pipe2.c

index 973e8ec..bbeabcb 100644 (file)
@@ -11,12 +11,12 @@ int pipe2(int fd[2], int flag)
        ret = __syscall(SYS_pipe, fd);
        if (ret) return __syscall_ret(ret);
        if (flag & O_CLOEXEC) {
        ret = __syscall(SYS_pipe, fd);
        if (ret) return __syscall_ret(ret);
        if (flag & O_CLOEXEC) {
-               fcntl(fd[0], F_SETFD, FD_CLOEXEC);
-               fcntl(fd[1], F_SETFD, FD_CLOEXEC);
+               __syscall(SYS_fcntl, fd[0], F_SETFD, FD_CLOEXEC);
+               __syscall(SYS_fcntl, fd[1], F_SETFD, FD_CLOEXEC);
        }
        if (flag & O_NONBLOCK) {
        }
        if (flag & O_NONBLOCK) {
-               fcntl(fd[0], F_SETFL, fcntl(fd[0], F_GETFL) | O_NONBLOCK);
-               fcntl(fd[1], F_SETFL, fcntl(fd[1], F_GETFL) | O_NONBLOCK);
+               __syscall(SYS_fcntl, fd[0], F_SETFL, O_NONBLOCK);
+               __syscall(SYS_fcntl, fd[1], F_SETFL, O_NONBLOCK);
        }
        return 0;
 }
        }
        return 0;
 }