Page MenuHomeFreeBSD
Differential D54073

bhyve/virtio-scsi: Make all I/O processing parameters configurable
Needs ReviewPublic
Actions

Authored by rosenfeld_grumpf.hope-2000.org on Dec 4 2025, 6:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 17, 9:40 PM
Unknown Object (File)
Sun, May 17, 9:39 PM
Unknown Object (File)
Mon, May 11, 2:43 PM
Unknown Object (File)
Mon, May 11, 1:08 PM
Unknown Object (File)
Mon, May 11, 2:29 AM
Unknown Object (File)
Mon, May 11, 1:48 AM
Unknown Object (File)
Mon, May 11, 12:58 AM
Unknown Object (File)
Sun, May 10, 7:59 PM
View All 88 Files
Subscribers
bcran
imp
novel
rgrimes
ziaee

Details

Reviewers
jhb
markj
corvink
Group Reviewers
bhyve
Summary

This includes:

  • seg_max, the number of segments allowed in a single command
  • {ctl,evt,req}_ringsz, the number of descriptors in a queue
  • thr_per_q, the number of processing threads per request queue
  • num_queues, the number of request queues

Diff Detail

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

Event Timeline

rosenfeld_grumpf.hope-2000.org created this revision.Dec 4 2025, 6:51 PM
Herald added subscribers: ziaee, bcran, rgrimes, imp. · View Herald TranscriptDec 4 2025, 6:51 PM
rosenfeld_grumpf.hope-2000.org requested review of this revision.Dec 4 2025, 6:51 PM
Harbormaster completed remote builds in B69052: Diff 167548.Dec 4 2025, 6:51 PM
rosenfeld_grumpf.hope-2000.org retitled this revision from bhyve/virtio-scsi: make all I/O processing parameters configurable to bhyve/virtio-scsi: Make all I/O processing parameters configurable.Feb 19 2026, 7:37 PM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 172279.Feb 19 2026, 7:38 PM

Rebase.

Harbormaster completed remote builds in B70799: Diff 172279.Feb 19 2026, 7:38 PM
markj accepted this revision as: markj.Feb 22 2026, 5:30 PM
markj added inline comments.
usr.sbin/bhyve/bhyve.8
745
This revision is now accepted and ready to land.Feb 22 2026, 5:30 PM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 173629.Mar 12 2026, 8:34 PM
rosenfeld_grumpf.hope-2000.org marked an inline comment as done.

Review from markj.

This revision now requires review to proceed.Mar 12 2026, 8:34 PM
Harbormaster completed remote builds in B71372: Diff 173629.Mar 12 2026, 8:34 PM
markj accepted this revision as: markj.Mar 14 2026, 3:06 AM
This revision is now accepted and ready to land.Mar 14 2026, 3:06 AM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 173686.Mar 14 2026, 11:07 AM

Rebase.

This revision now requires review to proceed.Mar 14 2026, 11:07 AM
Harbormaster completed remote builds in B71402: Diff 173686.Mar 14 2026, 11:07 AM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 173749.Mar 16 2026, 8:38 AM

Garbage-collect a comment obsoleted by this change.

Harbormaster completed remote builds in B71426: Diff 173749.Mar 16 2026, 8:38 AM
markj accepted this revision as: markj.Mar 20 2026, 2:22 AM
This revision is now accepted and ready to land.Mar 20 2026, 2:22 AM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 175089.Apr 8 2026, 10:55 AM

Rebase.

This revision now requires review to proceed.Apr 8 2026, 10:55 AM
Harbormaster completed remote builds in B72044: Diff 175089.Apr 8 2026, 10:55 AM
rosenfeld_grumpf.hope-2000.org updated this revision to Diff 175972.Tue, Apr 21, 10:43 AM

Rebase.

Harbormaster completed remote builds in B72408: Diff 175972.Tue, Apr 21, 10:43 AM
novel added a subscriber: novel.Sat, May 16, 9:15 AM
markj added inline comments.Sun, May 17, 10:44 PM
usr.sbin/bhyve/pci_virtio_scsi.c
918

I think this stack allocation can be quite large. Is there some reason not to do a heap allocation here, given that this is the control queue?

1466

sc->vss_queues can be NULL here.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
12 lines
16 lines
80 lines
217 lines

Diff 175972

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...