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 <as...@mellanox.com> > Acked-by: Matan Azrad <ma...@mellanox.com> > --- > drivers/vdpa/mlx5/mlx5_vdpa_virtq.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com> Thanks, Maxime