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 9DE32A0524; Fri, 31 Jan 2020 07:25:52 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 3DE491C043; Fri, 31 Jan 2020 07:25:51 +0100 (CET) Received: from mail-ot1-f67.google.com (mail-ot1-f67.google.com [209.85.210.67]) by dpdk.org (Postfix) with ESMTP id 89D751C030 for ; Fri, 31 Jan 2020 07:25:49 +0100 (CET) Received: by mail-ot1-f67.google.com with SMTP id z9so5575575oth.5 for ; Thu, 30 Jan 2020 22:25:49 -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=w6rtxwd9q1oqX9I647kjL5+dBbjUKU46nlZCVu7dgac=; b=gmXA3OfItuf25MXNVSKT8g02dr/iMG2SkAhCbkrviPLTqWZntQGvqRALzvl4Mwfvk6 dQi/nIKO/uZQQFSxg+fhHS6t1F5yt6Hi8xIBycJWr2X/yOkQxCaEXrsIy+eccKQyykCv Lla8QUO0HbzwQSYjHUAqfy5r+G200vYM6hfb0= 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=w6rtxwd9q1oqX9I647kjL5+dBbjUKU46nlZCVu7dgac=; b=MOr45Hn8tdOBD8E5NYDJmaxjv6+3y1itzoFb05a0RyhqWE24ZqS+JymzaMqmGWARe+ 1hhszPBXbvs8r87/CX3fpIX64z+juFCtmliuFF57/5/H5DnEfwbMkTGJudC+wgXwDfHB OUCCbUTW5rQGgX50UtQkC5p1lW6IwZoD2tTK520e5vlKlUtQbBiRAK0l7CBX+jsT50gU +dJ52g1zmfpYej7jip+wRSemgbNcoVT9tZTTpu71RlZNZltHG4a3ZI00PkKckrIQAiSe qR5hgIUinnEj023G++fDNpVLr8JuB+yizNuKhg3uyQPqT/u73McHnDB+RJobCiNF9fFn HKTA== X-Gm-Message-State: APjAAAWXGfGQcYz98IjSVhTF65ZPrrhqGQLfa/OHkkq4CVSwLmnnhqZ2 FOXzYX3L2u/5ERfEGYxVz8I08y8f8p37QBSd0IiaVw== X-Google-Smtp-Source: APXvYqxk5TWS3DkmH3+7xGlOQHzRKJPax2IAg4md8v4wRboHQQ9QgjH1DW0NSA7+5dm1u7/nZEnGXfYZRKbyKFoowvA= X-Received: by 2002:a05:6830:1110:: with SMTP id w16mr3985456otq.305.1580451948504; Thu, 30 Jan 2020 22:25:48 -0800 (PST) MIME-Version: 1.0 References: <20200131061658.17364-1-somnath.kotur@broadcom.com> In-Reply-To: <20200131061658.17364-1-somnath.kotur@broadcom.com> From: Ajit Khaparde Date: Thu, 30 Jan 2020 22:25:31 -0800 Message-ID: To: Somnath Kotur Cc: dpdk-dev , Ferruh Yigit Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v2] 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" On Thu, Jan 30, 2020 at 10:18 PM 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: 94eb699bc82e ("net/bnxt: support flow mark action") > > Reviewed-by: Ajit Khaparde > Signed-off-by: Somnath Kotur > Patch applied to dpdk-next-net-brcm. Thanks --- > v1->v2: Used the correct commit id for the fixes tag > 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 > >