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
F174389990: D36270.id109975.diff
Fri, Oct 2, 9:01 PM
F174351845: D36270.id109967.diff
Fri, Oct 2, 3:16 PM
F174268667: D36270.diff
Thu, Oct 1, 9:27 PM
Unknown Object (File)
Thu, Oct 1, 1:57 PM
Unknown Object (File)
Wed, Sep 30, 7:24 PM
Unknown Object (File)
Sun, Sep 27, 5:00 PM
Unknown Object (File)
Sat, Sep 26, 10:41 AM
Unknown Object (File)
Sat, Sep 12, 2:19 PM
View All 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 Not Applicable
Unit
Tests Not Applicable

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 109975

View Options

usr.sbin/bhyve/pci_virtio_scsi.c

Loading...