| Message ID | 20200916042851.32914-1-ajit.khaparde@broadcom.com (mailing list archive) |
|---|---|
| Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 5E84AA04C7; Wed, 16 Sep 2020 06:28:59 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 5421B1C119; Wed, 16 Sep 2020 06:28:58 +0200 (CEST) Received: from mail-pl1-f227.google.com (mail-pl1-f227.google.com [209.85.214.227]) by dpdk.org (Postfix) with ESMTP id 318931C116 for <dev@dpdk.org>; Wed, 16 Sep 2020 06:28:56 +0200 (CEST) Received: by mail-pl1-f227.google.com with SMTP id f1so2494597plo.13 for <dev@dpdk.org>; Tue, 15 Sep 2020 21:28:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=from:to:subject:date:message-id:in-reply-to:references:mime-version :content-transfer-encoding; bh=/5HTTarS5Ftrl/OTtm2xljqKDqdknTB+XL/NNZHgjUM=; b=OmV1SPRZucSDxV4om+erLDvoqGar3O3cfvzcd4F+NU5FHedjSeYRBH/imSLr/eEhpW q9agCV75/xej1Jrqdtj4JmU3B2fgt3FspEhCeGS3s87LfmiblVNKkz7Yk3HNTLgCu6HU USvpPSzo/RAavnKExKOj4vujLodVtZOCskBkQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=/5HTTarS5Ftrl/OTtm2xljqKDqdknTB+XL/NNZHgjUM=; b=YcvM8Z9m4usbdbh0pPeAx0SFW76As4ItWRPCI+qr0vgkhLYHf3rL6Opx/gb+sVNyrd Vcs6V7WCj5Cf5Rh1poKy/gIVuPoBBR25CvB9eCA3k+PIzLdM8AUvZp3t2lsfvUF3IBx4 wjSMI2iI8CxnS4kIQXAvlZo7UXozLR+mbvH1IBcgJMCq77ArFyODx0vUGw6SVeK519p3 Ns3NPuEkLAvaTgISNWGL1lJNEkxCcTB+fIbT2+5wPjE2uqDP/pgv4nShxYoqYEdjyXY4 SuKAK3ct2eLUVGH61BeAd3JlRWTcR8v9/4+eXRJECNckZ1YUFX3el8pxp38ObdTWbCjD GaOA== X-Gm-Message-State: AOAM531jexV+A/+G649QwalsCDTWl5mA2NlLccp1FD5v/68zPypF8mqe 7SZaKpTpGs0tj4tuyw/64I3PN0NphbLWjz9O36i6bYKcCbADRxqnpTmP0AVBcyyvyfrzeoJH3fq tbyEixnHsW9qxF1W0SS2GkE9oLG6Gf+W7pTU1LxcApwlo/TYt1iqFF32Otss8wRsQ7TgpF9IlDL Fx1w== X-Google-Smtp-Source: ABdhPJzIB9iBqjyAK2cS7OvvAzPMqfg1oSa+IRYfJ5nm/atit+Sdoos7qoofsygeexfSwB612UPrqqeZ1KQY X-Received: by 2002:a17:902:b20d:b029:d0:92cc:a1dd with SMTP id t13-20020a170902b20db02900d092cca1ddmr22053824plr.12.1600230535054; Tue, 15 Sep 2020 21:28:55 -0700 (PDT) Received: from localhost.localdomain ([192.19.223.252]) by smtp-relay.gmail.com with ESMTPS id hk6sm132636pjb.6.2020.09.15.21.28.54 for <dev@dpdk.org> (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 15 Sep 2020 21:28:55 -0700 (PDT) X-Relaying-Domain: broadcom.com From: Ajit Khaparde <ajit.khaparde@broadcom.com> To: dev@dpdk.org Date: Tue, 15 Sep 2020 21:28:26 -0700 Message-Id: <20200916042851.32914-1-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 2.21.1 (Apple Git-122.3) In-Reply-To: <20200911015603.88359-1-ajit.khaparde@broadcom.com> References: <20200911015603.88359-1-ajit.khaparde@broadcom.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v2 00/25] patchset for bnxt X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
| Series |
patchset for bnxt
|
|
Message
Ajit Khaparde
Sept. 16, 2020, 4:28 a.m. UTC
Some fixes and enhancements in the PMD and TruFlow layers. v1->v2: - rebased to latest - updated commit messages Kishore Padmanabha (14): net/bnxt: fix port stop process and cleanup resources net/bnxt: fix the drop action flow to support count net/bnxt: reject flow offload with invalid MAC net/bnxt: reduce debug log messages net/bnxt: fix coexistence of ipv4 and ipv6 ingress rules net/bnxt: modify default flow rule creation net/bnxt: fix the function id used in flow flush net/bnxt: refactor VFR port clean up net/bnxt: remove VLAN pop action for egress flows net/bnxt: increase counter support from 8K to 16K net/bnxt: enable VXLAN ipv6 encapsulation net/bnxt: enable NAT action with tagged traffic net/bnxt: fix out of bound access in bit handling net/bnxt: add locks in flow database Mike Baucom (1): net/bnxt: free the EM index on failure Randy Schacher (1): net/bnxt: use direct HWRM message for interface table Shahaji Bhosle (2): net/bnxt: add null pointer check for resource manager net/bnxt: update resource settings Somnath Kotur (6): net/bnxt: fix crash in VFR queue select net/bnxt: fix VFR cleanup during init failure net/bnxt: check and set initial counter ID net/bnxt: fix bugs in representor data path net/bnxt: fix to check VNIC in shutdown path net/bnxt: add separate mutex for FW health check Sriharsha Basavapatna (1): net/bnxt: provide switch info if VFR are configured drivers/net/bnxt/bnxt.h | 13 +- drivers/net/bnxt/bnxt_ethdev.c | 101 +- drivers/net/bnxt/bnxt_filter.c | 14 +- drivers/net/bnxt/bnxt_hwrm.c | 13 +- drivers/net/bnxt/bnxt_reps.c | 162 +- drivers/net/bnxt/bnxt_reps.h | 8 + drivers/net/bnxt/bnxt_rxr.c | 27 +- drivers/net/bnxt/bnxt_vnic.c | 3 + drivers/net/bnxt/hsi_struct_def_dpdk.h | 935 ++- drivers/net/bnxt/tf_core/tf_em_internal.c | 5 +- drivers/net/bnxt/tf_core/tf_msg.c | 58 +- drivers/net/bnxt/tf_core/tf_rm.c | 14 + drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 569 +- drivers/net/bnxt/tf_ulp/bnxt_ulp.h | 34 +- drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c | 56 +- drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 131 +- drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c | 8 +- drivers/net/bnxt/tf_ulp/ulp_fc_mgr.h | 1 + drivers/net/bnxt/tf_ulp/ulp_flow_db.c | 17 +- drivers/net/bnxt/tf_ulp/ulp_flow_db.h | 4 +- drivers/net/bnxt/tf_ulp/ulp_mapper.c | 52 +- drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c | 6 - drivers/net/bnxt/tf_ulp/ulp_port_db.c | 41 + drivers/net/bnxt/tf_ulp/ulp_port_db.h | 13 + drivers/net/bnxt/tf_ulp/ulp_rte_parser.c | 153 +- drivers/net/bnxt/tf_ulp/ulp_rte_parser.h | 5 +- drivers/net/bnxt/tf_ulp/ulp_template_db_act.c | 295 +- .../net/bnxt/tf_ulp/ulp_template_db_class.c | 5531 +++++++++++------ .../net/bnxt/tf_ulp/ulp_template_db_enum.h | 66 +- .../net/bnxt/tf_ulp/ulp_template_db_field.h | 767 ++- drivers/net/bnxt/tf_ulp/ulp_utils.c | 43 +- drivers/net/bnxt/tf_ulp/ulp_utils.h | 7 +- 32 files changed, 5836 insertions(+), 3316 deletions(-)
Comments
On Tue, Sep 15, 2020 at 9:28 PM Ajit Khaparde <ajit.khaparde@broadcom.com> wrote: > > Some fixes and enhancements in the PMD and TruFlow layers. > > v1->v2: > - rebased to latest > - updated commit messages Applied to dpdk-next-net-brcm. > > Kishore Padmanabha (14): > net/bnxt: fix port stop process and cleanup resources > net/bnxt: fix the drop action flow to support count > net/bnxt: reject flow offload with invalid MAC > net/bnxt: reduce debug log messages > net/bnxt: fix coexistence of ipv4 and ipv6 ingress rules > net/bnxt: modify default flow rule creation > net/bnxt: fix the function id used in flow flush > net/bnxt: refactor VFR port clean up > net/bnxt: remove VLAN pop action for egress flows > net/bnxt: increase counter support from 8K to 16K > net/bnxt: enable VXLAN ipv6 encapsulation > net/bnxt: enable NAT action with tagged traffic > net/bnxt: fix out of bound access in bit handling > net/bnxt: add locks in flow database > > Mike Baucom (1): > net/bnxt: free the EM index on failure > > Randy Schacher (1): > net/bnxt: use direct HWRM message for interface table > > Shahaji Bhosle (2): > net/bnxt: add null pointer check for resource manager > net/bnxt: update resource settings > > Somnath Kotur (6): > net/bnxt: fix crash in VFR queue select > net/bnxt: fix VFR cleanup during init failure > net/bnxt: check and set initial counter ID > net/bnxt: fix bugs in representor data path > net/bnxt: fix to check VNIC in shutdown path > net/bnxt: add separate mutex for FW health check > > Sriharsha Basavapatna (1): > net/bnxt: provide switch info if VFR are configured > > drivers/net/bnxt/bnxt.h | 13 +- > drivers/net/bnxt/bnxt_ethdev.c | 101 +- > drivers/net/bnxt/bnxt_filter.c | 14 +- > drivers/net/bnxt/bnxt_hwrm.c | 13 +- > drivers/net/bnxt/bnxt_reps.c | 162 +- > drivers/net/bnxt/bnxt_reps.h | 8 + > drivers/net/bnxt/bnxt_rxr.c | 27 +- > drivers/net/bnxt/bnxt_vnic.c | 3 + > drivers/net/bnxt/hsi_struct_def_dpdk.h | 935 ++- > drivers/net/bnxt/tf_core/tf_em_internal.c | 5 +- > drivers/net/bnxt/tf_core/tf_msg.c | 58 +- > drivers/net/bnxt/tf_core/tf_rm.c | 14 + > drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 569 +- > drivers/net/bnxt/tf_ulp/bnxt_ulp.h | 34 +- > drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c | 56 +- > drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 131 +- > drivers/net/bnxt/tf_ulp/ulp_fc_mgr.c | 8 +- > drivers/net/bnxt/tf_ulp/ulp_fc_mgr.h | 1 + > drivers/net/bnxt/tf_ulp/ulp_flow_db.c | 17 +- > drivers/net/bnxt/tf_ulp/ulp_flow_db.h | 4 +- > drivers/net/bnxt/tf_ulp/ulp_mapper.c | 52 +- > drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c | 6 - > drivers/net/bnxt/tf_ulp/ulp_port_db.c | 41 + > drivers/net/bnxt/tf_ulp/ulp_port_db.h | 13 + > drivers/net/bnxt/tf_ulp/ulp_rte_parser.c | 153 +- > drivers/net/bnxt/tf_ulp/ulp_rte_parser.h | 5 +- > drivers/net/bnxt/tf_ulp/ulp_template_db_act.c | 295 +- > .../net/bnxt/tf_ulp/ulp_template_db_class.c | 5531 +++++++++++------ > .../net/bnxt/tf_ulp/ulp_template_db_enum.h | 66 +- > .../net/bnxt/tf_ulp/ulp_template_db_field.h | 767 ++- > drivers/net/bnxt/tf_ulp/ulp_utils.c | 43 +- > drivers/net/bnxt/tf_ulp/ulp_utils.h | 7 +- > 32 files changed, 5836 insertions(+), 3316 deletions(-) > > -- > 2.21.1 (Apple Git-122.3) >
On 9/16/2020 5:21 PM, Ajit Khaparde wrote: > On Tue, Sep 15, 2020 at 9:28 PM Ajit Khaparde > <ajit.khaparde@broadcom.com> wrote: >> >> Some fixes and enhancements in the PMD and TruFlow layers. >> >> v1->v2: >> - rebased to latest >> - updated commit messages > Applied to dpdk-next-net-brcm. > >> >> Kishore Padmanabha (14): >> net/bnxt: fix port stop process and cleanup resources >> net/bnxt: fix the drop action flow to support count >> net/bnxt: reject flow offload with invalid MAC >> net/bnxt: reduce debug log messages >> net/bnxt: fix coexistence of ipv4 and ipv6 ingress rules >> net/bnxt: modify default flow rule creation >> net/bnxt: fix the function id used in flow flush >> net/bnxt: refactor VFR port clean up >> net/bnxt: remove VLAN pop action for egress flows >> net/bnxt: increase counter support from 8K to 16K >> net/bnxt: enable VXLAN ipv6 encapsulation >> net/bnxt: enable NAT action with tagged traffic >> net/bnxt: fix out of bound access in bit handling >> net/bnxt: add locks in flow database >> >> Mike Baucom (1): >> net/bnxt: free the EM index on failure >> >> Randy Schacher (1): >> net/bnxt: use direct HWRM message for interface table >> >> Shahaji Bhosle (2): >> net/bnxt: add null pointer check for resource manager >> net/bnxt: update resource settings >> >> Somnath Kotur (6): >> net/bnxt: fix crash in VFR queue select >> net/bnxt: fix VFR cleanup during init failure >> net/bnxt: check and set initial counter ID >> net/bnxt: fix bugs in representor data path >> net/bnxt: fix to check VNIC in shutdown path >> net/bnxt: add separate mutex for FW health check >> >> Sriharsha Basavapatna (1): >> net/bnxt: provide switch info if VFR are configured >> Hi Ajit, I assume the backporting of the majority of the patchset explicitly not requested, since two of them has stable tag but rest don't, but I would like to confirm. And I suggest backporting fixes as much as possible, since missing some fixes may cause conflict for future fixes and prevent backporting them.
On Wed, Sep 16, 2020 at 4:58 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote: > > On 9/16/2020 5:21 PM, Ajit Khaparde wrote: > > On Tue, Sep 15, 2020 at 9:28 PM Ajit Khaparde > > <ajit.khaparde@broadcom.com> wrote: > >> > >> Some fixes and enhancements in the PMD and TruFlow layers. > >> > >> v1->v2: > >> - rebased to latest > >> - updated commit messages > > Applied to dpdk-next-net-brcm. > > > >> > >> Kishore Padmanabha (14): > >> net/bnxt: fix port stop process and cleanup resources > >> net/bnxt: fix the drop action flow to support count > >> net/bnxt: reject flow offload with invalid MAC > >> net/bnxt: reduce debug log messages > >> net/bnxt: fix coexistence of ipv4 and ipv6 ingress rules > >> net/bnxt: modify default flow rule creation > >> net/bnxt: fix the function id used in flow flush > >> net/bnxt: refactor VFR port clean up > >> net/bnxt: remove VLAN pop action for egress flows > >> net/bnxt: increase counter support from 8K to 16K > >> net/bnxt: enable VXLAN ipv6 encapsulation > >> net/bnxt: enable NAT action with tagged traffic > >> net/bnxt: fix out of bound access in bit handling > >> net/bnxt: add locks in flow database > >> > >> Mike Baucom (1): > >> net/bnxt: free the EM index on failure > >> > >> Randy Schacher (1): > >> net/bnxt: use direct HWRM message for interface table > >> > >> Shahaji Bhosle (2): > >> net/bnxt: add null pointer check for resource manager > >> net/bnxt: update resource settings > >> > >> Somnath Kotur (6): > >> net/bnxt: fix crash in VFR queue select > >> net/bnxt: fix VFR cleanup during init failure > >> net/bnxt: check and set initial counter ID > >> net/bnxt: fix bugs in representor data path > >> net/bnxt: fix to check VNIC in shutdown path > >> net/bnxt: add separate mutex for FW health check > >> > >> Sriharsha Basavapatna (1): > >> net/bnxt: provide switch info if VFR are configured > >> > > Hi Ajit, > > I assume the backporting of the majority of the patchset explicitly not > requested, since two of them has stable tag but rest don't, but I would > like to confirm. > > And I suggest backporting fixes as much as possible, since missing some > fixes may cause conflict for future fixes and prevent backporting them. Hi Ferruh, Most of the fixes are actually for changes made in 20.08 Since it is not an LTS, I think not really porting them should not be a problem. But if there is any patch which needs to go into earlier LTS releases, I will add it to the list of backports. Thanks for checking. Ajit > >
On 9/17/2020 1:13 AM, Ajit Khaparde wrote: > On Wed, Sep 16, 2020 at 4:58 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote: >> >> On 9/16/2020 5:21 PM, Ajit Khaparde wrote: >>> On Tue, Sep 15, 2020 at 9:28 PM Ajit Khaparde >>> <ajit.khaparde@broadcom.com> wrote: >>>> >>>> Some fixes and enhancements in the PMD and TruFlow layers. >>>> >>>> v1->v2: >>>> - rebased to latest >>>> - updated commit messages >>> Applied to dpdk-next-net-brcm. >>> >>>> >>>> Kishore Padmanabha (14): >>>> net/bnxt: fix port stop process and cleanup resources >>>> net/bnxt: fix the drop action flow to support count >>>> net/bnxt: reject flow offload with invalid MAC >>>> net/bnxt: reduce debug log messages >>>> net/bnxt: fix coexistence of ipv4 and ipv6 ingress rules >>>> net/bnxt: modify default flow rule creation >>>> net/bnxt: fix the function id used in flow flush >>>> net/bnxt: refactor VFR port clean up >>>> net/bnxt: remove VLAN pop action for egress flows >>>> net/bnxt: increase counter support from 8K to 16K >>>> net/bnxt: enable VXLAN ipv6 encapsulation >>>> net/bnxt: enable NAT action with tagged traffic >>>> net/bnxt: fix out of bound access in bit handling >>>> net/bnxt: add locks in flow database >>>> >>>> Mike Baucom (1): >>>> net/bnxt: free the EM index on failure >>>> >>>> Randy Schacher (1): >>>> net/bnxt: use direct HWRM message for interface table >>>> >>>> Shahaji Bhosle (2): >>>> net/bnxt: add null pointer check for resource manager >>>> net/bnxt: update resource settings >>>> >>>> Somnath Kotur (6): >>>> net/bnxt: fix crash in VFR queue select >>>> net/bnxt: fix VFR cleanup during init failure >>>> net/bnxt: check and set initial counter ID >>>> net/bnxt: fix bugs in representor data path >>>> net/bnxt: fix to check VNIC in shutdown path >>>> net/bnxt: add separate mutex for FW health check >>>> >>>> Sriharsha Basavapatna (1): >>>> net/bnxt: provide switch info if VFR are configured >>>> >> >> Hi Ajit, >> >> I assume the backporting of the majority of the patchset explicitly not >> requested, since two of them has stable tag but rest don't, but I would >> like to confirm. >> >> And I suggest backporting fixes as much as possible, since missing some >> fixes may cause conflict for future fixes and prevent backporting them. > Hi Ferruh, > Most of the fixes are actually for changes made in 20.08 > Since it is not an LTS, I think not really porting them should not be a problem. > But if there is any patch which needs to go into earlier LTS releases, > I will add it to the list of backports. > This is not just for LTS, if somehow we have 20.08.1, they will matter. Also this is good for documenting the actual intention of the author related backporting the patches. As far as I can see there is not explicit request to "not" backport them, so I will add the missing tags while merging.