[PATCH virtio v2 10/19] virtio_blk: convert to use virtio_find_vqs_info()

[Jiri Pirko]

From: Jiri Pirko <j...@nvidia.com>

Instead of passing separate names and callbacks arrays
to virtio_find_vqs(), allocate one of virtual_queue_info structs and
pass it to virtio_find_vqs_info().

Signed-off-by: Jiri Pirko <j...@nvidia.com>
---
v1->v2:
- s/virtio_queue_info/virtqueue_info/
---
 drivers/block/virtio_blk.c | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c
index 2351f411fa46..ca3446503e82 100644
--- a/drivers/block/virtio_blk.c
+++ b/drivers/block/virtio_blk.c
@@ -964,8 +964,7 @@ static int init_vq(struct virtio_blk *vblk)
 {
        int err;
        unsigned short i;
-       vq_callback_t **callbacks;
-       const char **names;
+       struct virtqueue_info *vqs_info;
        struct virtqueue **vqs;
        unsigned short num_vqs;
        unsigned short num_poll_vqs;
@@ -1002,28 +1001,26 @@ static int init_vq(struct virtio_blk *vblk)
        if (!vblk->vqs)
                return -ENOMEM;
 
-       names = kmalloc_array(num_vqs, sizeof(*names), GFP_KERNEL);
-       callbacks = kmalloc_array(num_vqs, sizeof(*callbacks), GFP_KERNEL);
+       vqs_info = kcalloc(num_vqs, sizeof(*vqs_info), GFP_KERNEL);
        vqs = kmalloc_array(num_vqs, sizeof(*vqs), GFP_KERNEL);
-       if (!names || !callbacks || !vqs) {
+       if (!vqs_info || !vqs) {
                err = -ENOMEM;
                goto out;
        }
 
        for (i = 0; i < num_vqs - num_poll_vqs; i++) {
-               callbacks[i] = virtblk_done;
+               vqs_info[i].callback = virtblk_done;
                snprintf(vblk->vqs[i].name, VQ_NAME_LEN, "req.%u", i);
-               names[i] = vblk->vqs[i].name;
+               vqs_info[i].name = vblk->vqs[i].name;
        }
 
        for (; i < num_vqs; i++) {
-               callbacks[i] = NULL;
                snprintf(vblk->vqs[i].name, VQ_NAME_LEN, "req_poll.%u", i);
-               names[i] = vblk->vqs[i].name;
+               vqs_info[i].name = vblk->vqs[i].name;
        }
 
        /* Discover virtqueues and write information to configuration.  */
-       err = virtio_find_vqs(vdev, num_vqs, vqs, callbacks, names, &desc);
+       err = virtio_find_vqs_info(vdev, num_vqs, vqs, vqs_info, &desc);
        if (err)
                goto out;
 
@@ -1035,8 +1032,7 @@ static int init_vq(struct virtio_blk *vblk)
 
 out:
        kfree(vqs);
-       kfree(callbacks);
-       kfree(names);
+       kfree(vqs_info);
        if (err)
                kfree(vblk->vqs);
        return err;
-- 
2.45.2