DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices
@ 2020-03-23 17:50 Asaf Penso
  2020-04-15  7:34 ` Maxime Coquelin
  2020-04-17 17:14 ` Maxime Coquelin
  0 siblings, 2 replies; 4+ messages in thread
From: Asaf Penso @ 2020-03-23 17:50 UTC (permalink / raw)
  To: dev; +Cc: viacheslavo, matan

The rte_vhost_get_vring_base function is being called to get the values
of last_avail_idx and last_used_idx.
These fields will not have the correct values in case the function
returns an error.

Adding a check for the function return value, and in the case of an
error, set the fields to be zero and print a warning message.

Signed-off-by: Asaf Penso <asafp@mellanox.com>
Acked-by: Matan Azrad <matan@mellanox.com>
---
 drivers/vdpa/mlx5/mlx5_vdpa_virtq.c | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/drivers/vdpa/mlx5/mlx5_vdpa_virtq.c b/drivers/vdpa/mlx5/mlx5_vdpa_virtq.c
index 2312331..cb2d61b 100644
--- a/drivers/vdpa/mlx5/mlx5_vdpa_virtq.c
+++ b/drivers/vdpa/mlx5/mlx5_vdpa_virtq.c
@@ -236,10 +236,17 @@
 		}
 		attr.available_addr = gpa;
 	}
-	rte_vhost_get_vring_base(priv->vid, index, &last_avail_idx,
+	ret = rte_vhost_get_vring_base(priv->vid, index, &last_avail_idx,
 				 &last_used_idx);
-	DRV_LOG(INFO, "vid %d: Init last_avail_idx=%d, last_used_idx=%d for "
-		"virtq %d.", priv->vid, last_avail_idx, last_used_idx, index);
+	if (ret) {
+		last_avail_idx = 0;
+		last_used_idx = 0;
+		DRV_LOG(WARNING, "Couldn't get vring base, idx are set to 0");
+	} else {
+		DRV_LOG(INFO, "vid %d: Init last_avail_idx=%d, last_used_idx=%d for "
+				"virtq %d.", priv->vid, last_avail_idx,
+				last_used_idx, index);
+	}
 	attr.hw_available_index = last_avail_idx;
 	attr.hw_used_index = last_used_idx;
 	attr.q_size = vq.size;
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices
  2020-03-23 17:50 [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices Asaf Penso
@ 2020-04-15  7:34 ` Maxime Coquelin
  2020-04-15 11:52   ` Asaf Penso
  2020-04-17 17:14 ` Maxime Coquelin
  1 sibling, 1 reply; 4+ messages in thread
From: Maxime Coquelin @ 2020-04-15  7:34 UTC (permalink / raw)
  To: Asaf Penso, dev; +Cc: viacheslavo, matan

Hi Asaf,

On 3/23/20 6:50 PM, Asaf Penso wrote:
> The rte_vhost_get_vring_base function is being called to get the values
> of last_avail_idx and last_used_idx.
> These fields will not have the correct values in case the function
> returns an error.
> 
> Adding a check for the function return value, and in the case of an
> error, set the fields to be zero and print a warning message.

Fixes tag is missing here.
I will add it while merging:

Fixes: bff735011078 ("vdpa/mlx5: prepare virtio queues")

> Signed-off-by: Asaf Penso <asafp@mellanox.com>
> Acked-by: Matan Azrad <matan@mellanox.com>
> ---
>  drivers/vdpa/mlx5/mlx5_vdpa_virtq.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices
  2020-04-15  7:34 ` Maxime Coquelin
@ 2020-04-15 11:52   ` Asaf Penso
  0 siblings, 0 replies; 4+ messages in thread
From: Asaf Penso @ 2020-04-15 11:52 UTC (permalink / raw)
  To: Maxime Coquelin, dev; +Cc: Slava Ovsiienko, Matan Azrad

Thanks Maxime.

Regards,
Asaf Penso

> -----Original Message-----
> From: Maxime Coquelin <maxime.coquelin@redhat.com>
> Sent: Wednesday, April 15, 2020 10:35 AM
> To: Asaf Penso <asafp@mellanox.com>; dev@dpdk.org
> Cc: Slava Ovsiienko <viacheslavo@mellanox.com>; Matan Azrad
> <matan@mellanox.com>
> Subject: Re: [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices
> 
> Hi Asaf,
> 
> On 3/23/20 6:50 PM, Asaf Penso wrote:
> > The rte_vhost_get_vring_base function is being called to get the values
> > of last_avail_idx and last_used_idx.
> > These fields will not have the correct values in case the function
> > returns an error.
> >
> > Adding a check for the function return value, and in the case of an
> > error, set the fields to be zero and print a warning message.
> 
> Fixes tag is missing here.
> I will add it while merging:
> 
> Fixes: bff735011078 ("vdpa/mlx5: prepare virtio queues")
> 
> > Signed-off-by: Asaf Penso <asafp@mellanox.com>
> > Acked-by: Matan Azrad <matan@mellanox.com>
> > ---
> >  drivers/vdpa/mlx5/mlx5_vdpa_virtq.c | 13 ++++++++++---
> >  1 file changed, 10 insertions(+), 3 deletions(-)
> 
> Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> 
> Thanks,
> Maxime


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices
  2020-03-23 17:50 [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices Asaf Penso
  2020-04-15  7:34 ` Maxime Coquelin
@ 2020-04-17 17:14 ` Maxime Coquelin
  1 sibling, 0 replies; 4+ messages in thread
From: Maxime Coquelin @ 2020-04-17 17:14 UTC (permalink / raw)
  To: Asaf Penso, dev; +Cc: viacheslavo, matan



On 3/23/20 6:50 PM, Asaf Penso wrote:
> The rte_vhost_get_vring_base function is being called to get the values
> of last_avail_idx and last_used_idx.
> These fields will not have the correct values in case the function
> returns an error.
> 
> Adding a check for the function return value, and in the case of an
> error, set the fields to be zero and print a warning message.
> 
> Signed-off-by: Asaf Penso <asafp@mellanox.com>
> Acked-by: Matan Azrad <matan@mellanox.com>
> ---
>  drivers/vdpa/mlx5/mlx5_vdpa_virtq.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)

Applied to dpdk-next-virtio/master

Thanks,
Maxime


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-04-17 17:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-23 17:50 [dpdk-dev] [PATCH] vdpa/mlx5: set default queue indices Asaf Penso
2020-04-15  7:34 ` Maxime Coquelin
2020-04-15 11:52   ` Asaf Penso
2020-04-17 17:14 ` Maxime Coquelin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).