Skip to content

Fix handling of read_nonblock.#143

Merged
ioquatix merged 2 commits into
mainfrom
io_uring-non-blocking-read
Jul 2, 2025
Merged

Fix handling of read_nonblock.#143
ioquatix merged 2 commits into
mainfrom
io_uring-non-blocking-read

Conversation

@ioquatix
Copy link
Copy Markdown
Member

@ioquatix ioquatix commented Jul 2, 2025

io_uring blocks even when the length is zero and the IO is set to non-blocking mode. This caused unicorn in async mode to hang, because it depends on this behaviour. It worked fine with EPoll but not URing.

See socketry/async-examples#15 for more context.

Types of Changes

  • Bug fix.

Contribution

@ioquatix ioquatix force-pushed the io_uring-non-blocking-read branch 2 times, most recently from e06a080 to 2a0fde4 Compare July 2, 2025 21:58
@ioquatix ioquatix force-pushed the io_uring-non-blocking-read branch from 2a0fde4 to 237810b Compare July 2, 2025 22:00
@ioquatix ioquatix merged commit 1edf751 into main Jul 2, 2025
86 of 96 checks passed
@ioquatix ioquatix deleted the io_uring-non-blocking-read branch July 2, 2025 22:44
@ioquatix ioquatix self-assigned this Jul 2, 2025
@ioquatix ioquatix added this to the v1.11.1 milestone Jul 2, 2025
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.

1 participant