Skip to content

Conversation

@xemul
Copy link
Contributor

@xemul xemul commented Sep 11, 2025

The AIO requests may come via two places -- the reactor itself polling
for IO and thread-pool that retries requests that had been rejected by
the kernel on the first try.

This patch counts the latter events and exports them as metrics. The
former can be calculated using existing aio-reads and aio-writes.

@xemul xemul requested a review from avikivity September 11, 2025 10:14
The AIO requests may come via two places -- the reactor itself polling
for IO and thread-pool that retries requests that had been rejected by
the kernel on the first try.

This patch counts the latter events and exports them as metrics. The
former can be calculated using existing aio-reads and aio-writes.

Signed-off-by: Pavel Emelyanov <[email protected]>
@xemul xemul force-pushed the br-reactor-io-submit-resubmit-metrics branch from 66f7e18 to 8bf7ba2 Compare September 15, 2025 16:42
@xemul
Copy link
Contributor Author

xemul commented Sep 15, 2025

upd:

  • count only retries and export them under "aio_retries" name

@xemul xemul changed the title io: Count and export number of io_submit()-ed IOCBs io: Count and export number of AIO retries Sep 15, 2025
@xemul xemul requested a review from avikivity September 15, 2025 16:43
@xemul
Copy link
Contributor Author

xemul commented Sep 22, 2025

@avikivity , please re-review

@xemul
Copy link
Contributor Author

xemul commented Sep 29, 2025

@avikivity , please re-review

@travisdowns
Copy link
Contributor

This patch would be very nice!

@xemul
Copy link
Contributor Author

xemul commented Oct 6, 2025

@avikivity , please re-review

@avikivity avikivity merged commit f664b5c into scylladb:master Oct 13, 2025
16 checks passed
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