Skip to content

Fix issue with (and clarify) linked timeouts

Jens Axboe io-uring 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2io_uring: cleanup return values from the queueing functionsfs/io_uring.c+12 -160 replies
2/2io_uring: fix sequencing issues with linked timeoutsfs/io_uring.c+33 -180 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCHSET 0/2] Fix issue with (and clarify) linked timeouts

First patch is just a prep patch that cleans up the return values from the various queueing functions, and the second one fixes sequencing issues with linked timeouts, as well as makes it so that we only support one linked timeout in a chain (and only as the 2nd request). We can relax this restriction later, if it makes sense, for now I think we should just play it safe. This doesn't mean that you can only submit a request and timeout in a submission, just that requests that are linked with IOSQE_LINK must have a parent request and then a timeout tied to that. That's the intended use case anyway, a timeout is paired with one other parent request.

 fs/io_uring.c | 79 +++++++++++++++++++++++++++++----------------------
 1 file changed, 45 insertions(+), 34 deletions(-)
Signature · 1 line
Jens Axboe