[V2] vdpa/mlx5: return correct error code after rte_intr_instance_alloc failed
Checks
Commit Message
From: Shiqi Liu <835703180@qq.com>
After function rte_intr_instance_alloc failed, we should return ENOMEM
for error code.
Fixes: 5fe068bf7a2 ("vdpa/mlx5: reuse resources in reconfiguration")
Signed-off-by: Shiqi Liu <835703180@qq.com>
---
drivers/vdpa/mlx5/mlx5_vdpa_event.c | 1 +
1 file changed, 1 insertion(+)
Comments
On 6/14/22 15:36, 835703180@qq.com wrote:
> From: Shiqi Liu <835703180@qq.com>
>
> After function rte_intr_instance_alloc failed, we should return ENOMEM
> for error code.
>
> Fixes: 5fe068bf7a2 ("vdpa/mlx5: reuse resources in reconfiguration")
>
> Signed-off-by: Shiqi Liu <835703180@qq.com>
> ---
> drivers/vdpa/mlx5/mlx5_vdpa_event.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/vdpa/mlx5/mlx5_vdpa_event.c b/drivers/vdpa/mlx5/mlx5_vdpa_event.c
> index 7167a98db0..6223afaae8 100644
> --- a/drivers/vdpa/mlx5/mlx5_vdpa_event.c
> +++ b/drivers/vdpa/mlx5/mlx5_vdpa_event.c
> @@ -395,6 +395,7 @@ mlx5_vdpa_err_event_setup(struct mlx5_vdpa_priv *priv)
> rte_intr_instance_alloc(RTE_INTR_INSTANCE_F_SHARED);
> if (priv->err_intr_handle == NULL) {
> DRV_LOG(ERR, "Fail to allocate intr_handle");
> + rte_errno = ENOMEM;
> goto error;
> }
> if (rte_intr_fd_set(priv->err_intr_handle, priv->err_chnl->fd))
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
Thanks,
Maxime
@@ -395,6 +395,7 @@ mlx5_vdpa_err_event_setup(struct mlx5_vdpa_priv *priv)
rte_intr_instance_alloc(RTE_INTR_INSTANCE_F_SHARED);
if (priv->err_intr_handle == NULL) {
DRV_LOG(ERR, "Fail to allocate intr_handle");
+ rte_errno = ENOMEM;
goto error;
}
if (rte_intr_fd_set(priv->err_intr_handle, priv->err_chnl->fd))