Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Eli Cohen <elic@nvidia.com>
To:     <mst@redhat.com>, <virtualization@lists.linux-foundation.org>,
        <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <lulu@redhat.com>
CC:     <elic@nvidia.com>
Subject: [PATCH] vdpa/mlx5: Restore the hardware used index after change map
Date:   Tue, 2 Feb 2021 16:29:01 +0200
Message-ID: <20210202142901.7131-1-elic@nvidia.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain
Precedence: bulk

When a change of memory map occurs, the hardware resources are destroyed
and then re-created again with the new memory map. In such case, we need
to restore the hardware available and used indices. The driver failed to
restore the used index which is added here.

Fixes 1a86b377aa21 ("vdpa/mlx5: Add VDPA driver for supported mlx5 devices"=
)
Signed-off-by: Eli Cohen <elic@nvidia.com>
---
This patch is being sent again a single patch the fixes hot memory
addtion to a qemy process.

 drivers/vdpa/mlx5/net/mlx5_vnet.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/vdpa/mlx5/net/mlx5_vnet.c b/drivers/vdpa/mlx5/net/mlx5=
_vnet.c
index 88dde3455bfd..839f57c64a6f 100644
--- a/drivers/vdpa/mlx5/net/mlx5_vnet.c
+++ b/drivers/vdpa/mlx5/net/mlx5_vnet.c
@@ -87,6 +87,7 @@ struct mlx5_vq_restore_info {
 	u64 device_addr;
 	u64 driver_addr;
 	u16 avail_index;
+	u16 used_index;
 	bool ready;
 	struct vdpa_callback cb;
 	bool restore;
@@ -121,6 +122,7 @@ struct mlx5_vdpa_virtqueue {
 	u32 virtq_id;
 	struct mlx5_vdpa_net *ndev;
 	u16 avail_idx;
+	u16 used_idx;
 	int fw_state;
=20
 	/* keep last in the struct */
@@ -804,6 +806,7 @@ static int create_virtqueue(struct mlx5_vdpa_net *ndev,=
 struct mlx5_vdpa_virtque
=20
 	obj_context =3D MLX5_ADDR_OF(create_virtio_net_q_in, in, obj_context);
 	MLX5_SET(virtio_net_q_object, obj_context, hw_available_index, mvq->avail=
_idx);
+	MLX5_SET(virtio_net_q_object, obj_context, hw_used_index, mvq->used_idx);
 	MLX5_SET(virtio_net_q_object, obj_context, queue_feature_bit_mask_12_3,
 		 get_features_12_3(ndev->mvdev.actual_features));
 	vq_ctx =3D MLX5_ADDR_OF(virtio_net_q_object, obj_context, virtio_q_contex=
t);
@@ -1022,6 +1025,7 @@ static int connect_qps(struct mlx5_vdpa_net *ndev, st=
ruct mlx5_vdpa_virtqueue *m
 struct mlx5_virtq_attr {
 	u8 state;
 	u16 available_index;
+	u16 used_index;
 };
=20
 static int query_virtqueue(struct mlx5_vdpa_net *ndev, struct mlx5_vdpa_vi=
rtqueue *mvq,
@@ -1052,6 +1056,7 @@ static int query_virtqueue(struct mlx5_vdpa_net *ndev=
, struct mlx5_vdpa_virtqueu
 	memset(attr, 0, sizeof(*attr));
 	attr->state =3D MLX5_GET(virtio_net_q_object, obj_context, state);
 	attr->available_index =3D MLX5_GET(virtio_net_q_object, obj_context, hw_a=
vailable_index);
+	attr->used_index =3D MLX5_GET(virtio_net_q_object, obj_context, hw_used_i=
ndex);
 	kfree(out);
 	return 0;
=20
@@ -1610,6 +1615,7 @@ static int save_channel_info(struct mlx5_vdpa_net *nd=
ev, struct mlx5_vdpa_virtqu
 		return err;
=20
 	ri->avail_index =3D attr.available_index;
+	ri->used_index =3D attr.used_index;
 	ri->ready =3D mvq->ready;
 	ri->num_ent =3D mvq->num_ent;
 	ri->desc_addr =3D mvq->desc_addr;
@@ -1654,6 +1660,7 @@ static void restore_channels_info(struct mlx5_vdpa_ne=
t *ndev)
 			continue;
=20
 		mvq->avail_idx =3D ri->avail_index;
+		mvq->used_idx =3D ri->used_index;
 		mvq->ready =3D ri->ready;
 		mvq->num_ent =3D ri->num_ent;
 		mvq->desc_addr =3D ri->desc_addr;
--=20
2.29.2