Hi newsky647, On 6/12/22 16:09, 835703...@qq.com wrote:
From: newsky647 <835703...@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: newsky647 <835703...@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))
The patch looks good to me, but I would need your real name for the sign-off. Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com> Thanks, Maxime
