Skip to content

v3.0: Retry IO on short write in io_uring file creator (backport of #8053)#8173

Merged
kskalski merged 1 commit into
v3.0from
mergify/bp/v3.0/pr-8053
Sep 26, 2025
Merged

v3.0: Retry IO on short write in io_uring file creator (backport of #8053)#8173
kskalski merged 1 commit into
v3.0from
mergify/bp/v3.0/pr-8053

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Sep 24, 2025

Problem

As reported in #8036 (comment) ubuntu 22 with 5.15 kernel and ZFS filesystem experiences error due to short writes.
It's not completely clear which kernel / FS combination give 100% guarantee of not returning short write, this comment axboe/liburing#766 (comment) suggests 5.15 fixed some issues, but given reports from the wild it's worth fixing.

Summary of Changes

Re-submit write when getting short write (unless Ok with 0-size write happens, which is treated as hard error)

Fixes #8036


This is an automatic backport of pull request #8053 done by [Mergify](https://mergify.com).

* Support short writes in io_uring file creator

* Handle resource busy err as short write. Update comment. Fix calculating total written for stats

* Remove handling of busy error

* Add warn for short write

(cherry picked from commit a10a2c8)
@mergify mergify Bot requested a review from a team as a code owner September 24, 2025 14:03
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 56.00000% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.4%. Comparing base (9c181a4) to head (83658d5).

Additional details and impacted files
@@           Coverage Diff           @@
##             v3.0    #8173   +/-   ##
=======================================
  Coverage    83.4%    83.4%           
=======================================
  Files         810      810           
  Lines      365773   365792   +19     
=======================================
+ Hits       305395   305435   +40     
+ Misses      60378    60357   -21     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread accounts-db/src/io_uring/file_creator.rs
@kskalski kskalski merged commit 5310b35 into v3.0 Sep 26, 2025
42 checks passed
@kskalski kskalski deleted the mergify/bp/v3.0/pr-8053 branch September 26, 2025 01:32
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.

4 participants