diff options
| author | Pavel Begunkov <asml.silence@gmail.com> | 2024-07-26 15:24:30 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2024-07-26 08:31:59 -0600 |
| commit | 342b2e395d5f34c9f111a818556e617939f83a8c (patch) | |
| tree | 4e09d19146f6e741dd98faeb071514a376085ff4 /include | |
| parent | 0db4618e8fabfcc404af4dda23799bba726785a5 (diff) | |
| download | linux-stable-342b2e395d5f34c9f111a818556e617939f83a8c.tar.gz linux-stable-342b2e395d5f34c9f111a818556e617939f83a8c.tar.bz2 linux-stable-342b2e395d5f34c9f111a818556e617939f83a8c.zip | |
io_uring/napi: use ktime in busy polling
It's more natural to use ktime/ns instead of keeping around usec,
especially since we're comparing it against user provided timers,
so convert napi busy poll internal handling to ktime. It's also nicer
since the type (ktime_t vs unsigned long) now tells the unit of measure.
Keep everything as ktime, which we convert to/from micro seconds for
IORING_[UN]REGISTER_NAPI. The net/ busy polling works seems to work with
usec, however it's not real usec as shift by 10 is used to get it from
nsecs, see busy_loop_current_time(), so it's easy to get truncated nsec
back and we get back better precision.
Note, we can further improve it later by removing the truncation and
maybe convincing net/ to use ktime/ns instead.
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/95e7ec8d095069a3ed5d40a4bc6f8b586698bc7e.1722003776.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/io_uring_types.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/io_uring_types.h b/include/linux/io_uring_types.h index e62aa9f0629f..3315005df117 100644 --- a/include/linux/io_uring_types.h +++ b/include/linux/io_uring_types.h @@ -404,7 +404,7 @@ struct io_ring_ctx { spinlock_t napi_lock; /* napi_list lock */ /* napi busy poll default timeout */ - unsigned int napi_busy_poll_to; + ktime_t napi_busy_poll_dt; bool napi_prefer_busy_poll; bool napi_enabled; |