Page MenuHomeFreeBSD
Differential D36270

bhyve virtio-scsi: Tidy warning and debug prints.
ClosedPublic
Actions

Authored by jhb on Aug 19 2022, 11:42 PM.
Tags
None
Referenced Files
F125513540: D36270.id109967.diff
Fri, Aug 8, 2:08 PM
Unknown Object (File)
Sun, Jul 27, 5:00 PM
Unknown Object (File)
Fri, Jul 25, 12:43 PM
Unknown Object (File)
Fri, Jul 25, 7:32 AM
Unknown Object (File)
Wed, Jul 23, 9:57 PM
Unknown Object (File)
Sat, Jul 19, 7:39 PM
Unknown Object (File)
Jul 9 2025, 7:46 PM
Unknown Object (File)
Jul 9 2025, 6:46 PM
View All 88 Files
Subscribers
bcran
emaste
freqlabs
imp
rgrimes

Details

Reviewers
mav
emaste
Group Reviewers
bhyve
Commits
rGffbe54c2175c: bhyve virtio-scsi: Tidy warning and debug prints.
rG597a39a77132: bhyve virtio-scsi: Tidy warning and debug prints.
rG62806a7f31e3: bhyve virtio-scsi: Tidy warning and debug prints.
Summary

Use a consistent prefix ("virtio-scsi: ") similar to the e1000 device
model.

Sponsored by: The FreeBSD Foundation

Diff Detail

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

Event Timeline

jhb created this revision.Aug 19 2022, 11:42 PM
Herald added a reviewer: bhyve. · View Herald TranscriptAug 19 2022, 11:42 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript
jhb requested review of this revision.Aug 19 2022, 11:42 PM
Harbormaster completed remote builds in B47028: Diff 109577.Aug 19 2022, 11:42 PM
jhb added a child revision: D36271: bhyve virtio-scsi: Avoid out of bounds accesses to guest requests..Aug 19 2022, 11:42 PM
freqlabs added inline comments.Aug 20 2022, 1:09 PM
usr.sbin/bhyve/pci_virtio_scsi.c
529

Left "pci_virtio_scsi: " on this one

mav accepted this revision.Aug 20 2022, 8:38 PM

Looks good to me, considering Ryan's comment.

This revision is now accepted and ready to land.Aug 20 2022, 8:38 PM
jhb marked an inline comment as done.Aug 29 2022, 5:06 PM
jhb updated this revision to Diff 109967.Aug 29 2022, 5:13 PM
  • Fix a nit noticed by Ryan.
This revision now requires review to proceed.Aug 29 2022, 5:13 PM
Harbormaster completed remote builds in B47174: Diff 109967.Aug 29 2022, 5:13 PM
emaste accepted this revision.Aug 29 2022, 5:45 PM
This revision is now accepted and ready to land.Aug 29 2022, 5:45 PM
Closed by commit rG62806a7f31e3: bhyve virtio-scsi: Tidy warning and debug prints. (authored by jhb). · Explain WhyAug 29 2022, 10:39 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rG62806a7f31e3: bhyve virtio-scsi: Tidy warning and debug prints..
jhb added a commit: rG597a39a77132: bhyve virtio-scsi: Tidy warning and debug prints..Nov 11 2022, 1:24 AM
jhb added a commit: rGffbe54c2175c: bhyve virtio-scsi: Tidy warning and debug prints..

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
22 lines

Diff 109967

View Options

usr.sbin/bhyve/pci_virtio_scsi.c

Loading...