Skip to content

Conversation

@bsbernd
Copy link
Collaborator

@bsbernd bsbernd commented Nov 10, 2025

With the reduced queue feature io-uring is marked as ready after receiving the 1st ring entry. At this time other queues just might be in the process of registration and then a race happens

fuse_uring_queue_fuse_req -> no queue entry registered yet
list_add_tail -> fuse request gets queues

So far fetching requests from the list only happened from FUSE_IO_URING_CMD_COMMIT_AND_FETCH, but without new requests on the same queue, it would actually never send requests from that queue - the request was stuck.

@bsbernd bsbernd requested review from hbirth and yongzech November 10, 2025 14:59
Copy link
Collaborator

@hbirth hbirth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks splendid to me

With the reduced queue feature io-uring is marked as ready after
receiving the 1st ring entry. At this time other queues just
might be in the process of registration and then a race happens

fuse_uring_queue_fuse_req -> no queue entry registered yet
    list_add_tail -> fuse request gets queued

So far fetching requests from the list only happened from
FUSE_IO_URING_CMD_COMMIT_AND_FETCH, but without new requests
on the same queue, it would actually never send requests
from that queue - the request was stuck.
@bsbernd bsbernd force-pushed the ubuntu-noble-6.8.0-58.60-startup-race branch from e059e06 to 4cbee2e Compare November 10, 2025 15:46
@bsbernd bsbernd merged commit 3bfb6cd into redfs-ubuntu-noble-6.8.0-58.60 Nov 10, 2025
@bsbernd bsbernd deleted the ubuntu-noble-6.8.0-58.60-startup-race branch November 12, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants