Page MenuHomeFreeBSD

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

Authored by siva on Tue, Mar 10, 12:51 PM.
Tags
None
Referenced Files
F151357000: D55795.id173504.diff
Tue, Apr 7, 8:55 PM
F151319880: D55795.id173504.diff
Tue, Apr 7, 2:06 PM
Unknown Object (File)
Sun, Apr 5, 1:18 PM
Unknown Object (File)
Sat, Apr 4, 1:00 PM
Unknown Object (File)
Mon, Mar 23, 11:50 PM
Unknown Object (File)
Fri, Mar 20, 4:37 PM
Unknown Object (File)
Fri, Mar 20, 5:37 AM
Unknown Object (File)
Fri, Mar 20, 5:20 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.Tue, Mar 10, 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.