virtio_scsi: pend CAM commands when insufficient descriptors
Needs RevisionPublic
Actions

Authored by adam_fenn.io on Aug 2 2021, 7:49 PM.

Tags

None

Referenced Files

	F137876279: D31385.diff
	Wed, Nov 26, 6:25 PM

	Unknown Object (File)
	Tue, Nov 25, 1:09 AM

	Unknown Object (File)
	Tue, Nov 18, 8:14 AM

	Unknown Object (File)
	Thu, Nov 13, 2:15 PM

	Unknown Object (File)
	Oct 28 2025, 8:36 AM

	Unknown Object (File)
	Oct 27 2025, 10:10 PM

	Unknown Object (File)
	Oct 19 2025, 12:41 PM

	Unknown Object (File)
	Oct 19 2025, 12:41 PM

Subscribers

Contributor Reviews (src)

Details

Reviewers

imp

Summary

Instead of CAM requeues on resource depletion, pend the cmds. Only
un-pend when there are enough virtio descriptors available.

Sponsored by: NetApp, Inc.
Sponsored by: Klara, Inc.
X-NetApp-PR: #60

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 40808
Build 37697: arc lint + arc unit

Event Timeline

adam_fenn.io created this revision.Aug 2 2021, 7:49 PM

adam_fenn.io held this revision as a draft.

Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptAug 2 2021, 7:49 PM

Harbormaster completed remote builds in B40808: Diff 93132.Aug 2 2021, 7:49 PM

adam_fenn.io published this revision for review.Aug 2 2021, 7:52 PM

SIMs generally are supposed to return the requests to CAM for requeueing like was done before.
Why do this? What's the benefit? Almost no other drivers do this.

This revision now requires changes to proceed.Aug 2 2021, 10:37 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

virtio/

scsi/

virtio_scsi.c

28 lines

virtio_scsivar.h

3 lines

Diff 93132

View Options

sys/dev/virtio/scsi/virtio_scsi.c