Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/106662/?format=api
{ "id": 106662, "url": "http://patches.dpdk.org/api/patches/106662/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20220128124830.427-3-kalesh-anakkur.purayil@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": "<20220128124830.427-3-kalesh-anakkur.purayil@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20220128124830.427-3-kalesh-anakkur.purayil@broadcom.com", "date": "2022-01-28T12:48:28", "name": "[v7,2/4] app/testpmd: handle device recovery event", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "9b00abfb65b4156ae0e35f8768790a7f48c3fbad", "submitter": { "id": 1479, "url": "http://patches.dpdk.org/api/people/1479/?format=api", "name": "Kalesh A P", "email": "kalesh-anakkur.purayil@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/20220128124830.427-3-kalesh-anakkur.purayil@broadcom.com/mbox/", "series": [ { "id": 21415, "url": "http://patches.dpdk.org/api/series/21415/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=21415", "date": "2022-01-28T12:48:26", "name": "ethdev: error recovery support", "version": 7, "mbox": "http://patches.dpdk.org/series/21415/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/106662/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/106662/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 BF184A034D;\n\tFri, 28 Jan 2022 13:21:22 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id CBF714289F;\n\tFri, 28 Jan 2022 13:21:13 +0100 (CET)", "from relay.smtp-ext.broadcom.com (relay.smtp-ext.broadcom.com\n [192.19.11.229]) by mails.dpdk.org (Postfix) with ESMTP id C223542890\n for <dev@dpdk.org>; Fri, 28 Jan 2022 13:21:12 +0100 (CET)", "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 1EDC6C0000EC;\n Fri, 28 Jan 2022 04:21:10 -0800 (PST)" ], "DKIM-Filter": "OpenDKIM Filter v2.11.0 relay.smtp-ext.broadcom.com 1EDC6C0000EC", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n s=dkimrelay; t=1643372472;\n bh=cABqB8X2S2RGDVsg8wc56XfAIbyi35KkguJ99abzivQ=;\n h=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n b=YfcwHNVb9e0yssLexGrIeihkvMTskxvpmj1y4+GuFubfJGHWKN4yrzC2LUISt9xsk\n w445bQmBSflwTeJKvwT5envTI36tG709GIw0ya8/P8o450ecXVfTIrdOFoiXREtcd7\n hOHg5HbQb3PBUzlxw8FsKLdf/EK1kURlUqVxUvho=", "From": "Kalesh A P <kalesh-anakkur.purayil@broadcom.com>", "To": "dev@dpdk.org", "Cc": "ferruh.yigit@intel.com,\n\tajit.khaparde@broadcom.com,\n\tasafp@nvidia.com", "Subject": "[dpdk-dev] [PATCH v7 2/4] app/testpmd: handle device recovery event", "Date": "Fri, 28 Jan 2022 18:18:28 +0530", "Message-Id": "<20220128124830.427-3-kalesh-anakkur.purayil@broadcom.com>", "X-Mailer": "git-send-email 2.10.1", "In-Reply-To": "<20220128124830.427-1-kalesh-anakkur.purayil@broadcom.com>", "References": "<20201009034832.10302-1-kalesh-anakkur.purayil@broadcom.com>\n <20220128124830.427-1-kalesh-anakkur.purayil@broadcom.com>", "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" }, "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\nSigned-off-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>\nSigned-off-by: Somnath Kotur <somnath.kotur@broadcom.com>\nReviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>\n---\n app/test-pmd/parameters.c | 8 ++++++--\n app/test-pmd/testpmd.c | 10 +++++++++-\n 2 files changed, 15 insertions(+), 3 deletions(-)", "diff": "diff --git a/app/test-pmd/parameters.c b/app/test-pmd/parameters.c\nindex daf6a31..deea29f 100644\n--- a/app/test-pmd/parameters.c\n+++ b/app/test-pmd/parameters.c\n@@ -175,9 +175,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@@ -461,6 +461,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 e1da961..7b64751 100644\n--- a/app/test-pmd/testpmd.c\n+++ b/app/test-pmd/testpmd.c\n@@ -417,6 +417,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, recovery in progress\",\n+\t[RTE_ETH_EVENT_RECOVERED] = \"device recovered\",\n \t[RTE_ETH_EVENT_MAX] = NULL,\n };\n \n@@ -431,7 +433,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 */\n@@ -3564,6 +3568,10 @@ eth_event_callback(portid_t port_id, enum rte_eth_event_type type, void *param,\n \t\tports[port_id].port_status = RTE_PORT_CLOSED;\n \t\tprintf(\"Port %u is closed\\n\", port_id);\n \t\tbreak;\n+\tcase RTE_ETH_EVENT_RECOVERED:\n+\t\t/* for now, flush flows to avoid displaying stale entries */\n+\t\tport_flow_flush(port_id);\n+\t\tbreak;\n \tdefault:\n \t\tbreak;\n \t}\n", "prefixes": [ "v7", "2/4" ] }