romange / helio

A modern framework for backend development based on io_uring Linux interface
Apache License 2.0
435 stars 49 forks source link

feat: add support for IORING_CQE_F_SOCK_NONEMPTY flag during socket recv #192

Closed romange closed 9 months ago

romange commented 9 months ago

This applies only for recent version of iouring. In case the receive buffer was empty (i.e. we have not got IORING_CQE_F_SOCK_NONEMPTY flag) we can save the internal call to recv inside kernel by flagging the receive operation with IORING_RECVSEND_POLL_FIRST.

codecov-commenter commented 9 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (321d4d6) 79.12% compared to head (d9e5fbc) 79.24%.

Files Patch % Lines
util/fibers/fiber_socket_test.cc 95.83% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #192 +/- ## ========================================== + Coverage 79.12% 79.24% +0.12% ========================================== Files 100 100 Lines 7462 7496 +34 ========================================== + Hits 5904 5940 +36 + Misses 1558 1556 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.