net/bnxt: add a failure log

Check and log an error message if switch domain free API fails

Coverity issue: 362757
Fixes: 322bd6e70272 ("net/bnxt: add port representor infrastructure")
Cc: stable@dpdk.org
Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index b199abb783961e4c1f7c1f401708dfc904c9281c..64a5278f964a186bf88080615ec9f4cad0584b2e 100644 (file)
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -1346,8 +1346,14 @@ static int bnxt_dev_set_link_down_op(struct rte_eth_dev *eth_dev)
 
 static void bnxt_free_switch_domain(struct bnxt *bp)
 {
-       if (bp->switch_domain_id)
-               rte_eth_switch_domain_free(bp->switch_domain_id);
+       int rc = 0;
+
+       if (bp->switch_domain_id) {
+               rc = rte_eth_switch_domain_free(bp->switch_domain_id);
+               if (rc)
+                       PMD_DRV_LOG(ERR, "free switch domain:%d fail: %d\n",
+                                   bp->switch_domain_id, rc);
+       }
 }
 
 /* Unload the driver, release resources */