diff options
author | Markus Armbruster <armbru@redhat.com> | 2023-09-28 15:19:33 +0200 |
---|---|---|
committer | Juan Quintela <quintela@redhat.com> | 2023-10-11 11:17:03 +0200 |
commit | 8ff58b05a3f6294731b9defb1d821e2f8703c8f1 (patch) | |
tree | 179a5d5a059973a3a4872d56cf1a9a600819ce01 /gdbstub/trace-events | |
parent | 25352b371b37b038d50b75ec0a3557e77f1d74e3 (diff) | |
download | qemu-8ff58b05a3f6294731b9defb1d821e2f8703c8f1.zip qemu-8ff58b05a3f6294731b9defb1d821e2f8703c8f1.tar.gz qemu-8ff58b05a3f6294731b9defb1d821e2f8703c8f1.tar.bz2 |
migration/rdma: Clean up two more harmless signed vs. unsigned issues
qemu_rdma_exchange_get_response() compares int parameter @expecting
with uint32_t head->type. Actual arguments are non-negative
enumeration constants, RDMAControlHeader uint32_t member type, or
qemu_rdma_exchange_recv() int parameter expecting. Actual arguments
for the latter are non-negative enumeration constants. Change both
parameters to uint32_t.
In qio_channel_rdma_readv(), loop control variable @i is ssize_t, and
counts from 0 up to @niov, which is size_t. Change @i to size_t.
While there, make qio_channel_rdma_readv() and
qio_channel_rdma_writev() more consistent: change the former's @done
to ssize_t, and delete the latter's useless initialization of @len.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Fabiano Rosas <farosas@suse.de>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Message-ID: <20230928132019.2544702-8-armbru@redhat.com>
Diffstat (limited to 'gdbstub/trace-events')
0 files changed, 0 insertions, 0 deletions