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
F174274224: D55795.id173504.diff
Thu, Oct 1, 10:35 PM
F174274220: D55795.id.diff
Thu, Oct 1, 10:35 PM
Unknown Object (File)
Wed, Sep 30, 4:26 PM
Unknown Object (File)
Tue, Sep 29, 3:55 PM
Unknown Object (File)
Sun, Sep 27, 6:07 AM
Unknown Object (File)
Sun, Sep 27, 3:00 AM
Unknown Object (File)
Wed, Sep 16, 6:19 AM
Unknown Object (File)
Tue, Sep 8, 6:24 PM

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.