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>
---
 fs/fuse/virtio_fs.c | 22 +++++++++-------------
 1 file changed, 9 insertions(+), 13 deletions(-)

diff --git a/fs/fuse/virtio_fs.c b/fs/fuse/virtio_fs.c
index ca7b64f9c3c7..466d4888d664 100644
--- a/fs/fuse/virtio_fs.c
+++ b/fs/fuse/virtio_fs.c
@@ -740,9 +740,8 @@ static void virtio_fs_init_vq(struct virtio_fs_vq *fsvq, 
char *name,
 static int virtio_fs_setup_vqs(struct virtio_device *vdev,
                               struct virtio_fs *fs)
 {
+       struct virtio_queue_info *vqs_info;
        struct virtqueue **vqs;
-       vq_callback_t **callbacks;
-       const char **names;
        unsigned int i;
        int ret = 0;
 
@@ -757,18 +756,16 @@ static int virtio_fs_setup_vqs(struct virtio_device *vdev,
                return -ENOMEM;
 
        vqs = kmalloc_array(fs->nvqs, sizeof(vqs[VQ_HIPRIO]), GFP_KERNEL);
-       callbacks = kmalloc_array(fs->nvqs, sizeof(callbacks[VQ_HIPRIO]),
-                                       GFP_KERNEL);
-       names = kmalloc_array(fs->nvqs, sizeof(names[VQ_HIPRIO]), GFP_KERNEL);
-       if (!vqs || !callbacks || !names) {
+       vqs_info = kcalloc(fs->nvqs, sizeof(*vqs_info), GFP_KERNEL);
+       if (!vqs || !vqs_info) {
                ret = -ENOMEM;
                goto out;
        }
 
        /* Initialize the hiprio/forget request virtqueue */
-       callbacks[VQ_HIPRIO] = virtio_fs_vq_done;
+       vqs_info[VQ_HIPRIO].callback = virtio_fs_vq_done;
        virtio_fs_init_vq(&fs->vqs[VQ_HIPRIO], "hiprio", VQ_HIPRIO);
-       names[VQ_HIPRIO] = fs->vqs[VQ_HIPRIO].name;
+       vqs_info[VQ_HIPRIO].name = fs->vqs[VQ_HIPRIO].name;
 
        /* Initialize the requests virtqueues */
        for (i = VQ_REQUEST; i < fs->nvqs; i++) {
@@ -776,11 +773,11 @@ static int virtio_fs_setup_vqs(struct virtio_device *vdev,
 
                snprintf(vq_name, VQ_NAME_LEN, "requests.%u", i - VQ_REQUEST);
                virtio_fs_init_vq(&fs->vqs[i], vq_name, VQ_REQUEST);
-               callbacks[i] = virtio_fs_vq_done;
-               names[i] = fs->vqs[i].name;
+               vqs_info[i].callback = virtio_fs_vq_done;
+               vqs_info[i].name = fs->vqs[i].name;
        }
 
-       ret = virtio_find_vqs(vdev, fs->nvqs, vqs, callbacks, names, NULL);
+       ret = virtio_find_vqs_info(vdev, fs->nvqs, vqs, vqs_info, NULL);
        if (ret < 0)
                goto out;
 
@@ -789,8 +786,7 @@ static int virtio_fs_setup_vqs(struct virtio_device *vdev,
 
        virtio_fs_start_all_queues(fs);
 out:
-       kfree(names);
-       kfree(callbacks);
+       kfree(vqs_info);
        kfree(vqs);
        if (ret)
                kfree(fs->vqs);
-- 
2.45.2


Reply via email to