vhost: check vDPA configuration succeed

This patch checks whether vDPA device configuration
succeed and does not set the CONFIGURED flag if it
didn't.

Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
Reviewed-by: Chenbo Xia <chenbo.xia@intel.com>

diff --git a/lib/librte_vhost/vhost_user.c b/lib/librte_vhost/vhost_user.c
index 1d9e112..a96bc5c 100644 (file)
--- a/lib/librte_vhost/vhost_user.c
+++ b/lib/librte_vhost/vhost_user.c
@@ -2866,8 +2866,11 @@ skip_to_post_handle:
                goto out;
 
        if (!(dev->flags & VIRTIO_DEV_VDPA_CONFIGURED)) {
-               vdpa_dev->ops->dev_conf(dev->vid);
-               dev->flags |= VIRTIO_DEV_VDPA_CONFIGURED;
+               if (vdpa_dev->ops->dev_conf(dev->vid))
+                       VHOST_LOG_CONFIG(ERR,
+                                        "Failed to configure vDPA device\n");
+               else
+                       dev->flags |= VIRTIO_DEV_VDPA_CONFIGURED;
        }
 
 out: