Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/80094/?format=api
http://patchwork.dpdk.org/api/patches/80094/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20201009034832.10302-2-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-2-kalesh-anakkur.purayil@broadcom.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20201009034832.10302-2-kalesh-anakkur.purayil@broadcom.com", "date": "2020-10-09T03:48:30", "name": "[v6,1/3] ethdev: support device reset and recovery events", "commit_ref": null, "pull_url": null, "state": "changes-requested", "archived": true, "hash": "86913608da9ef71c0f4c64fadc46e3a7dae4f20a", "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-2-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/80094/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/80094/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 CC33AA04BC;\n\tFri, 9 Oct 2020 05:34:22 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 13D2F1BED5;\n\tFri, 9 Oct 2020 05:34:06 +0200 (CEST)", "from relay.smtp-ext.broadcom.com (lpdvacalvio01.broadcom.com\n [192.19.229.182]) by dpdk.org (Postfix) with ESMTP id A5BA71BECA\n for <dev@dpdk.org>; Fri, 9 Oct 2020 05:34:03 +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 5B8547E072\n for <dev@dpdk.org>; Thu, 8 Oct 2020 20:34:01 -0700 (PDT)" ], "DKIM-Filter": "OpenDKIM Filter v2.11.0 relay.smtp-ext.broadcom.com 5B8547E072", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=broadcom.com;\n s=dkimrelay; t=1602214442;\n bh=dLbErOAA6HbQHq/MOt9o4uutyMHZFA2G015hxXE23uk=;\n h=From:To:Subject:Date:In-Reply-To:References:From;\n b=RHGo/JP3mMdx3uCPjhGM6BfTushvVmrQX0banG8fxEbLw8TIj/SaYANaT9jHIZPLH\n Eno5coY084jkR59DwDCm61FJEp7aFKuEu+ALVY+7v8NL1xU7WgplcK7974t6NUYogy\n WlsYJZr37m8vxWkkJF1cdEx1iRUkCO9uFiIgEtgM=", "From": "Kalesh A P <kalesh-anakkur.purayil@broadcom.com>", "To": "dev@dpdk.org", "Date": "Fri, 9 Oct 2020 09:18:30 +0530", "Message-Id": "<20201009034832.10302-2-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 1/3] ethdev: support device reset and recovery\n\tevents", "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\nAdding support for device reset and recovery events in the\nrte_eth_event framework. FW error and FW reset conditions would be\nmanaged internally by PMD without needing application intervention.\nIn such cases, PMD would need reset/recovery events to notify application\nthat PMD is undergoing a reset.\n\nSigned-off-by: Somnath Kotur <somnath.kotur@broadcom.com>\nSigned-off-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>\nReviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>\nReviewed-by: Asaf Penso <asafp@nvidia.com>\n---\n doc/guides/prog_guide/poll_mode_drv.rst | 18 ++++++++++++++++++\n lib/librte_ethdev/rte_ethdev.h | 17 +++++++++++++++++\n 2 files changed, 35 insertions(+)", "diff": "diff --git a/doc/guides/prog_guide/poll_mode_drv.rst b/doc/guides/prog_guide/poll_mode_drv.rst\nindex 86e0a14..c03f0ef 100644\n--- a/doc/guides/prog_guide/poll_mode_drv.rst\n+++ b/doc/guides/prog_guide/poll_mode_drv.rst\n@@ -615,3 +615,21 @@ by application.\n The PMD itself should not call rte_eth_dev_reset(). The PMD can trigger\n the application to handle reset event. It is duty of application to\n handle all synchronization before it calls rte_eth_dev_reset().\n+\n+Error recovery support\n+~~~~~~~~~~~~~~~~~~~~~~\n+\n+When the PMD detects a FW reset or error condition, it will try to recover\n+from the error without needing the application intervention. In such cases,\n+PMD would need events to notify the application that it is undergoing\n+an error recovery.\n+\n+The PMD will trigger RTE_ETH_EVENT_ERR_RECOVERING event to notify the\n+application that PMD detected a FW reset or FW error condition. PMD will\n+try to recover from the error by itself. Data path will be halted and\n+control path operations would fail during the recovery period.\n+\n+The PMD will trigger RTE_ETH_EVENT_RECOVERED event to notify the application\n+that the it has recovered from the error condition. Control path and data path\n+are up now. Since the device undergone a reset, flow rules offloaded prior to\n+the reset will be lost and the application has to recreate the rules again.\ndiff --git a/lib/librte_ethdev/rte_ethdev.h b/lib/librte_ethdev/rte_ethdev.h\nindex 9759f13..9b4b015 100644\n--- a/lib/librte_ethdev/rte_ethdev.h\n+++ b/lib/librte_ethdev/rte_ethdev.h\n@@ -3207,6 +3207,23 @@ enum rte_eth_event_type {\n \tRTE_ETH_EVENT_DESTROY, /**< port is released */\n \tRTE_ETH_EVENT_IPSEC, /**< IPsec offload related event */\n \tRTE_ETH_EVENT_FLOW_AGED,/**< New aged-out flows is detected */\n+\tRTE_ETH_EVENT_ERR_RECOVERING,\n+\t\t\t/**< port recovering from an error\n+\t\t\t *\n+\t\t\t * PMD detected a FW reset or error condition.\n+\t\t\t * PMD will try to recover from the error.\n+\t\t\t * Data path will be halted and Control path operations\n+\t\t\t * would fail at this time.\n+\t\t\t */\n+\tRTE_ETH_EVENT_RECOVERED,\n+\t\t\t/**< port recovered from an error\n+\t\t\t *\n+\t\t\t * PMD has recovered from the error condition.\n+\t\t\t * Control path and Data path are up now.\n+\t\t\t * Since the device undergone a reset, flow rules\n+\t\t\t * offloaded prior to the reset will be lost and\n+\t\t\t * the application has to recreate the rules again.\n+\t\t\t */\n \tRTE_ETH_EVENT_MAX /**< max value of this enum */\n };\n \n", "prefixes": [ "v6", "1/3" ] }{ "id": 80094, "url": "