Differential D36664

vtblk: Make vtblk_request_execute return void.
ClosedPublic
Actions

Authored by cperciva on Sep 22 2022, 7:00 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jan 12, 10:54 AM

	Unknown Object (File)
	Sat, Jan 10, 3:22 PM

	Unknown Object (File)
	Dec 11 2025, 6:32 AM

	Unknown Object (File)
	Dec 6 2025, 1:35 PM

	Unknown Object (File)
	Dec 4 2025, 2:19 AM

	Unknown Object (File)
	Nov 21 2025, 9:40 AM

	Unknown Object (File)
	Nov 13 2025, 7:47 AM

	Unknown Object (File)
	Nov 5 2025, 4:28 AM

Subscribers

Details

Reviewers

bryanv
imp

Commits

rG86f8f5ccb798: vtblk: Make vtblk_request_execute return void.

Summary

The error, if any, now gets stashed in the request structure. (Step 1
of reworking this driver to use busdma.)

No functional change intended.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 47485
Build 44372: arc lint + arc unit

Event Timeline

cperciva created this revision.Sep 22 2022, 7:00 AM

Herald added a subscriber: imp. · View Herald TranscriptSep 22 2022, 7:00 AM

cperciva requested review of this revision.Sep 22 2022, 7:00 AM

Harbormaster completed remote builds in B47485: Diff 110838.Sep 22 2022, 7:00 AM

cperciva added a child revision: D36665: vtblk: Requeue inside vtblk_request_execute.Sep 22 2022, 7:00 AM

cperciva mentioned this in D36667: vtblk: Use busdma.Sep 22 2022, 7:03 AM

bryanv accepted this revision.Oct 3 2022, 2:15 AM

This revision is now accepted and ready to land.Oct 3 2022, 2:15 AM

This is basically what CAM does and most of the SIMs inside of CAM: requests are queued and eventually they get tagged with an error when they complete.
It's also what the non-CAM drivers do when they have a work queue that needs to deal with resource shortages and limited queue depth.

Closed by commit rG86f8f5ccb798: vtblk: Make vtblk_request_execute return void. (authored by cperciva). · Explain WhyOct 18 2022, 6:03 AM

This revision was automatically updated to reflect the committed changes.

cperciva added a commit: rG86f8f5ccb798: vtblk: Make vtblk_request_execute return void..

Revision Contents
Changeset List

Path

Size

sys/

dev/

virtio/

block/

32 lines

Diff 110838

sys/dev/virtio/block/virtio_blk.c

Loading...