OSDN Git Service

vdpa/mlx5: Fix umem sizes assignments on VQ create
authorEli Cohen <elic@nvidia.com>
Sun, 30 May 2021 09:03:17 +0000 (12:03 +0300)
committerMichael S. Tsirkin <mst@redhat.com>
Sat, 3 Jul 2021 08:50:56 +0000 (04:50 -0400)
Fix copy paste bug assigning umem1 size to umem2 and umem3. The issue
was discovered when trying to use a 1:1 MR that covers the entire
address space where firmware complained that provided sizes are not
large enough. 1:1 MRs are required to support virtio_vdpa.

Fixes: 1a86b377aa21 ("vdpa/mlx5: Add VDPA driver for supported mlx5 devices")
Signed-off-by: Eli Cohen <elic@nvidia.com>
Link: https://lore.kernel.org/r/20210530090317.8284-1-elic@nvidia.com
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Jason Wang <jasowang@redhat.com>
drivers/vdpa/mlx5/net/mlx5_vnet.c

index dda5dc6..e8865e6 100644 (file)
@@ -829,9 +829,9 @@ static int create_virtqueue(struct mlx5_vdpa_net *ndev, struct mlx5_vdpa_virtque
        MLX5_SET(virtio_q, vq_ctx, umem_1_id, mvq->umem1.id);
        MLX5_SET(virtio_q, vq_ctx, umem_1_size, mvq->umem1.size);
        MLX5_SET(virtio_q, vq_ctx, umem_2_id, mvq->umem2.id);
-       MLX5_SET(virtio_q, vq_ctx, umem_2_size, mvq->umem1.size);
+       MLX5_SET(virtio_q, vq_ctx, umem_2_size, mvq->umem2.size);
        MLX5_SET(virtio_q, vq_ctx, umem_3_id, mvq->umem3.id);
-       MLX5_SET(virtio_q, vq_ctx, umem_3_size, mvq->umem1.size);
+       MLX5_SET(virtio_q, vq_ctx, umem_3_size, mvq->umem3.size);
        MLX5_SET(virtio_q, vq_ctx, pd, ndev->mvdev.res.pdn);
        if (MLX5_CAP_DEV_VDPA_EMULATION(ndev->mvdev.mdev, eth_frame_offload_type))
                MLX5_SET(virtio_q, vq_ctx, virtio_version_1_0, 1);