| 
    
 [Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [UNIKRAFT PATCH RFCv3 29/32] plat/virtio_9p: Remove excessive check for max virtqueue size
 For qemu virtio mmio read with VIRTIO_MMIO_QUEUE_NUM_MAX, it will
unconditionally return VIRTQUEUE_MAX_SIZE(1024) unlike virtio pci.
But it can set the number of queue size when seting up the virtqueue.
See vm_setup_vq which will be fixed in coming patch.
So the excessive check in virtio 9p is not necessary.
Signed-off-by: Jia He <justin.he@xxxxxxx>
---
 plat/drivers/virtio/virtio_9p.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/plat/drivers/virtio/virtio_9p.c b/plat/drivers/virtio/virtio_9p.c
index fe0d69a..7c138f9 100644
--- a/plat/drivers/virtio/virtio_9p.c
+++ b/plat/drivers/virtio/virtio_9p.c
@@ -307,17 +307,15 @@ static int virtio_9p_vq_alloc(struct virtio_9p_device *d)
 
        d->hwvq_id = 0;
        if (unlikely(qdesc_size != NUM_SEGMENTS)) {
-               uk_pr_err(DRIVER_NAME": Expected %d descriptors, found %d 
(virtqueue %"
+               uk_pr_info(DRIVER_NAME": Expected %d descriptors, found %d 
(virtqueue %"
                          PRIu16")\n", NUM_SEGMENTS, qdesc_size, d->hwvq_id);
-               rc = -EINVAL;
-               goto exit;
        }
 
        uk_sglist_init(&d->sg, ARRAY_SIZE(d->sgsegs), &d->sgsegs[0]);
 
        d->vq = virtio_vqueue_setup(d->vdev,
                                    d->hwvq_id,
-                                   qdesc_size,
+                                   NUM_SEGMENTS,
                                    virtio_9p_recv,
                                    a);
        if (unlikely(PTRISERR(d->vq))) {
-- 
2.17.1
 
  | 
  
![]()  | 
            
         Lists.xenproject.org is hosted with RackSpace, monitoring our  |