net/bnxt: add a failure log
authorSomnath Kotur <somnath.kotur@broadcom.com>
Fri, 30 Oct 2020 07:03:03 +0000 (12:33 +0530)
committerFerruh Yigit <ferruh.yigit@intel.com>
Tue, 3 Nov 2020 22:35:07 +0000 (23:35 +0100)
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>
drivers/net/bnxt/bnxt_ethdev.c

index b199abb..64a5278 100644 (file)
@@ -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 */