| Message ID | 20200713094213.21410-1-somnath.kotur@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 0CC64A0540; Mon, 13 Jul 2020 11:47:28 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B01431D573; Mon, 13 Jul 2020 11:47:26 +0200 (CEST) Received: from relay.smtp.broadcom.com (unknown [192.19.232.149]) by dpdk.org (Postfix) with ESMTP id 2FB3B1D56D for <dev@dpdk.org>; Mon, 13 Jul 2020 11:47:25 +0200 (CEST) Received: from dhcp-10-123-153-55.dhcp.broadcom.net (dhcp-10-123-153-55.dhcp.broadcom.net [10.123.153.55]) by relay.smtp.broadcom.com (Postfix) with ESMTP id 159131BB543; Mon, 13 Jul 2020 02:47:23 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.10.3 relay.smtp.broadcom.com 159131BB543 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com; s=dkimrelay; t=1594633644; bh=Gc6HNT0FxSi/A4iBQT87xgvP7hDaxGdKh+ON0/re4+c=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=iWiRIDKfrNmLogzceNhxnMmarHJZKy2dwssFdFOzU8bG478zCSF9Kl/+dClIuyha/ ZlLbdKXBKEay/+5UtKqoDxr2kWzhb83DT8b2KISBmnBFlfvpeQqBhbkVhrRCj50UaD 66vdPrBGJ3H+orkGZNcvyCuJsrAK0Buv9WIQ64RU= From: Somnath Kotur <somnath.kotur@broadcom.com> To: dev@dpdk.org Cc: ferruh.yigit@intel.com Date: Mon, 13 Jul 2020 15:12:03 +0530 Message-Id: <20200713094213.21410-1-somnath.kotur@broadcom.com> X-Mailer: git-send-email 2.10.1.613.g2cc2e70 In-Reply-To: <20200713061600.19456-1-somnath.kotur@broadcom.com> References: <20200713061600.19456-1-somnath.kotur@broadcom.com> Subject: [dpdk-dev] [PATCH v3 00/10] bnxt patches 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 |
bnxt patches
|
|
Message
Somnath Kotur
July 13, 2020, 9:42 a.m. UTC
Minor enhancments added to the TF-ULP/Core layers Farah Smith (1): net/bnxt: add changes to support 2 table scopes Jay Ding (2): net/bnxt: implement TF Identifier search net/bnxt: check index range in bulk get Kishore Padmanabha (6): net/bnxt: add support to extract data from the ulp blob net/bnxt: ignore ipv4 TOS mask net/bnxt: add support for identifier search and ref count net/bnxt: consider VLAN fields for template match criteria net/bnxt: increase the number of egress flow entries net/bnxt: add support for decrement TTL action Peter Spreadborough (1): net/bnxt: add option to delay EEM sysmem mapping drivers/net/bnxt/meson.build | 3 +- drivers/net/bnxt/tf_core/Makefile | 1 + drivers/net/bnxt/tf_core/tf_core.c | 78 +++++++- drivers/net/bnxt/tf_core/tf_core.h | 78 +++++++- drivers/net/bnxt/tf_core/tf_device.h | 20 +++ drivers/net/bnxt/tf_core/tf_device_p4.c | 2 + drivers/net/bnxt/tf_core/tf_em.h | 11 ++ drivers/net/bnxt/tf_core/tf_em_common.c | 145 ++++++++++++++- drivers/net/bnxt/tf_core/tf_em_common.h | 27 ++- drivers/net/bnxt/tf_core/tf_em_host.c | 23 +-- drivers/net/bnxt/tf_core/tf_em_internal.c | 2 +- drivers/net/bnxt/tf_core/tf_em_system.c | 25 +-- drivers/net/bnxt/tf_core/tf_identifier.c | 148 +++++++++++++++ drivers/net/bnxt/tf_core/tf_identifier.h | 47 +++++ drivers/net/bnxt/tf_core/tf_msg.c | 3 +- drivers/net/bnxt/tf_core/tf_rm.c | 43 +++++ drivers/net/bnxt/tf_core/tf_rm.h | 45 +++++ drivers/net/bnxt/tf_core/tf_shadow_identifier.c | 190 ++++++++++++++++++++ drivers/net/bnxt/tf_core/tf_shadow_identifier.h | 229 ++++++++++++++++++++++++ drivers/net/bnxt/tf_core/tf_tbl.c | 40 ++--- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 2 +- drivers/net/bnxt/tf_ulp/ulp_mapper.c | 117 +++++++++++- drivers/net/bnxt/tf_ulp/ulp_rte_parser.c | 74 +++++++- drivers/net/bnxt/tf_ulp/ulp_rte_parser.h | 5 + drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c | 4 +- drivers/net/bnxt/tf_ulp/ulp_utils.c | 76 ++++++++ drivers/net/bnxt/tf_ulp/ulp_utils.h | 17 ++ 27 files changed, 1367 insertions(+), 88 deletions(-) create mode 100644 drivers/net/bnxt/tf_core/tf_shadow_identifier.c create mode 100644 drivers/net/bnxt/tf_core/tf_shadow_identifier.h
Comments
On Mon, Jul 13, 2020 at 2:47 AM Somnath Kotur <somnath.kotur@broadcom.com> wrote: > Minor enhancments added to the TF-ULP/Core layers > > Farah Smith (1): > net/bnxt: add changes to support 2 table scopes > > Jay Ding (2): > net/bnxt: implement TF Identifier search > net/bnxt: check index range in bulk get > > Kishore Padmanabha (6): > net/bnxt: add support to extract data from the ulp blob > net/bnxt: ignore ipv4 TOS mask > net/bnxt: add support for identifier search and ref count > net/bnxt: consider VLAN fields for template match criteria > net/bnxt: increase the number of egress flow entries > net/bnxt: add support for decrement TTL action > > Peter Spreadborough (1): > net/bnxt: add option to delay EEM sysmem mapping > > drivers/net/bnxt/meson.build | 3 +- > drivers/net/bnxt/tf_core/Makefile | 1 + > drivers/net/bnxt/tf_core/tf_core.c | 78 +++++++- > drivers/net/bnxt/tf_core/tf_core.h | 78 +++++++- > drivers/net/bnxt/tf_core/tf_device.h | 20 +++ > drivers/net/bnxt/tf_core/tf_device_p4.c | 2 + > drivers/net/bnxt/tf_core/tf_em.h | 11 ++ > drivers/net/bnxt/tf_core/tf_em_common.c | 145 ++++++++++++++- > drivers/net/bnxt/tf_core/tf_em_common.h | 27 ++- > drivers/net/bnxt/tf_core/tf_em_host.c | 23 +-- > drivers/net/bnxt/tf_core/tf_em_internal.c | 2 +- > drivers/net/bnxt/tf_core/tf_em_system.c | 25 +-- > drivers/net/bnxt/tf_core/tf_identifier.c | 148 +++++++++++++++ > drivers/net/bnxt/tf_core/tf_identifier.h | 47 +++++ > drivers/net/bnxt/tf_core/tf_msg.c | 3 +- > drivers/net/bnxt/tf_core/tf_rm.c | 43 +++++ > drivers/net/bnxt/tf_core/tf_rm.h | 45 +++++ > drivers/net/bnxt/tf_core/tf_shadow_identifier.c | 190 ++++++++++++++++++++ > drivers/net/bnxt/tf_core/tf_shadow_identifier.h | 229 > ++++++++++++++++++++++++ > drivers/net/bnxt/tf_core/tf_tbl.c | 40 ++--- > drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 2 +- > drivers/net/bnxt/tf_ulp/ulp_mapper.c | 117 +++++++++++- > drivers/net/bnxt/tf_ulp/ulp_rte_parser.c | 74 +++++++- > drivers/net/bnxt/tf_ulp/ulp_rte_parser.h | 5 + > drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c | 4 +- > drivers/net/bnxt/tf_ulp/ulp_utils.c | 76 ++++++++ > drivers/net/bnxt/tf_ulp/ulp_utils.h | 17 ++ > 27 files changed, 1367 insertions(+), 88 deletions(-) > create mode 100644 drivers/net/bnxt/tf_core/tf_shadow_identifier.c > create mode 100644 drivers/net/bnxt/tf_core/tf_shadow_identifier.h > > -- > v1->v2: Corrected some typos in patch 5 and 6 > v2->v3: Modified case of some words in commit summary as per > git-check-log.sh > Patchset applied to dpdk-next-net-brcm. Thanks > 2.7.4 > >
On 7/14/2020 11:36 PM, Ajit Khaparde wrote: > > > On Mon, Jul 13, 2020 at 2:47 AM Somnath Kotur <somnath.kotur@broadcom.com > <mailto:somnath.kotur@broadcom.com>> wrote: > > Minor enhancments added to the TF-ULP/Core layers > > Farah Smith (1): > net/bnxt: add changes to support 2 table scopes > > Jay Ding (2): > net/bnxt: implement TF Identifier search > net/bnxt: check index range in bulk get > > Kishore Padmanabha (6): > net/bnxt: add support to extract data from the ulp blob > net/bnxt: ignore ipv4 TOS mask > net/bnxt: add support for identifier search and ref count > net/bnxt: consider VLAN fields for template match criteria > net/bnxt: increase the number of egress flow entries > net/bnxt: add support for decrement TTL action > > Peter Spreadborough (1): > net/bnxt: add option to delay EEM sysmem mapping > > drivers/net/bnxt/meson.build | 3 +- > drivers/net/bnxt/tf_core/Makefile | 1 + > drivers/net/bnxt/tf_core/tf_core.c | 78 +++++++- > drivers/net/bnxt/tf_core/tf_core.h | 78 +++++++- > drivers/net/bnxt/tf_core/tf_device.h | 20 +++ > drivers/net/bnxt/tf_core/tf_device_p4.c | 2 + > drivers/net/bnxt/tf_core/tf_em.h | 11 ++ > drivers/net/bnxt/tf_core/tf_em_common.c | 145 ++++++++++++++- > drivers/net/bnxt/tf_core/tf_em_common.h | 27 ++- > drivers/net/bnxt/tf_core/tf_em_host.c | 23 +-- > drivers/net/bnxt/tf_core/tf_em_internal.c | 2 +- > drivers/net/bnxt/tf_core/tf_em_system.c | 25 +-- > drivers/net/bnxt/tf_core/tf_identifier.c | 148 +++++++++++++++ > drivers/net/bnxt/tf_core/tf_identifier.h | 47 +++++ > drivers/net/bnxt/tf_core/tf_msg.c | 3 +- > drivers/net/bnxt/tf_core/tf_rm.c | 43 +++++ > drivers/net/bnxt/tf_core/tf_rm.h | 45 +++++ > drivers/net/bnxt/tf_core/tf_shadow_identifier.c | 190 ++++++++++++++++++++ > drivers/net/bnxt/tf_core/tf_shadow_identifier.h | 229 ++++++++++++++++++++++++ > drivers/net/bnxt/tf_core/tf_tbl.c | 40 ++--- > drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 2 +- > drivers/net/bnxt/tf_ulp/ulp_mapper.c | 117 +++++++++++- > drivers/net/bnxt/tf_ulp/ulp_rte_parser.c | 74 +++++++- > drivers/net/bnxt/tf_ulp/ulp_rte_parser.h | 5 + > drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c | 4 +- > drivers/net/bnxt/tf_ulp/ulp_utils.c | 76 ++++++++ > drivers/net/bnxt/tf_ulp/ulp_utils.h | 17 ++ > 27 files changed, 1367 insertions(+), 88 deletions(-) > create mode 100644 drivers/net/bnxt/tf_core/tf_shadow_identifier.c > create mode 100644 drivers/net/bnxt/tf_core/tf_shadow_identifier.h > > -- > v1->v2: Corrected some typos in patch 5 and 6 > v2->v3: Modified case of some words in commit summary as per git-check-log.sh > > Patchset applied to dpdk-next-net-brcm. Thanks > Hi Ajit, The patch "net/bnxt: add support for identifier search and ref count" is causing build error [1], can you please check it? [1] .../drivers/net/bnxt/tf_ulp/ulp_mapper.c: In function ‘ulp_mapper_ident_extract’: .../drivers/net/bnxt/tf_ulp/ulp_mapper.c:723:8: error: implicit declaration of function ‘tf_dir_2_str’ [-Werror=implicit-function-declaration] 723 | tf_dir_2_str(sparms.dir), | ^~~~~~~~~~~~ .../drivers/net/bnxt/bnxt.h:867:16: note: in definition of macro ‘PMD_DRV_LOG_RAW’ 867 | __func__, ## args) | ^~~~ .../drivers/net/bnxt/tf_ulp/bnxt_tf_common.h:14:40: note: in expansion of macro ‘PMD_DRV_LOG’ 14 | #define BNXT_TF_DBG(lvl, fmt, args...) PMD_DRV_LOG(lvl, fmt, ## args) | ^~~~~~~~~~~ .../drivers/net/bnxt/tf_ulp/ulp_mapper.c:722:3: note: in expansion of macro ‘BNXT_TF_DBG’ 722 | BNXT_TF_DBG(ERR, "Search ident %s:%s:%x failed.\n", | ^~~~~~~~~~~ .../drivers/net/bnxt/bnxt.h:866:50: error: format ‘%s’ expects argument of type ‘char *’, but argument 5 has type ‘int’ [-Werror=format=] 866 | rte_log(RTE_LOG_ ## level, bnxt_logtype_driver, "%s(): " fmt, \ | ^~~~~~~~ .../drivers/net/bnxt/bnxt.h:870:4: note: in expansion of macro ‘PMD_DRV_LOG_RAW’ 870 | PMD_DRV_LOG_RAW(level, fmt, ## args) | ^~~~~~~~~~~~~~~ .../drivers/net/bnxt/tf_ulp/bnxt_tf_common.h:14:40: note: in expansion of macro ‘PMD_DRV_LOG’ 14 | #define BNXT_TF_DBG(lvl, fmt, args...) PMD_DRV_LOG(lvl, fmt, ## args) | ^~~~~~~~~~~ .../drivers/net/bnxt/tf_ulp/ulp_mapper.c:722:3: note: in expansion of macro ‘BNXT_TF_DBG’ 722 | BNXT_TF_DBG(ERR, "Search ident %s:%s:%x failed.\n", | ^~~~~~~~~~~ .../drivers/net/bnxt/tf_ulp/ulp_mapper.c:722:35: note: format string is defined here 722 | BNXT_TF_DBG(ERR, "Search ident %s:%s:%x failed.\n", | ~^ | | | char * | %d