Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/41864/?format=api
http://patches.dpdk.org/api/patches/41864/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20180628201549.3507-2-ajit.khaparde@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": "<20180628201549.3507-2-ajit.khaparde@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20180628201549.3507-2-ajit.khaparde@broadcom.com", "date": "2018-06-28T20:15:27", "name": "[v2,01/23] net/bnxt: fix clear port stats", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "55e8609275bc00882955157171507d3c6cf7fb06", "submitter": { "id": 501, "url": "http://patches.dpdk.org/api/people/501/?format=api", "name": "Ajit Khaparde", "email": "ajit.khaparde@broadcom.com" }, "delegate": { "id": 319, "url": "http://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20180628201549.3507-2-ajit.khaparde@broadcom.com/mbox/", "series": [ { "id": 300, "url": "http://patches.dpdk.org/api/series/300/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=300", "date": "2018-06-28T20:15:29", "name": "bnxt patchset", "version": 2, "mbox": "http://patches.dpdk.org/series/300/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/41864/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/41864/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id DD0C61B50F;\n\tThu, 28 Jun 2018 22:16:04 +0200 (CEST)", "from rnd-relay.smtp.broadcom.com (rnd-relay.smtp.broadcom.com\n\t[192.19.229.170]) by dpdk.org (Postfix) with ESMTP id 76BCC1B4FE;\n\tThu, 28 Jun 2018 22:15:57 +0200 (CEST)", "from nis-sj1-27.broadcom.com (nis-sj1-27.lvn.broadcom.net\n\t[10.75.144.136])\n\tby rnd-relay.smtp.broadcom.com (Postfix) with ESMTP id F05AA30C01B;\n\tThu, 28 Jun 2018 13:15:52 -0700 (PDT)", "from C02VPB22HTD6.dhcp.broadcom.net (c02vpb22htd6.dhcp.broadcom.net\n\t[10.136.50.120])\n\tby nis-sj1-27.broadcom.com (Postfix) with ESMTP id 6ED0CAC078F;\n\tThu, 28 Jun 2018 13:15:52 -0700 (PDT)" ], "DKIM-Filter": "OpenDKIM Filter v2.10.3 rnd-relay.smtp.broadcom.com F05AA30C01B", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n\ts=dkimrelay; t=1530216953;\n\tbh=0ZXfCu0u00vfAN/qin8qEMIhfh5nHce4OOANyyy8c9w=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=EH9NVxyMx1eqXrolmmaQ182m3ZCabVgcFH4JEW9W88yen2oExj9N2uK3aHRczq9or\n\tQozfNLc5HQB7feauPOHZMS0dVZoqwCFM6moS+Y6Du+itTPw4l1Oy/wQry9aMkFbLB1\n\tiqpd6lg8Lr1rbYNUvvouGhVdXmvGLdJijtm2nUE8=", "From": "Ajit Khaparde <ajit.khaparde@broadcom.com>", "To": "dev@dpdk.org", "Cc": "ferruh.yigit@intel.com,\n\tstable@dpdk.org", "Date": "Thu, 28 Jun 2018 13:15:27 -0700", "Message-Id": "<20180628201549.3507-2-ajit.khaparde@broadcom.com>", "X-Mailer": "git-send-email 2.15.2 (Apple Git-101.1)", "In-Reply-To": "<20180628201549.3507-1-ajit.khaparde@broadcom.com>", "References": "<f02289d7-4111-ab00-9ce9-00dd5c8b3c18@intel.com>\n\t<20180628201549.3507-1-ajit.khaparde@broadcom.com>", "Subject": "[dpdk-dev] [PATCH v2 01/23] net/bnxt: fix clear port stats", "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\t<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\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "PORT_CLR_STATS is not allowed for VFs, NPAR, MultiHost functions\nor when SR-IOV is enabled.\nDon't send the HWRM command in such cases.\n\nFixes: bfb9c2260be2 (\"net/bnxt: support xstats get/reset\")\nCc: stable@dpdk.org\n\nSigned-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>\n--\nv1->v2: Fix a checkpatch warning.\n---\n drivers/net/bnxt/bnxt.h | 4 ++++\n drivers/net/bnxt/bnxt_hwrm.c | 5 ++++-\n 2 files changed, 8 insertions(+), 1 deletion(-)", "diff": "diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h\nindex afaaf8c41..d19aea569 100644\n--- a/drivers/net/bnxt/bnxt.h\n+++ b/drivers/net/bnxt/bnxt.h\n@@ -98,6 +98,7 @@ struct bnxt_child_vf_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_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)\n #define BNXT_PF_RINGS_AVAIL(bp)\t(bp->pf.max_cp_rings - BNXT_PF_RINGS_USED(bp))\n@@ -105,6 +106,9 @@ struct bnxt_pf_info {\n \tuint16_t\t\tfirst_vf_id;\n \tuint16_t\t\tactive_vfs;\n \tuint16_t\t\tmax_vfs;\n+\tuint16_t\t\ttotal_vfs; /* Total VFs possible.\n+\t\t\t\t\t * Not necessarily enabled.\n+\t\t\t\t\t */\n \tuint32_t\t\tfunc_cfg_flags;\n \tvoid\t\t\t*vf_req_buf;\n \trte_iova_t\t\tvf_req_buf_dma_addr;\ndiff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c\nindex d6fdc1b88..f441d4610 100644\n--- a/drivers/net/bnxt/bnxt_hwrm.c\n+++ b/drivers/net/bnxt/bnxt_hwrm.c\n@@ -506,6 +506,7 @@ static int __bnxt_hwrm_func_qcaps(struct bnxt *bp)\n \tif (BNXT_PF(bp)) {\n \t\tbp->pf.port_id = resp->port_id;\n \t\tbp->pf.first_vf_id = rte_le_to_cpu_16(resp->first_vf_id);\n+\t\tbp->pf.total_vfs = rte_le_to_cpu_16(resp->max_vfs);\n \t\tnew_max_vfs = bp->pdev->max_vfs;\n \t\tif (new_max_vfs != bp->pf.max_vfs) {\n \t\t\tif (bp->pf.vf_info)\n@@ -3151,7 +3152,9 @@ int bnxt_hwrm_port_clr_stats(struct bnxt *bp)\n \tstruct bnxt_pf_info *pf = &bp->pf;\n \tint rc;\n \n-\tif (!(bp->flags & BNXT_FLAG_PORT_STATS))\n+\t/* Not allowed on NS2 device, NPAR, MultiHost, VF */\n+\tif (!(bp->flags & BNXT_FLAG_PORT_STATS) || BNXT_VF(bp) ||\n+\t BNXT_NPAR(bp) || BNXT_MH(bp) || BNXT_TOTAL_VFS(bp))\n \t\treturn 0;\n \n \tHWRM_PREP(req, PORT_CLR_STATS);\n", "prefixes": [ "v2", "01/23" ] }{ "id": 41864, "url": "