[SLOF] [PATCH slof 06/13] virtio: Compile with -Wextra

Alexey Kardashevskiy aik at ozlabs.ru
Wed Jan 27 19:57:45 AEDT 2021


-Wextra enables a bunch of rather useful checks which this fixes.

Signed-off-by: Alexey Kardashevskiy <aik at ozlabs.ru>
---
 lib/libvirtio/p9.c     | 8 ++++----
 lib/libvirtio/virtio.c | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/lib/libvirtio/p9.c b/lib/libvirtio/p9.c
index 0e595303196a..e9ba22886c06 100644
--- a/lib/libvirtio/p9.c
+++ b/lib/libvirtio/p9.c
@@ -142,7 +142,7 @@ void reset_buffers(void)
 int p9_transaction(p9_connection_t *connection)
 {
 	int rc;
-	int tx_size = GET_SIZE;
+	uint32_t tx_size = GET_SIZE;
 	uint32_t rx_size = connection->message_size;
 
 	if (transact == NULL) {
@@ -248,7 +248,7 @@ int p9_version(p9_connection_t *connection)
 int p9_attach(p9_connection_t *connection)
 {
 	int rc;
-	int length = 19 + strlen(connection->uname) + strlen(connection->aname);
+	unsigned length = 19 + strlen(connection->uname) + strlen(connection->aname);
 
 	if (length > connection->message_size) {
 		return P9_MSG_TOO_LONG;
@@ -372,8 +372,8 @@ int p9_walk(p9_connection_t *connection, uint32_t fid, uint32_t new_fid,
 			s_tok = e_tok;
 			continue;
 		}
-		int tx_size = (e_tok - s_tok + 2 + GET_SIZE);
-		int rx_size = ((element_count + 1) * MSG_QID_SIZE
+		unsigned tx_size = (e_tok - s_tok + 2 + GET_SIZE);
+		unsigned rx_size = ((element_count + 1) * MSG_QID_SIZE
 				+ MSG_WALK_RX_HDR_SIZE);
 		if ((tx_size > connection->message_size)
 			|| (rx_size > connection->message_size)) {
diff --git a/lib/libvirtio/virtio.c b/lib/libvirtio/virtio.c
index 69ac4394e104..9dfc93cb2876 100644
--- a/lib/libvirtio/virtio.c
+++ b/lib/libvirtio/virtio.c
@@ -436,7 +436,7 @@ struct vqs *virtio_queue_init_vq(struct virtio_device *dev, unsigned int id)
 void virtio_queue_term_vq(struct virtio_device *dev, struct vqs *vq, unsigned int id)
 {
 	if (vq->desc_gpas) {
-		int i;
+		unsigned i;
 
 		for (i = 0; i < vq->size; ++i)
 			virtio_free_desc(vq, i, dev->features);
-- 
2.17.1



More information about the SLOF mailing list