diff options
author | Jens Axboe <axboe@kernel.dk> | 2022-04-10 19:05:09 -0600 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2022-04-11 17:06:20 -0600 |
commit | 82733d168cbd3fe9dab603f05894316b99008924 (patch) | |
tree | eb34dfacd2df84b3683b2cc7bd66c0f2d231a57b /fs/io-wq.h | |
parent | 2804ecd8d3e3730b4f999cc1ff4b2441e1f4d513 (diff) | |
download | linux-stable-82733d168cbd3fe9dab603f05894316b99008924.tar.gz linux-stable-82733d168cbd3fe9dab603f05894316b99008924.tar.bz2 linux-stable-82733d168cbd3fe9dab603f05894316b99008924.zip |
io_uring: stop using io_wq_work as an fd placeholder
There are two reasons why this isn't the best idea:
- It's an odd area to grab a bit of storage space, hence it's an odd area
to grab storage from.
- It puts the 3rd io_kiocb cacheline into the hot path, where normal hot
path just needs the first two.
Use 'cflags' for joint fd/cflags storage. We only need fd until we
successfully issue, and we only need cflags once a request is done and is
completed.
Fixes: 6bf9c47a3989 ("io_uring: defer file assignment")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/io-wq.h')
-rw-r--r-- | fs/io-wq.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/io-wq.h b/fs/io-wq.h index 04d374e65e54..dbecd27656c7 100644 --- a/fs/io-wq.h +++ b/fs/io-wq.h @@ -155,7 +155,6 @@ struct io_wq_work_node *wq_stack_extract(struct io_wq_work_node *stack) struct io_wq_work { struct io_wq_work_node list; unsigned flags; - int fd; }; static inline struct io_wq_work *wq_next_work(struct io_wq_work *work) |