Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/80096/?format=api
{ "id": 80096, "url": "http://patchwork.dpdk.org/api/patches/80096/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20201009034832.10302-4-kalesh-anakkur.purayil@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": "<20201009034832.10302-4-kalesh-anakkur.purayil@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20201009034832.10302-4-kalesh-anakkur.purayil@broadcom.com", "date": "2020-10-09T03:48:32", "name": "[v6,3/3] app/testpmd: handle device recovery event", "commit_ref": null, "pull_url": null, "state": "changes-requested", "archived": true, "hash": "b0472a2313127a3d32d2c9be29dfa3f4bc346ec6", "submitter": { "id": 1479, "url": "http://patchwork.dpdk.org/api/people/1479/?format=api", "name": "Kalesh A P", "email": "kalesh-anakkur.purayil@broadcom.com" }, "delegate": { "id": 319, "url": "http://patchwork.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20201009034832.10302-4-kalesh-anakkur.purayil@broadcom.com/mbox/", "series": [ { "id": 12803, "url": "http://patchwork.dpdk.org/api/series/12803/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=12803", "date": "2020-10-09T03:48:29", "name": "librte_ethdev: error recovery support", "version": 6, "mbox": "http://patchwork.dpdk.org/series/12803/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/80096/comments/", "check": "fail", "checks": "http://patchwork.dpdk.org/api/patches/80096/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 61583A04BC;\n\tFri, 9 Oct 2020 05:34:56 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id C45C41BEEA;\n\tFri, 9 Oct 2020 05:34:09 +0200 (CEST)", "from relay.smtp-ext.broadcom.com (lpdvacalvio01.broadcom.com\n [192.19.229.182]) by dpdk.org (Postfix) with ESMTP id 5ED7C1BEDD\n for <dev@dpdk.org>; Fri, 9 Oct 2020 05:34:06 +0200 (CEST)", "from dhcp-10-123-153-22.dhcp.broadcom.net\n (bgccx-dev-host-lnx2.bec.broadcom.net [10.123.153.22])\n by relay.smtp-ext.broadcom.com (Postfix) with ESMTP id 21A517E081\n for <dev@dpdk.org>; Thu, 8 Oct 2020 20:34:03 -0700 (PDT)" ], "DKIM-Filter": "OpenDKIM Filter v2.11.0 relay.smtp-ext.broadcom.com 21A517E081", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n s=dkimrelay; t=1602214444;\n bh=7Zls0PYL+mV5puk6MnoVJIQ1OFcOJFAmZBwOA11gn5k=;\n h=From:To:Subject:Date:In-Reply-To:References:From;\n b=bcY+fM9kMWx3R2ovLQqSFgVN90b/XdqOlJH6KL2aTQpwkIrfCrTJ+XoVbRDEAOwrs\n sJLZCrphzZUceaqT+uUIRkZuG1L+DJ2lNORy4ODLoT3udxq2JYN9sxiVVjcz22C1Ij\n EKI9zr1PF1m/y74EZ5rAxbjOhKNPvJ3SUkFQU29U=", "From": "Kalesh A P <kalesh-anakkur.purayil@broadcom.com>", "To": "dev@dpdk.org", "Date": "Fri, 9 Oct 2020 09:18:32 +0530", "Message-Id": "<20201009034832.10302-4-kalesh-anakkur.purayil@broadcom.com>", "X-Mailer": "git-send-email 2.10.1", "In-Reply-To": "<20201009034832.10302-1-kalesh-anakkur.purayil@broadcom.com>", "References": "<20200122101654.20824-1-kalesh-anakkur.purayil@broadcom.com>\n <20201009034832.10302-1-kalesh-anakkur.purayil@broadcom.com>", "Subject": "[dpdk-dev] [PATCH v6 3/3] app/testpmd: handle device recovery event", "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: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>\n\nAdded code to handle error recovery events in testpmd.\nThis is an indication from the PMD that it is undergoing\nan error recovery and recovered from the error condition.\n\nUpdated 20.11 release notes as well.\n\nSigned-off-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>\nSigned-off-by: Somnath Kotur <somnath.kotur@broadcom.com>\nReviewed-by: Ajit Kumar Khaparde <ajit.khaparde@broadcom.com>\n---\n app/test-pmd/parameters.c | 8 ++++++--\n app/test-pmd/testpmd.c | 6 +++++-\n doc/guides/rel_notes/release_20_11.rst | 10 ++++++++++\n 3 files changed, 21 insertions(+), 3 deletions(-)", "diff": "diff --git a/app/test-pmd/parameters.c b/app/test-pmd/parameters.c\nindex 1ead595..560f9ba 100644\n--- a/app/test-pmd/parameters.c\n+++ b/app/test-pmd/parameters.c\n@@ -192,9 +192,9 @@ usage(char* progname)\n \tprintf(\" --no-rmv-interrupt: disable device removal interrupt.\\n\");\n \tprintf(\" --bitrate-stats=N: set the logical core N to perform \"\n \t\t\"bit-rate calculation.\\n\");\n-\tprintf(\" --print-event <unknown|intr_lsc|queue_state|intr_reset|vf_mbox|macsec|intr_rmv|flow_aged|all>: \"\n+\tprintf(\" --print-event <unknown|intr_lsc|queue_state|intr_reset|vf_mbox|macsec|intr_rmv|flow_aged|err_recovering|recovered|all>: \"\n \t \"enable print of designated event or all of them.\\n\");\n-\tprintf(\" --mask-event <unknown|intr_lsc|queue_state|intr_reset|vf_mbox|macsec|intr_rmv|flow_aged|all>: \"\n+\tprintf(\" --mask-event <unknown|intr_lsc|queue_state|intr_reset|vf_mbox|macsec|intr_rmv|flow_aged|err_recovering|recovered|all>: \"\n \t \"disable print of designated event or all of them.\\n\");\n \tprintf(\" --flow-isolate-all: \"\n \t \"requests flow API isolated mode on all ports at initialization time.\\n\");\n@@ -556,6 +556,10 @@ parse_event_printing_config(const char *optarg, int enable)\n \t\tmask = UINT32_C(1) << RTE_ETH_EVENT_DESTROY;\n \telse if (!strcmp(optarg, \"flow_aged\"))\n \t\tmask = UINT32_C(1) << RTE_ETH_EVENT_FLOW_AGED;\n+\telse if (!strcmp(optarg, \"err_recovering\"))\n+\t\tmask = UINT32_C(1) << RTE_ETH_EVENT_ERR_RECOVERING;\n+\telse if (!strcmp(optarg, \"recovered\"))\n+\t\tmask = UINT32_C(1) << RTE_ETH_EVENT_RECOVERED;\n \telse if (!strcmp(optarg, \"all\"))\n \t\tmask = ~UINT32_C(0);\n \telse {\ndiff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c\nindex fe6450c..80ae3fa 100644\n--- a/app/test-pmd/testpmd.c\n+++ b/app/test-pmd/testpmd.c\n@@ -380,6 +380,8 @@ static const char * const eth_event_desc[] = {\n \t[RTE_ETH_EVENT_NEW] = \"device probed\",\n \t[RTE_ETH_EVENT_DESTROY] = \"device released\",\n \t[RTE_ETH_EVENT_FLOW_AGED] = \"flow aged\",\n+\t[RTE_ETH_EVENT_ERR_RECOVERING] = \"device error under recovery\",\n+\t[RTE_ETH_EVENT_RECOVERED] = \"device recovered\",\n \t[RTE_ETH_EVENT_MAX] = NULL,\n };\n \n@@ -394,7 +396,9 @@ uint32_t event_print_mask = (UINT32_C(1) << RTE_ETH_EVENT_UNKNOWN) |\n \t\t\t (UINT32_C(1) << RTE_ETH_EVENT_IPSEC) |\n \t\t\t (UINT32_C(1) << RTE_ETH_EVENT_MACSEC) |\n \t\t\t (UINT32_C(1) << RTE_ETH_EVENT_INTR_RMV) |\n-\t\t\t (UINT32_C(1) << RTE_ETH_EVENT_FLOW_AGED);\n+\t\t\t (UINT32_C(1) << RTE_ETH_EVENT_FLOW_AGED) |\n+\t\t\t (UINT32_C(1) << RTE_ETH_EVENT_ERR_RECOVERING) |\n+\t\t\t (UINT32_C(1) << RTE_ETH_EVENT_RECOVERED);\n /*\n * Decide if all memory are locked for performance.\n */\ndiff --git a/doc/guides/rel_notes/release_20_11.rst b/doc/guides/rel_notes/release_20_11.rst\nindex 4bcf220..f732ff6 100644\n--- a/doc/guides/rel_notes/release_20_11.rst\n+++ b/doc/guides/rel_notes/release_20_11.rst\n@@ -78,11 +78,21 @@ New Features\n ``--portmask=N``\n where N represents the hexadecimal bitmask of ports used.\n \n+* **Added error recovery support.**\n+\n+ Added error recovery support to detect and recover from device errors including:\n+\n+ * Added new event: ``RTE_ETH_EVENT_ERR_RECOVERING`` for the driver to report\n+ that the port is recovering from an error.\n+ * Added new event: ``RTE_ETH_EVENT_RECOVERED`` for the driver to report\n+ that the port has recovered from an error.\n+\n * **Updated Broadcom bnxt driver.**\n \n Updated the Broadcom bnxt driver with new features and improvements, including:\n \n * Added support for 200G PAM4 link speed.\n+ * Added support to handle device recovery events.\n \n \n Removed Items\n", "prefixes": [ "v6", "3/3" ] }