Page MenuHomeFreeBSD

bhyve/virtio-scsi: fix sign-compare warning on gcc15
AbandonedPublic

Authored by siva on Mar 10 2026, 12:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 8, 12:42 AM
Unknown Object (File)
Sun, Jul 5, 5:40 AM
Unknown Object (File)
Sat, Jul 4, 4:01 PM
Unknown Object (File)
Thu, Jul 2, 8:40 AM
Unknown Object (File)
Wed, Jul 1, 12:18 PM
Unknown Object (File)
Wed, Jul 1, 11:31 AM
Unknown Object (File)
Tue, Jun 30, 10:56 AM
Unknown Object (File)
Fri, Jun 26, 4:26 AM

Details

Summary

In file included from /usr/src/usr.sbin/bhyve/pci_emul.h:41,

from /usr/src/usr.sbin/bhyve/pci_virtio_scsi.c:64:

/usr/src/usr.sbin/bhyve/pci_virtio_scsi.c: In function 'pci_vtscsi_queue_request':
/usr/src/usr.sbin/bhyve/pci_virtio_scsi.c:671:40: error: comparison of integer expressions of different signedness: 'ssize_t' {aka 'long int'} and 'long unsigned int' [-Werror=sign-compare]

671 |             (void **)&req->vsr_cmd_rd) == VTSCSI_IN_HEADER_LEN(q->vsq_sc));

Fixes: 2a514d377b37a30356c5c27a07b4fb0fd6df4881

Diff Detail

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

Event Timeline

siva requested review of this revision.Mar 10 2026, 12:51 PM

Sigh, this is kind of terrible. Code that is required should never be invoked as part of assert(). Your change is correct, but I think style-wise it is better to declare bufsize at the start of the function instead of creating a nested block.

Thanks, abandoning this review in favour of D55800 and D55803 by the author then.