]> git.ipfire.org Git - thirdparty/curl.git/commit
socketpair: add `eventfd` and use `SOCK_NONBLOCK` for `socketpair()`
authorAndy Pan <i@andypan.me>
Sat, 1 Jun 2024 18:26:11 +0000 (02:26 +0800)
committerDaniel Stenberg <daniel@haxx.se>
Tue, 4 Jun 2024 21:45:36 +0000 (23:45 +0200)
commit23fe1a52dc8a2ffd74e19b956927bbccdc07f15f
tree022e3636be15f43843f5201ff88be5b0ca5e8c90
parentb71916b859b98c942e7754ecb469430ebbb9b665
socketpair: add `eventfd` and use `SOCK_NONBLOCK` for `socketpair()`

Currently, we use `pipe` for `wakeup_create`, which requires ***two***
file descriptors. Furthermore, given its complexity inside, `pipe` is a
bit heavyweight for just a simple event wait/notify mechanism.

`eventfd` would be a more suitable solution for this kind of scenario,
kernel also advocates for developers to use `eventfd` instead of `pipe`
in some simple use cases:

    Applications can use an eventfd file descriptor instead of a pipe
    (see pipe(2) in all cases where a pipe is used simply to signal
    events. The kernel overhead of an eventfd file descriptor is much
    lower than that of a pipe, and only one file descriptor is required
    (versus the two required for a pipe).

This change adds the new backend of `eventfd` for `wakeup_create` and
uses it where available, eliminating the overhead of `pipe`. Also, it
optimizes the `wakeup_create` to eliminate the system calls that make
file descriptors non-blocking by moving the logic of setting
non-blocking flags on file descriptors to `socketpair.c` and using
`SOCK_NONBLOCK` for `socketpair(2)`, `EFD_NONBLOCK` for `eventfd(2)`.

Ref:
https://man7.org/linux/man-pages/man7/pipe.7.html
https://man7.org/linux/man-pages/man2/eventfd.2.html
https://man7.org/linux/man-pages/man2/socketpair.2.html
https://www.gnu.org/software/gnulib/manual/html_node/eventfd.html

Closes #13874
CMake/Platforms/WindowsCache.cmake
CMakeLists.txt
configure.ac
lib/asyn-thread.c
lib/curl_config.h.cmake
lib/multi.c
lib/multihandle.h
lib/socketpair.c
lib/socketpair.h
lib/vquic/curl_msh3.c