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.

(cherry picked from commit e059e06)

@bsbernd bsbernd requested a review from hbirth November 10, 2025 15:12
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.

Never been happier to approve a change

@bsbernd bsbernd force-pushed the startup-race-redfs-rhel9_6-570.12.1 branch from 84e20c2 to b7db16a Compare November 10, 2025 15:44
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.

(cherry picked from commit 4cbee2e)
@bsbernd bsbernd force-pushed the startup-race-redfs-rhel9_6-570.12.1 branch from b7db16a to c958b50 Compare November 10, 2025 15:47
@bsbernd bsbernd merged commit d344f76 into redfs-rhel9_6-570.12.1 Nov 10, 2025
0 of 2 checks passed
@bsbernd bsbernd deleted the startup-race-redfs-rhel9_6-570.12.1 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