Page MenuHomeFreeBSD

vtblk: Requeue inside vtblk_request_execute
ClosedPublic

Authored by cperciva on Sep 22 2022, 7:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 10, 12:27 AM
Unknown Object (File)
Tue, Nov 19, 6:07 AM
Unknown Object (File)
Tue, Nov 19, 4:54 AM
Unknown Object (File)
Nov 16 2024, 4:42 PM
Unknown Object (File)
Nov 16 2024, 6:18 AM
Unknown Object (File)
Oct 19 2024, 5:38 AM
Unknown Object (File)
Oct 12 2024, 2:21 AM
Unknown Object (File)
Sep 30 2024, 1:42 PM
Subscribers

Details

Summary

Most virtio_blk requests are launched from vtblk_startio; prior to this
commit, if vtblk_request_execute failed (e.g. due to a lack of space on
the virtio queue) vtblk_startio would requeue the request to be
reattempted later.

Add a flag "vbr_requeue_on_error" to requests and perform the requeuing
from inside vtblk_request_execute instead.

No functional change intended.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 47486
Build 44373: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Oct 3 2022, 3:19 AM
imp added inline comments.
sys/dev/virtio/block/virtio_blk.c
1139

One can quibble over whether we should transition to bool for these, but the driver already has a few in the old-style.

This revision was automatically updated to reflect the committed changes.