Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/81189/?format=api
{ "id": 81189, "url": "http://patchwork.dpdk.org/api/patches/81189/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/1602916089-18576-6-git-send-email-venkatkumar.duvvuru@broadcom.com/", "project": { "id": 1, "url": "http://patchwork.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": "<1602916089-18576-6-git-send-email-venkatkumar.duvvuru@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1602916089-18576-6-git-send-email-venkatkumar.duvvuru@broadcom.com", "date": "2020-10-17T06:28:00", "name": "[05/14] net/bnxt: table scope to PF Mapping for SR and Wh+", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "91d9f13082439885d1eb70f8f8b675d6f45e8f80", "submitter": { "id": 1635, "url": "http://patchwork.dpdk.org/api/people/1635/?format=api", "name": "Venkat Duvvuru", "email": "venkatkumar.duvvuru@broadcom.com" }, "delegate": { "id": 1766, "url": "http://patchwork.dpdk.org/api/users/1766/?format=api", "username": "ajitkhaparde", "first_name": "Ajit", "last_name": "Khaparde", "email": "ajit.khaparde@broadcom.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/1602916089-18576-6-git-send-email-venkatkumar.duvvuru@broadcom.com/mbox/", "series": [ { "id": 13073, "url": "http://patchwork.dpdk.org/api/series/13073/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=13073", "date": "2020-10-17T06:27:55", "name": "bnxt patches", "version": 1, "mbox": "http://patchwork.dpdk.org/series/13073/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/81189/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/81189/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 dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 2B7C8A04DB;\n\tSat, 17 Oct 2020 08:30:59 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 13FA0E31D;\n\tSat, 17 Oct 2020 08:28:29 +0200 (CEST)", "from relay.smtp-ext.broadcom.com (unknown [192.19.221.30])\n by dpdk.org (Postfix) with ESMTP id 61A6EE26B\n for <dev@dpdk.org>; Sat, 17 Oct 2020 08:28:15 +0200 (CEST)", "from S60.dhcp.broadcom.net (unknown [10.123.66.170])\n (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits))\n (No client certificate requested)\n by relay.smtp-ext.broadcom.com (Postfix) with ESMTPS id C8D9C24751;\n Fri, 16 Oct 2020 23:28:12 -0700 (PDT)" ], "DKIM-Filter": "OpenDKIM Filter v2.11.0 relay.smtp-ext.broadcom.com C8D9C24751", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n s=dkimrelay; t=1602916092;\n bh=iJ2/K1AsfJ3HRRu6c6kau7YN2zk2+p42ILOSh4YoGEU=;\n h=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n b=sPGwt9rLNg/q9siMtw71maTzCxEpehIq3zt+BPmt+VdBXetXJn49rZcym2BH0as0r\n cBu9ztiUWlh8WOW8yLACjCObzCB/ob4JX4+/AdC5jXijooJNc6tKMXWUg7h6FWh5rR\n 6RdeKSMCVv5jPkO2fuiUuUXGWoum2ZtFsy20g+6o=", "From": "Venkat Duvvuru <venkatkumar.duvvuru@broadcom.com>", "To": "dev@dpdk.org", "Cc": "Farah Smith <farah.smith@broadcom.com>", "Date": "Sat, 17 Oct 2020 11:58:00 +0530", "Message-Id": "\n <1602916089-18576-6-git-send-email-venkatkumar.duvvuru@broadcom.com>", "X-Mailer": "git-send-email 2.7.4", "In-Reply-To": "\n <1602916089-18576-1-git-send-email-venkatkumar.duvvuru@broadcom.com>", "References": "\n <1602916089-18576-1-git-send-email-venkatkumar.duvvuru@broadcom.com>", "Subject": "[dpdk-dev] [PATCH 05/14] net/bnxt: table scope to PF Mapping for SR\n\tand Wh+", "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>,\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: Farah Smith <farah.smith@broadcom.com>\n\nLegacy devices require PF set of base addresses for EEM operation.\nA table scope id is a logical construct and is mapped to the PF\nassociated with the communications channel used.\nIn the case of a VF, the parent PF is used.\n\nSigned-off-by: Farah Smith <farah.smith@broadcom.com>\nReviewed-by: Randy Schacher <stuart.schacher@broadcom.com>\n---\n drivers/net/bnxt/tf_core/tf_device.h | 10 +++++++---\n drivers/net/bnxt/tf_core/tf_device_p4.c | 10 +++++-----\n drivers/net/bnxt/tf_core/tf_em_common.c | 6 ++++--\n drivers/net/bnxt/tf_core/tf_em_host.c | 8 ++++++++\n drivers/net/bnxt/tf_core/tf_tbl.h | 3 +++\n drivers/net/bnxt/tf_core/tfp.c | 19 +++++++++++++++++++\n drivers/net/bnxt/tf_core/tfp.h | 16 ++++++++++++++++\n 7 files changed, 62 insertions(+), 10 deletions(-)", "diff": "diff --git a/drivers/net/bnxt/tf_core/tf_device.h b/drivers/net/bnxt/tf_core/tf_device.h\nindex cf7c36e..b5fc695 100644\n--- a/drivers/net/bnxt/tf_core/tf_device.h\n+++ b/drivers/net/bnxt/tf_core/tf_device.h\n@@ -579,8 +579,11 @@ struct tf_dev_ops {\n \t * [in] tfp\n \t * Pointer to TF handle\n \t *\n-\t * [in] parms\n-\t * Pointer to table scope map parameters\n+\t * [in] pf\n+\t * PF associated with the table scope\n+\t *\n+\t * [in] parif_bitmask\n+\t * Bitmask of PARIFs to enable\n \t *\n \t * [in/out] pointer to the parif_2_pf data to be updated\n \t *\n@@ -593,7 +596,8 @@ struct tf_dev_ops {\n \t * -EINVAL - Error\n \t */\n \tint (*tf_dev_map_parif)(struct tf *tfp,\n-\t\t\t\tstruct tf_map_tbl_scope_parms *parms,\n+\t\t\t\tuint16_t parif_bitmask,\n+\t\t\t\tuint16_t pf,\n \t\t\t\tuint8_t *data,\n \t\t\t\tuint8_t *mask,\n \t\t\t\tuint16_t sz_in_bytes);\ndiff --git a/drivers/net/bnxt/tf_core/tf_device_p4.c b/drivers/net/bnxt/tf_core/tf_device_p4.c\nindex 07c8d02..b35e65a 100644\n--- a/drivers/net/bnxt/tf_core/tf_device_p4.c\n+++ b/drivers/net/bnxt/tf_core/tf_device_p4.c\n@@ -103,7 +103,8 @@ tf_dev_p4_get_tcam_slice_info(struct tf *tfp __rte_unused,\n \n static int\n tf_dev_p4_map_parif(struct tf *tfp __rte_unused,\n-\t\t struct tf_map_tbl_scope_parms *parms,\n+\t\t uint16_t parif_bitmask,\n+\t\t uint16_t pf,\n \t\t uint8_t *data,\n \t\t uint8_t *mask,\n \t\t uint16_t sz_in_bytes)\n@@ -112,21 +113,20 @@ tf_dev_p4_map_parif(struct tf *tfp __rte_unused,\n \tuint32_t parif_pf_mask[2] = { 0 };\n \tuint32_t parif;\n \tuint32_t shift;\n-\tuint32_t scope_id = (uint32_t)(parms->tbl_scope_id);\n \n \tif (sz_in_bytes != sizeof(uint64_t))\n \t\treturn -ENOTSUP;\n \n \tfor (parif = 0; parif < TF_DEV_P4_PARIF_MAX; parif++) {\n-\t\tif (parms->parif_bitmask & (1UL << parif)) {\n+\t\tif (parif_bitmask & (1UL << parif)) {\n \t\t\tif (parif < 8) {\n \t\t\t\tshift = 4 * parif;\n \t\t\t\tparif_pf_mask[0] |= TF_DEV_P4_PF_MASK << shift;\n-\t\t\t\tparif_pf[0] |= scope_id << shift;\n+\t\t\t\tparif_pf[0] |= pf << shift;\n \t\t\t} else {\n \t\t\t\tshift = 4 * (parif - 8);\n \t\t\t\tparif_pf_mask[1] |= TF_DEV_P4_PF_MASK << shift;\n-\t\t\t\tparif_pf[1] |= scope_id << shift;\n+\t\t\t\tparif_pf[1] |= pf << shift;\n \t\t\t}\n \t\t}\n \t}\ndiff --git a/drivers/net/bnxt/tf_core/tf_em_common.c b/drivers/net/bnxt/tf_core/tf_em_common.c\nindex c4f3de0..c975bca 100644\n--- a/drivers/net/bnxt/tf_core/tf_em_common.c\n+++ b/drivers/net/bnxt/tf_core/tf_em_common.c\n@@ -1104,8 +1104,10 @@ int tf_em_ext_map_tbl_scope(struct tf *tfp,\n \t}\n \tmask = aparms.mem_va;\n \n-\trc = dev->ops->tf_dev_map_parif(tfp, parms, (uint8_t *)data,\n-\t\t\t\t\t(uint8_t *)mask, sz_in_bytes);\n+\trc = dev->ops->tf_dev_map_parif(tfp, parms->parif_bitmask,\n+\t\t\t\t\ttbl_scope_cb->pf,\n+\t\t\t\t\t(uint8_t *)data, (uint8_t *)mask,\n+\t\t\t\t\tsz_in_bytes);\n \n \tif (rc) {\n \t\tTFP_DRV_LOG(ERR,\ndiff --git a/drivers/net/bnxt/tf_core/tf_em_host.c b/drivers/net/bnxt/tf_core/tf_em_host.c\nindex b5db94f..a106bdf 100644\n--- a/drivers/net/bnxt/tf_core/tf_em_host.c\n+++ b/drivers/net/bnxt/tf_core/tf_em_host.c\n@@ -392,6 +392,14 @@ tf_em_ext_alloc(struct tf *tfp, struct tf_alloc_tbl_scope_parms *parms)\n \ttbl_scope_cb->index = parms->tbl_scope_id;\n \ttbl_scope_cb->tbl_scope_id = parms->tbl_scope_id;\n \n+\trc = tfp_get_pf(tfp, &tbl_scope_cb->pf);\n+\tif (rc) {\n+\t\tTFP_DRV_LOG(ERR,\n+\t\t\t \"EEM: PF query error rc:%s\\n\",\n+\t\t\t strerror(-rc));\n+\t\tgoto cleanup;\n+\t}\n+\n \tfor (dir = 0; dir < TF_DIR_MAX; dir++) {\n \t\trc = tf_msg_em_qcaps(tfp,\n \t\t\t\t dir,\ndiff --git a/drivers/net/bnxt/tf_core/tf_tbl.h b/drivers/net/bnxt/tf_core/tf_tbl.h\nindex 2a5d24c..230338c 100644\n--- a/drivers/net/bnxt/tf_core/tf_tbl.h\n+++ b/drivers/net/bnxt/tf_core/tf_tbl.h\n@@ -38,6 +38,9 @@ struct tf_em_caps {\n */\n struct tf_tbl_scope_cb {\n \tuint32_t tbl_scope_id;\n+ /** The pf or parent pf of the vf used for table scope creation\n+\t*/\n+\tuint16_t pf;\n \tint index;\n \tstruct hcapi_cfa_em_ctx_mem_info em_ctx_info[TF_DIR_MAX];\n \tstruct tf_em_caps em_caps[TF_DIR_MAX];\ndiff --git a/drivers/net/bnxt/tf_core/tfp.c b/drivers/net/bnxt/tf_core/tfp.c\nindex 426a182..0f6d63c 100644\n--- a/drivers/net/bnxt/tf_core/tfp.c\n+++ b/drivers/net/bnxt/tf_core/tfp.c\n@@ -178,3 +178,22 @@ tfp_get_fid(struct tf *tfp, uint16_t *fw_fid)\n \n \treturn 0;\n }\n+\n+int\n+tfp_get_pf(struct tf *tfp, uint16_t *pf)\n+{\n+\tstruct bnxt *bp = NULL;\n+\n+\tif (tfp == NULL || pf == NULL)\n+\t\treturn -EINVAL;\n+\n+\tbp = container_of(tfp, struct bnxt, tfp);\n+\tif (BNXT_VF(bp) && bp->parent) {\n+\t\t*pf = bp->parent->fid - 1;\n+\t\treturn 0;\n+\t} else if (BNXT_PF(bp)) {\n+\t\t*pf = bp->fw_fid - 1;\n+\t\treturn 0;\n+\t}\n+\treturn -EINVAL;\n+}\ndiff --git a/drivers/net/bnxt/tf_core/tfp.h b/drivers/net/bnxt/tf_core/tfp.h\nindex 421a7d9..551b9c5 100644\n--- a/drivers/net/bnxt/tf_core/tfp.h\n+++ b/drivers/net/bnxt/tf_core/tfp.h\n@@ -268,4 +268,20 @@ int tfp_get_fid(struct tf *tfp, uint16_t *fw_fid);\n */\n int tfp_get_fid(struct tf *tfp, uint16_t *fw_fid);\n \n+/**\n+ * Get the PF associated with the fw communications channel.\n+ *\n+ * [in] session\n+ * Pointer to session handle\n+ *\n+ * [out] pf\n+ * Pointer to the pf id\n+ *\n+ * Returns:\n+ * 0 - Success\n+ * -EINVAL - Failure\n+ *\n+ */\n+int tfp_get_pf(struct tf *tfp, uint16_t *pf);\n+\n #endif /* _TFP_H_ */\n", "prefixes": [ "05/14" ] }