Page MenuHomeFreeBSD
Differential D53223

bhyve/virtio-scsi: Support multiple backends
AcceptedPublic
Actions

Authored by rosenfeld_grumpf.hope-2000.org on Oct 20 2025, 5:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 24, 11:25 AM
Unknown Object (File)
Mon, Feb 23, 10:01 PM
Unknown Object (File)
Mon, Feb 23, 12:29 PM
Unknown Object (File)
Mon, Feb 23, 7:32 AM
Unknown Object (File)
Sun, Feb 22, 5:01 PM
Unknown Object (File)
Sun, Feb 22, 5:01 PM
Unknown Object (File)
Wed, Feb 18, 2:42 PM
Unknown Object (File)
Tue, Feb 17, 1:14 AM
View All 46 Files
Subscribers
bcran
imp
rgrimes
ziaee

Details

Reviewers
jhb
corvink
markj
Group Reviewers
bhyve
Summary

In order to support multiple backends for virtio-scsi, we should isolate
the core of virtio-scsi from the backend-specific code. The existing
interface to CTL will become the new "CTL" backend for virtio-scsi.

Care has been taken to keep compatibility with previous configurations:
The first backend linked (CTL) will be the default backend if none is
specified, and it does provide a default configuration if no further
options such as targets are explicitly configured.

This is currently also under review in illumos: https://code.illumos.org/c/illumos-gate/+/4425

If you're curious about what another backend does look like, here's the review for the illumos USCSI(4i) backend for SCSI passthrough: https://code.illumos.org/c/illumos-gate/+/4426

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 68271
Build 65154: arc lint + arc unit

Event Timeline

rosenfeld_grumpf.hope-2000.org created this revision.Oct 20 2025, 5:46 PM
Herald added a reviewer: bhyve. · View Herald TranscriptOct 20 2025, 5:46 PM
Herald added subscribers: ziaee, bcran, rgrimes, imp. · View Herald Transcript
rosenfeld_grumpf.hope-2000.org requested review of this revision.Oct 20 2025, 5:46 PM
Harbormaster completed remote builds in B67944: Diff 164636.Oct 20 2025, 5:46 PM
rosenfeld_grumpf.hope-2000.org added a parent revision: D53222: bhyve/virtio-scsi: Implement task management functions.Oct 20 2025, 5:46 PM
rosenfeld_grumpf.hope-2000.org edited the summary of this revision. (Show Details)Oct 20 2025, 7:35 PM
rosenfeld_grumpf.hope-2000.org mentioned this in D53220: bhyve/virtio-scsi: various small improvements.Oct 21 2025, 9:38 AM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 165407.Oct 30 2025, 7:57 AM

Rebase.

Harbormaster completed remote builds in B68271: Diff 165407.Oct 30 2025, 7:57 AM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 165415.Oct 30 2025, 8:28 AM

Rebase.

Harbormaster completed remote builds in B68277: Diff 165415.Oct 30 2025, 8:28 AM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 167546.Dec 4 2025, 6:34 PM

Minor updates.

Harbormaster completed remote builds in B69051: Diff 167546.Dec 4 2025, 6:34 PM
rosenfeld_grumpf.hope-2000.org retitled this revision from bhyve/virtio-scsi: support multiple backends to bhyve/virtio-scsi: Support multiple backends.Thu, Feb 19, 7:36 PM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 172278.Thu, Feb 19, 7:37 PM

Rebase.

Harbormaster completed remote builds in B70798: Diff 172278.Thu, Feb 19, 7:37 PM
markj accepted this revision as: markj.Sun, Feb 22, 5:24 PM
markj added inline comments.
usr.sbin/bhyve/pci_virtio_scsi_ctl.c
173

Same style issue below.

This revision is now accepted and ready to land.Sun, Feb 22, 5:24 PM

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
1 line
44 lines
26 lines
358 lines
546 lines
370 lines

Diff 165407

View Options

usr.sbin/bhyve/Makefile

Loading...
View Options

usr.sbin/bhyve/bhyve.8

Loading...
View Options

usr.sbin/bhyve/bhyve_config.5

Loading...
View Options

usr.sbin/bhyve/pci_virtio_scsi.h

Loading...
View Options

usr.sbin/bhyve/pci_virtio_scsi.c

Loading...
View Options

usr.sbin/bhyve/pci_virtio_scsi_ctl.c

Loading...