From: Gregory Etelson Date: Wed, 25 Nov 2020 09:44:27 +0000 (+0200) Subject: net/mlx5: fix tunnel offload freeing X-Git-Url: http://git.droids-corp.org/?p=dpdk.git;a=commitdiff_plain;h=bb5d49c6165b0c6ac32499ddc3e7fee41cfda51d net/mlx5: fix tunnel offload freeing PMD did not remove tunnel offload object from tunnels database before it released the object memory. As the result, the tunnels database become corrupted and subsequent search operations triggered PMD crash. The patch removes tunnel offload object from the tunnels database when the object is not in-use by PMD any more. Fixes: bc1d90a3cf6f ("net/mlx5: fix build with Direct Verbs disabled") Signed-off-by: Gregory Etelson Acked-by: Viacheslav Ovsiienko --- diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 642516ad4b..52ade39a42 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -7423,6 +7423,7 @@ mlx5_flow_tunnel_free(struct rte_eth_dev *dev, DRV_LOG(DEBUG, "port %u release pmd tunnel id=0x%x", dev->data->port_id, tunnel->tunnel_id); + LIST_REMOVE(tunnel, chain); mlx5_hlist_destroy(tunnel->groups); ipool = priv->sh->ipool[MLX5_IPOOL_TUNNEL_ID]; mlx5_ipool_free(ipool, tunnel->tunnel_id);