From: Somnath Kotur <somnath.kotur@broadcom.com>
Date: Fri, 31 Jan 2020 06:16:58 +0000 (+0530)
Subject: net/bnxt: remove spurious warning in Rx handler
X-Git-Url: http://git.droids-corp.org/?p=dpdk.git;a=commitdiff_plain;h=8522f7b12c146e7fa95728b2f23b4029709e14e6

net/bnxt: remove spurious warning in Rx handler

HW seems to populate the cfa code in the Rx descriptor even
if an explicit flow rule is not configured via application as
there might be a default rule configured in HW even for promisc
mode.

Fixes: 94eb699bc82e ("net/bnxt: support flow mark action")

Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
Reviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---

diff --git a/drivers/net/bnxt/bnxt_rxr.c b/drivers/net/bnxt/bnxt_rxr.c
index 1960b05151..1f47db97ba 100644
--- a/drivers/net/bnxt/bnxt_rxr.c
+++ b/drivers/net/bnxt/bnxt_rxr.c
@@ -909,11 +909,8 @@ void bnxt_set_mark_in_mbuf(struct bnxt *bp,
 	if (!cfa_code)
 		return;
 
-	if (cfa_code && !bp->mark_table[cfa_code].valid) {
-		PMD_DRV_LOG(WARNING, "Invalid mark_tbl entry! cfa_code: 0x%x\n",
-			    cfa_code);
+	if (cfa_code && !bp->mark_table[cfa_code].valid)
 		return;
-	}
 
 	flags2 = rte_le_to_cpu_16(rxcmp1->flags2);
 	meta = rte_le_to_cpu_32(rxcmp1->metadata);