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

	F168699540: D36664.id.diff
	Sat, Aug 29, 5:05 PM

	Unknown Object (File)
	Tue, Aug 18, 6:26 PM

	Unknown Object (File)
	Wed, Aug 12, 5:21 PM

	Unknown Object (File)
	Tue, Aug 11, 5:45 AM

	Unknown Object (File)
	Sun, Aug 9, 10:06 PM

	Unknown Object (File)
	Sat, Aug 8, 4:45 PM

	Unknown Object (File)
	Mon, Aug 3, 7:45 AM

	Unknown Object (File)
	Sun, Aug 2, 4:43 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...