[dpdk-stable] [PATCH 16.11] net/bnxt: fix close operation

[dpdk-stable] [PATCH 16.11] net/bnxt: fix close operation

[Ajit Khaparde]

[backported from commit 3c2e87c10b6f2c249a26e419b565bff5589b4637]

We are not cleaning up all the memory and also not unregistering
the driver during device close operation. This patch fixes the issue.

Fixes: 893074951314 ("net/bnxt: free memory in close operation")

Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt_ethdev.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index 86c2a8108..44bf6ba98 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -137,6 +137,7 @@ static struct rte_pci_id bnxt_pci_id_map[] = {
 	ETH_RSS_NONFRAG_IPV6_UDP)
 
 static void bnxt_print_link_info(struct rte_eth_dev *eth_dev);
+static int bnxt_dev_uninit(struct rte_eth_dev *eth_dev);
 
 /***********************/
 
@@ -586,6 +587,8 @@ static void bnxt_dev_close_op(struct rte_eth_dev *eth_dev)
 		rte_free(bp->grp_info);
 		bp->grp_info = NULL;
 	}
+
+	bnxt_dev_uninit(eth_dev);
 }
 
 static void bnxt_mac_addr_remove_op(struct rte_eth_dev *eth_dev,
-- 
2.15.2 (Apple Git-101.1)

Re: [dpdk-stable] [PATCH 16.11] net/bnxt: fix close operation

[Luca Boccassi]

On Thu, 2018-08-09 at 18:11 -0700, Ajit Khaparde wrote:
> [backported from commit 3c2e87c10b6f2c249a26e419b565bff5589b4637]
> 
> We are not cleaning up all the memory and also not unregistering
> the driver during device close operation. This patch fixes the issue.
> 
> Fixes: 893074951314 ("net/bnxt: free memory in close operation")
> 
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> ---
>  drivers/net/bnxt/bnxt_ethdev.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/net/bnxt/bnxt_ethdev.c
> b/drivers/net/bnxt/bnxt_ethdev.c
> index 86c2a8108..44bf6ba98 100644
> --- a/drivers/net/bnxt/bnxt_ethdev.c
> +++ b/drivers/net/bnxt/bnxt_ethdev.c
> @@ -137,6 +137,7 @@ static struct rte_pci_id bnxt_pci_id_map[] = {
>  	ETH_RSS_NONFRAG_IPV6_UDP)
>  
>  static void bnxt_print_link_info(struct rte_eth_dev *eth_dev);
> +static int bnxt_dev_uninit(struct rte_eth_dev *eth_dev);
>  
>  /***********************/
>  
> @@ -586,6 +587,8 @@ static void bnxt_dev_close_op(struct rte_eth_dev
> *eth_dev)
>  		rte_free(bp->grp_info);
>  		bp->grp_info = NULL;
>  	}
> +
> +	bnxt_dev_uninit(eth_dev);
>  }
>  
>  static void bnxt_mac_addr_remove_op(struct rte_eth_dev *eth_dev,

Thanks, applied and pushed to dpdk-stable/16.11.

-- 
Kind regards,
Luca Boccassi