From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1F466A0524; Fri, 31 Jan 2020 06:09:46 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 5EB401C043; Fri, 31 Jan 2020 06:09:45 +0100 (CET) Received: from mail-vs1-f68.google.com (mail-vs1-f68.google.com [209.85.217.68]) by dpdk.org (Postfix) with ESMTP id AE1821C037 for ; Fri, 31 Jan 2020 06:09:43 +0100 (CET) Received: by mail-vs1-f68.google.com with SMTP id b79so3603992vsd.9 for ; Thu, 30 Jan 2020 21:09:43 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=CUwevbNfIuQ+HWLHZeN5hiJd1fC0+0+MckLEpSRB8UA=; b=AV0KWrkN5+qec1hxx4J4CbM7GCJ6ABBlML8cvV8F2+axcdUadNXlm1gVZvyfKWXMWd LtAMDk74zUsBu3LRHOW+3BqEc3cs0UpAsNojWhTT/YXHMRKSLyw67nC+Iuv7T2oeYDcY m1f1ADjz5LeXfwfQk5HWTx46WmabBa+z7TH8o= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=CUwevbNfIuQ+HWLHZeN5hiJd1fC0+0+MckLEpSRB8UA=; b=q1mmKu0HGK1795JcgDgBIZcEVrzHmgXjrD3IyQo/xxtkWlw4cV49OXmHJCnI/CdYpD morr/myagbctK5PWxbpPs4HFQ+jdT5GWLe8wMladtAFwPS1na2ifD5aHXZFLa6Cpf7SO rYhCh3oCXzXbMnsvEsmyorK3/1RwspWhpF1j081AmZVAI56gCl1S5URtxVmhxrxpSwsJ Lwl0mOO+XZfEQUCYXpB4Fx2tFbLJtmKF4PxRoZpgYBbr3c5dOXeVuXF5gm5+mvLJyAMt SumKM4Uw2CUjeWO/3y0W7tVXpMvSrHJkcIkNiF+3wcrcO5Mcsa96gq94t9HK/EtztbUF FhWw== X-Gm-Message-State: APjAAAW7JnI2zXJiKS7lwzcIE0rP8SgJzD3e0/i/3CLcP9cH2YuNE23+ wyLpMT/uC0bD3ECJrZ7OiHNbMkLJkFf7hfFqBN4EXdiuKAE= X-Google-Smtp-Source: APXvYqxk7/4Vi6Mf9eLaGLDG7RTuqllQNBV/AmQQCyT18rO9IKKdIFUB9MGpXWM2rDBJTfY11YQShPT0sGtqiFXDf2o= X-Received: by 2002:a05:6102:48b:: with SMTP id n11mr5504621vsa.181.1580447382728; Thu, 30 Jan 2020 21:09:42 -0800 (PST) MIME-Version: 1.0 References: <20200131050547.16665-1-somnath.kotur@broadcom.com> In-Reply-To: <20200131050547.16665-1-somnath.kotur@broadcom.com> From: Somnath Kotur Date: Fri, 31 Jan 2020 10:39:31 +0530 Message-ID: To: dev Cc: Ferruh Yigit Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH] net/bnxt: fix to remove spurious warning in Rx handler X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Please ignore this patch Thanks On Fri, Jan 31, 2020 at 10:37 AM Somnath Kotur wrote: > 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: ff88f133be45 ("net/bnxt: fix to support zero mark id along with RSS > action") > > Bug: CTRL-45914 > Change-Id: I1c1a95b0a4661fd7b8bbe02de62d18e0119d15d2 > Signed-off-by: Somnath Kotur > --- > 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 1960b05..1f47db9 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); > -- > 1.8.3.1 > >