diff options
author | Gonglei <arei.gonglei@huawei.com> | 2016-11-22 00:51:50 -0500 |
---|---|---|
committer | Michael S. Tsirkin <mst@redhat.com> | 2016-12-14 23:39:47 -0500 |
commit | c60923cb9cb5e042790839d553ed77e68ef45adf (patch) | |
tree | 0f22f5d2b7cb69bf083feb1a96950b29bd84eda3 | |
parent | 61bd405f4edcf7396cc7853e48212342feead06d (diff) |
virtio_ring: fix complaint by sparse
# make C=2 CF="-D__CHECK_ENDIAN__" ./drivers/virtio/
drivers/virtio/virtio_ring.c:423:19: warning: incorrect type in assignment (different base types)
drivers/virtio/virtio_ring.c:423:19: expected unsigned int [unsigned] [assigned] i
drivers/virtio/virtio_ring.c:423:19: got restricted __virtio16 [usertype] next
drivers/virtio/virtio_ring.c:423:19: warning: incorrect type in assignment (different base types)
drivers/virtio/virtio_ring.c:423:19: expected unsigned int [unsigned] [assigned] i
drivers/virtio/virtio_ring.c:423:19: got restricted __virtio16 [usertype] next
drivers/virtio/virtio_ring.c:423:19: warning: incorrect type in assignment (different base types)
drivers/virtio/virtio_ring.c:423:19: expected unsigned int [unsigned] [assigned] i
drivers/virtio/virtio_ring.c:423:19: got restricted __virtio16 [usertype] next
drivers/virtio/virtio_ring.c:604:39: warning: incorrect type in initializer (different base types)
drivers/virtio/virtio_ring.c:604:39: expected unsigned short [unsigned] [usertype] nextflag
drivers/virtio/virtio_ring.c:604:39: got restricted __virtio16
drivers/virtio/virtio_ring.c:612:33: warning: restricted __virtio16 degrades to integer
Signed-off-by: Gonglei <arei.gonglei@huawei.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
-rw-r--r-- | drivers/virtio/virtio_ring.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c index 489bfc61cf30..d2863c321e93 100644 --- a/drivers/virtio/virtio_ring.c +++ b/drivers/virtio/virtio_ring.c | |||
@@ -420,7 +420,7 @@ unmap_release: | |||
420 | if (i == err_idx) | 420 | if (i == err_idx) |
421 | break; | 421 | break; |
422 | vring_unmap_one(vq, &desc[i]); | 422 | vring_unmap_one(vq, &desc[i]); |
423 | i = vq->vring.desc[i].next; | 423 | i = virtio16_to_cpu(_vq->vdev, vq->vring.desc[i].next); |
424 | } | 424 | } |
425 | 425 | ||
426 | vq->vq.num_free += total_sg; | 426 | vq->vq.num_free += total_sg; |
@@ -601,7 +601,7 @@ EXPORT_SYMBOL_GPL(virtqueue_kick); | |||
601 | static void detach_buf(struct vring_virtqueue *vq, unsigned int head) | 601 | static void detach_buf(struct vring_virtqueue *vq, unsigned int head) |
602 | { | 602 | { |
603 | unsigned int i, j; | 603 | unsigned int i, j; |
604 | u16 nextflag = cpu_to_virtio16(vq->vq.vdev, VRING_DESC_F_NEXT); | 604 | __virtio16 nextflag = cpu_to_virtio16(vq->vq.vdev, VRING_DESC_F_NEXT); |
605 | 605 | ||
606 | /* Clear data ptr. */ | 606 | /* Clear data ptr. */ |
607 | vq->desc_state[head].data = NULL; | 607 | vq->desc_state[head].data = NULL; |