From 8522f7b12c146e7fa95728b2f23b4029709e14e6 Mon Sep 17 00:00:00 2001 From: Somnath Kotur Date: Fri, 31 Jan 2020 11:46:58 +0530 Subject: [PATCH] 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 Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_rxr.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) 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); -- 2.20.1