Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/103408/?format=api
http://patches.dpdk.org/api/patches/103408/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20211102040556.7840-10-venkatkumar.duvvuru@broadcom.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20211102040556.7840-10-venkatkumar.duvvuru@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20211102040556.7840-10-venkatkumar.duvvuru@broadcom.com", "date": "2021-11-02T04:05:45", "name": "[v3,09/20] net/bnxt: increase flow scale for Thor", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "f2b9ea0e7e803e7e344ebaf031096fc79eacdadb", "submitter": { "id": 1635, "url": "http://patches.dpdk.org/api/people/1635/?format=api", "name": "Venkat Duvvuru", "email": "venkatkumar.duvvuru@broadcom.com" }, "delegate": { "id": 1766, "url": "http://patches.dpdk.org/api/users/1766/?format=api", "username": "ajitkhaparde", "first_name": "Ajit", "last_name": "Khaparde", "email": "ajit.khaparde@broadcom.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20211102040556.7840-10-venkatkumar.duvvuru@broadcom.com/mbox/", "series": [ { "id": 20191, "url": "http://patches.dpdk.org/api/series/20191/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=20191", "date": "2021-11-02T04:05:36", "name": "fixes and enhancements to Truflow", "version": 3, "mbox": "http://patches.dpdk.org/series/20191/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/103408/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/103408/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 48499A0C4D;\n\tTue, 2 Nov 2021 05:07:17 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 7E23A4115A;\n\tTue, 2 Nov 2021 05:06:21 +0100 (CET)", "from relay.smtp-ext.broadcom.com (lpdvsmtp09.broadcom.com\n [192.19.166.228])\n by mails.dpdk.org (Postfix) with ESMTP id F342941157\n for <dev@dpdk.org>; Tue, 2 Nov 2021 05:06:19 +0100 (CET)", "from S60.dhcp.broadcom.net (unknown [10.123.66.170])\n (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))\n (No client certificate requested)\n by relay.smtp-ext.broadcom.com (Postfix) with ESMTPS id 0A0977FF6;\n Mon, 1 Nov 2021 21:06:17 -0700 (PDT)" ], "DKIM-Filter": "OpenDKIM Filter v2.11.0 relay.smtp-ext.broadcom.com 0A0977FF6", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n s=dkimrelay; t=1635825979;\n bh=7Lnv1VRnoTF06IJRFDg3tZuZ9wNB1D6HH2O1E3jcAtg=;\n h=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n b=SXqIs5+EODQQJQrHZrOrWhw0BE3rS3ovWzqdwBjTIdHIuvG49Rt8FDPXeVobg8UNq\n q3IKseDcLlkNohgHblmxGoXteUmFfmYwphPwwcw3mM6PmAjb4lhb4R9UMtqGP10OID\n EDSKUHxwbt0WnWkidqxwTeAkcRsuIWWOna+mURTo=", "From": "Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>", "To": "dev@dpdk.org", "Cc": "Shahaji Bhosle <sbhosle@broadcom.com>,\n Kishore Padmanabha <kishore.padmanabha@broadcom.com>,\n Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>", "Date": "Tue, 2 Nov 2021 09:35:45 +0530", "Message-Id": "<20211102040556.7840-10-venkatkumar.duvvuru@broadcom.com>", "X-Mailer": "git-send-email 2.17.1", "In-Reply-To": "<20211102040556.7840-1-venkatkumar.duvvuru@broadcom.com>", "References": "<20211001055909.27276-1-venkatkumar.duvvuru@broadcom.com>\n <20211102040556.7840-1-venkatkumar.duvvuru@broadcom.com>", "Subject": "[dpdk-dev] [PATCH v3 09/20] net/bnxt: increase flow scale for Thor", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <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>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "From: Shahaji Bhosle <sbhosle@broadcom.com>\n\n* Updated defines and data types to allow 256 VFRs.\n* Increased the encap record cache to support 256 to 4K entries. So\n VxLAN connections can be scaled to 4K entries.\n\nSigned-off-by: Shahaji Bhosle <sbhosle@broadcom.com>\nSigned-off-by: Kishore Padmanabha <kishore.padmanabha@broadcom.com>\nSigned-off-by: Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>\nReviewed-by: Kishore Padmanabha <kishore.padmanabha@broadcom.com>\nReviewed-by: Randy Schacher <stuart.schacher@broadcom.com>\n---\n drivers/net/bnxt/bnxt.h | 6 +++++-\n drivers/net/bnxt/bnxt_cpr.c | 2 +-\n drivers/net/bnxt/bnxt_ethdev.c | 18 +++++++++---------\n drivers/net/bnxt/bnxt_reps.c | 3 +--\n drivers/net/bnxt/tf_ulp/bnxt_ulp.h | 2 +-\n .../generic_templates/ulp_template_db_tbl.c | 8 ++++----\n drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 6 +++---\n 7 files changed, 24 insertions(+), 21 deletions(-)", "diff": "diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h\nindex c65e360446..5c064fd119 100644\n--- a/drivers/net/bnxt/bnxt.h\n+++ b/drivers/net/bnxt/bnxt.h\n@@ -240,7 +240,11 @@ struct bnxt_parent_info {\n struct bnxt_pf_info {\n #define BNXT_FIRST_PF_FID\t1\n #define BNXT_MAX_VFS(bp)\t((bp)->pf->max_vfs)\n-#define BNXT_MAX_VF_REPS\t64\n+#define BNXT_MAX_VF_REPS_WH 64\n+#define BNXT_MAX_VF_REPS_TH 256\n+#define BNXT_MAX_VF_REPS(bp) \\\n+\t\t\t\t(BNXT_CHIP_P5(bp) ? BNXT_MAX_VF_REPS_TH : \\\n+\t\t\t\tBNXT_MAX_VF_REPS_WH)\n #define BNXT_TOTAL_VFS(bp)\t((bp)->pf->total_vfs)\n #define BNXT_FIRST_VF_FID\t128\n #define BNXT_PF_RINGS_USED(bp)\tbnxt_get_num_queues(bp)\ndiff --git a/drivers/net/bnxt/bnxt_cpr.c b/drivers/net/bnxt/bnxt_cpr.c\nindex 63ff02a198..6bb70d516e 100644\n--- a/drivers/net/bnxt/bnxt_cpr.c\n+++ b/drivers/net/bnxt/bnxt_cpr.c\n@@ -74,7 +74,7 @@ bnxt_process_default_vnic_change(struct bnxt *bp,\n \t\t\tBNXT_DEFAULT_VNIC_CHANGE_VF_ID_SFT;\n \tPMD_DRV_LOG(INFO, \"async event received vf_id 0x%x\\n\", vf_fid);\n \n-\tfor (vf_id = 0; vf_id < BNXT_MAX_VF_REPS; vf_id++) {\n+\tfor (vf_id = 0; vf_id < BNXT_MAX_VF_REPS(bp); vf_id++) {\n \t\teth_dev = bp->rep_info[vf_id].vfr_eth_dev;\n \t\tif (!eth_dev)\n \t\t\tcontinue;\ndiff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c\nindex bf034db336..8c72ab8fd1 100644\n--- a/drivers/net/bnxt/bnxt_ethdev.c\n+++ b/drivers/net/bnxt/bnxt_ethdev.c\n@@ -1179,7 +1179,7 @@ void bnxt_print_link_info(struct rte_eth_dev *eth_dev)\n \tstruct rte_eth_link *link = ð_dev->data->dev_link;\n \n \tif (link->link_status)\n-\t\tPMD_DRV_LOG(INFO, \"Port %d Link Up - speed %u Mbps - %s\\n\",\n+\t\tPMD_DRV_LOG(DEBUG, \"Port %d Link Up - speed %u Mbps - %s\\n\",\n \t\t\teth_dev->data->port_id,\n \t\t\t(uint32_t)link->link_speed,\n \t\t\t(link->link_duplex == ETH_LINK_FULL_DUPLEX) ?\n@@ -6034,7 +6034,7 @@ static int bnxt_init_rep_info(struct bnxt *bp)\n \t\treturn 0;\n \n \tbp->rep_info = rte_zmalloc(\"bnxt_rep_info\",\n-\t\t\t\t sizeof(bp->rep_info[0]) * BNXT_MAX_VF_REPS,\n+\t\t\t\t sizeof(bp->rep_info[0]) * BNXT_MAX_VF_REPS(bp),\n \t\t\t\t 0);\n \tif (!bp->rep_info) {\n \t\tPMD_DRV_LOG(ERR, \"Failed to alloc memory for rep info\\n\");\n@@ -6076,7 +6076,9 @@ static int bnxt_rep_port_probe(struct rte_pci_device *pci_dev,\n {\n \tstruct rte_eth_dev *vf_rep_eth_dev;\n \tchar name[RTE_ETH_NAME_MAX_LEN];\n-\tstruct bnxt *backing_bp;\n+\tstruct bnxt *backing_bp = backing_eth_dev->data->dev_private;\n+\tuint16_t max_vf_reps = BNXT_MAX_VF_REPS(backing_bp);\n+\n \tuint16_t num_rep;\n \tint i, ret = 0;\n \tstruct rte_kvargs *kvlist = NULL;\n@@ -6089,9 +6091,9 @@ static int bnxt_rep_port_probe(struct rte_pci_device *pci_dev,\n \t\treturn -ENOTSUP;\n \t}\n \tnum_rep = eth_da->nb_representor_ports;\n-\tif (num_rep > BNXT_MAX_VF_REPS) {\n+\tif (num_rep > max_vf_reps) {\n \t\tPMD_DRV_LOG(ERR, \"nb_representor_ports = %d > %d MAX VF REPS\\n\",\n-\t\t\t num_rep, BNXT_MAX_VF_REPS);\n+\t\t\t num_rep, max_vf_reps);\n \t\treturn -EINVAL;\n \t}\n \n@@ -6102,8 +6104,6 @@ static int bnxt_rep_port_probe(struct rte_pci_device *pci_dev,\n \t\treturn -EINVAL;\n \t}\n \n-\tbacking_bp = backing_eth_dev->data->dev_private;\n-\n \tif (!(BNXT_PF(backing_bp) || BNXT_VF_IS_TRUSTED(backing_bp))) {\n \t\tPMD_DRV_LOG(ERR,\n \t\t\t \"Not a PF or trusted VF. No Representor support\\n\");\n@@ -6123,9 +6123,9 @@ static int bnxt_rep_port_probe(struct rte_pci_device *pci_dev,\n \t\t\t.parent_dev = backing_eth_dev\n \t\t};\n \n-\t\tif (representor.vf_id >= BNXT_MAX_VF_REPS) {\n+\t\tif (representor.vf_id >= max_vf_reps) {\n \t\t\tPMD_DRV_LOG(ERR, \"VF-Rep id %d >= %d MAX VF ID\\n\",\n-\t\t\t\t representor.vf_id, BNXT_MAX_VF_REPS);\n+\t\t\t\t representor.vf_id, max_vf_reps);\n \t\t\tcontinue;\n \t\t}\n \ndiff --git a/drivers/net/bnxt/bnxt_reps.c b/drivers/net/bnxt/bnxt_reps.c\nindex b7e88e013a..988f5a33a7 100644\n--- a/drivers/net/bnxt/bnxt_reps.c\n+++ b/drivers/net/bnxt/bnxt_reps.c\n@@ -210,7 +210,6 @@ int bnxt_representor_init(struct rte_eth_dev *eth_dev, void *params)\n \teth_dev->data->dev_link.link_status = link->link_status;\n \teth_dev->data->dev_link.link_autoneg = link->link_autoneg;\n \n-\tPMD_DRV_LOG(INFO, \"calling bnxt_print_link_info\\n\");\n \tbnxt_print_link_info(eth_dev);\n \n \tPMD_DRV_LOG(INFO,\n@@ -821,7 +820,7 @@ int bnxt_rep_stop_all(struct bnxt *bp)\n \tif (!bp->rep_info)\n \t\treturn 0;\n \n-\tfor (vf_id = 0; vf_id < BNXT_MAX_VF_REPS; vf_id++) {\n+\tfor (vf_id = 0; vf_id < BNXT_MAX_VF_REPS(bp); vf_id++) {\n \t\trep_eth_dev = bp->rep_info[vf_id].vfr_eth_dev;\n \t\tif (!rep_eth_dev)\n \t\t\tcontinue;\ndiff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp.h b/drivers/net/bnxt/tf_ulp/bnxt_ulp.h\nindex 68f1470c61..960a5a0c93 100644\n--- a/drivers/net/bnxt/tf_ulp/bnxt_ulp.h\n+++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp.h\n@@ -262,7 +262,7 @@ bnxt_ulp_cntxt_ptr2_ulp_flags_get(struct bnxt_ulp_context *ulp_ctx,\n \t\t\t\t uint32_t *flags);\n \n int32_t\n-bnxt_ulp_get_df_rule_info(uint8_t port_id, struct bnxt_ulp_context *ulp_ctx,\n+bnxt_ulp_get_df_rule_info(uint16_t port_id, struct bnxt_ulp_context *ulp_ctx,\n \t\t\t struct bnxt_ulp_df_rule_info *info);\n \n struct bnxt_ulp_vfr_rule_info*\ndiff --git a/drivers/net/bnxt/tf_ulp/generic_templates/ulp_template_db_tbl.c b/drivers/net/bnxt/tf_ulp/generic_templates/ulp_template_db_tbl.c\nindex 5383e2cd70..e43e341927 100644\n--- a/drivers/net/bnxt/tf_ulp/generic_templates/ulp_template_db_tbl.c\n+++ b/drivers/net/bnxt/tf_ulp/generic_templates/ulp_template_db_tbl.c\n@@ -3,7 +3,7 @@\n * All rights reserved.\n */\n \n-/* date: Thu Sep 9 11:11:05 2021 */\n+/* date: Thu Sep 16 11:49:55 2021 */\n \n #include \"ulp_template_db_enum.h\"\n #include \"ulp_template_db_field.h\"\n@@ -164,12 +164,12 @@ struct bnxt_ulp_generic_tbl_params ulp_generic_tbl_params[] = {\n \t},\n \t[BNXT_ULP_RESOURCE_SUB_TYPE_GENERIC_TABLE_VXLAN_ENCAP_REC_CACHE << 1 |\n \t\tBNXT_ULP_DIRECTION_EGRESS] = {\n-\t.name = \"INGRESS GEN_TABLE_VXLAN_ENCAP_REC_CACHE\",\n-\t.result_num_entries = 256,\n+\t.name = \"EGRESS GEN_TABLE_VXLAN_ENCAP_REC_CACHE\",\n+\t.result_num_entries = 4096,\n \t.result_num_bytes = 6,\n \t.key_num_bytes = 17,\n \t.num_buckets = 8,\n-\t.hash_tbl_entries = 1024,\n+\t.hash_tbl_entries = 16384,\n \t.result_byte_order = BNXT_ULP_BYTE_ORDER_LE\n \t}\n };\ndiff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c\nindex d8336d164e..01233c0f5e 100644\n--- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c\n+++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c\n@@ -441,7 +441,7 @@ void\n bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)\n {\n \tstruct bnxt_ulp_df_rule_info *info;\n-\tuint8_t port_id;\n+\tuint16_t port_id;\n \n \tif (!BNXT_TRUFLOW_EN(bp) ||\n \t BNXT_ETH_DEV_IS_REPRESENTOR(bp->eth_dev))\n@@ -505,7 +505,7 @@ int32_t\n bnxt_ulp_create_df_rules(struct bnxt *bp)\n {\n \tstruct bnxt_ulp_df_rule_info *info;\n-\tuint8_t port_id;\n+\tuint16_t port_id;\n \tint rc = 0;\n \n \tif (!BNXT_TRUFLOW_EN(bp) ||\n@@ -562,7 +562,7 @@ bnxt_ulp_create_vfr_default_rules(struct rte_eth_dev *vfr_ethdev)\n \tstruct rte_eth_dev *parent_dev = vfr->parent_dev;\n \tstruct bnxt *bp = parent_dev->data->dev_private;\n \tuint16_t vfr_port_id = vfr_ethdev->data->port_id;\n-\tuint8_t port_id;\n+\tuint16_t port_id;\n \tint rc;\n \n \tif (!bp || !BNXT_TRUFLOW_EN(bp))\n", "prefixes": [ "v3", "09/20" ] }{ "id": 103408, "url": "