net/bnxt: fix overwriting error message
authorAjit Khaparde <ajit.khaparde@broadcom.com>
Sat, 21 Dec 2019 02:42:18 +0000 (18:42 -0800)
committerFerruh Yigit <ferruh.yigit@intel.com>
Fri, 17 Jan 2020 18:46:01 +0000 (19:46 +0100)
In some cases when flow creation fails, we overwrite the specific
error message with a generic error message. This patch fixes it.

Fixes: d24610f7bfda ("net/bnxt: allow flow creation when RSS is enabled")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Reviewed-by: Lance Richardson <lance.richardson@broadcom.com>
drivers/net/bnxt/bnxt_flow.c

index 707aedc..cde1fa4 100644 (file)
@@ -1485,7 +1485,7 @@ ret:
                if (rxq && !vnic->rx_queue_cnt)
                        rxq->vnic = &bp->vnic_info[0];
        }
-       return rc;
+       return -rte_errno;
 }
 
 static
@@ -1815,7 +1815,7 @@ free_flow:
                rte_flow_error_set(error, 0,
                                   RTE_FLOW_ERROR_TYPE_NONE, NULL,
                                   "Flow with pattern exists, updating destination queue");
-       else
+       else if (!rte_errno)
                rte_flow_error_set(error, -ret,
                                   RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
                                   "Failed to create flow.");