Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/136253/?format=api
{ "id": 136253, "url": "http://patchwork.dpdk.org/api/patches/136253/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20240201095907.3564286-1-okl-plv@napatech.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": "<20240201095907.3564286-1-okl-plv@napatech.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20240201095907.3564286-1-okl-plv@napatech.com", "date": "2024-02-01T09:59:07", "name": "[v2] app/testpmd: support updating flow rule actions", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "68c152ed4cebca1ae74f086a25b4c323a4475c2f", "submitter": { "id": 3265, "url": "http://patchwork.dpdk.org/api/people/3265/?format=api", "name": "Oleksandr Kolomeiets", "email": "okl-plv@napatech.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/20240201095907.3564286-1-okl-plv@napatech.com/mbox/", "series": [ { "id": 30962, "url": "http://patchwork.dpdk.org/api/series/30962/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=30962", "date": "2024-02-01T09:59:07", "name": "[v2] app/testpmd: support updating flow rule actions", "version": 2, "mbox": "http://patchwork.dpdk.org/series/30962/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/136253/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/136253/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 667D043A35;\n\tThu, 1 Feb 2024 10:59:27 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id E94A040299;\n\tThu, 1 Feb 2024 10:59:26 +0100 (CET)", "from egress-ip42a.ess.de.barracuda.com\n (egress-ip42a.ess.de.barracuda.com [18.185.115.201])\n by mails.dpdk.org (Postfix) with ESMTP id 4E3F540275\n for <dev@dpdk.org>; Thu, 1 Feb 2024 10:59:25 +0100 (CET)", "from EUR04-DB3-obe.outbound.protection.outlook.com\n (mail-db3eur04lp2051.outbound.protection.outlook.com [104.47.12.51]) by\n mx-outbound12-216.eu-central-1a.ess.aws.cudaops.com (version=TLSv1.2\n cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO);\n Thu, 01 Feb 2024 09:59:23 +0000", "from DB8PR03CA0019.eurprd03.prod.outlook.com (2603:10a6:10:be::32)\n by DU0P190MB1907.EURP190.PROD.OUTLOOK.COM (2603:10a6:10:3bd::6) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7249.17; Thu, 1 Feb\n 2024 09:58:46 +0000", "from DU2PEPF00028D0C.eurprd03.prod.outlook.com\n (2603:10a6:10:be:cafe::6b) by DB8PR03CA0019.outlook.office365.com\n (2603:10a6:10:be::32) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7249.26 via Frontend\n Transport; Thu, 1 Feb 2024 09:58:46 +0000", "from localhost.localdomain.com (178.72.21.4) by\n DU2PEPF00028D0C.mail.protection.outlook.com (10.167.242.20) with Microsoft\n SMTP Server id 15.20.7249.19 via Frontend Transport; Thu, 1 Feb 2024 09:58:45\n +0000" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=RXKdwfAX/mdyhiOHH2fS2Kw1IK5TLOPTieIpWiewIaHxhaOIEL6xZsgd4IzuIoUW1qByGDdJTxVPh4TVLdz+rlpLVUM/+mOl2XjVyb+AhJ3fCUk24+TC7dAg5nBynwES8Al2hIOv41WV3iJq6cAjCw3ylBJ2Geif4ursRjtCiHtOv5OdB/Zq+9v3auTDIAK2pwAwI7yVLer9vARi4puUt5tezxFSWMhf3ImtxlNct/7uxg/w/MkQtJGFhIfTK70i+J+t1e/zF71dGapaYP53rUf15zd4Q3CPrCjFqGkQFBfHx3pnT7V2ubCGlt6YTRHkXTPucX4CPjQPvuqaOkqtew==", "ARC-Message-Signature": "i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=bDMjX0sFCEvHXSSHfs82dc3Z2D9JhWe6EX6fdCyFS1M=;\n b=GOIYdxuai1jtm2gftPBiLgLCKlVegOrAvF8kgXC92OFncxJNM4X8kVjl+eNQZjV7g4jy9Z4NLjY1OXz8kQnTxAfuY6b7HrXyYN+YJAkojV0i4Q86FlmScTzHNdX1jCiN5PnLWBtlXIkzqXd7xxkJcO1ou3FDAOLJZvOQJbqYaWVg4cVELgCzPkEMPIl1Mr36IlODpkabV0slPiWnJIh9KKDrLSbubKNWcG+T2XcR/sycj2yzKGIXb0j4YB0EDpaL1sSGnabqzUo2pkSmPC6CVjNXD2G1OOujWsQ4rhN3zoO5Sg4njJxiECTan6M6zjSurtdv3NoUrc08ivOYyUdI+g==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=fail (sender ip is\n 178.72.21.4) smtp.rcpttodomain=monjalon.net smtp.mailfrom=napatech.com;\n dmarc=fail (p=none sp=none pct=100) action=none header.from=napatech.com;\n dkim=none (message not signed); arc=none (0)", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=napatech.com;\n s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=bDMjX0sFCEvHXSSHfs82dc3Z2D9JhWe6EX6fdCyFS1M=;\n b=DqDQ5tBMoNWr4oLLgcK4yoip+G1J2mV0pTaLn/kKOPdtRtqOz9qCsXtQUpVpl0tlvCWDcuq9EYtnbc34cU7NN8FR+XVcriprB8BSHm7RQJH9hTYkKuYS2M87GEVrvRRZJ3uoID5kW/U79aEtF2FhsYZgIKnku14wM5vIV0LYlAE=", "X-MS-Exchange-Authentication-Results": "spf=fail (sender IP is 178.72.21.4)\n smtp.mailfrom=napatech.com; dkim=none (message not signed)\n header.d=none;dmarc=fail action=none header.from=napatech.com;", "Received-SPF": "Fail (protection.outlook.com: domain of napatech.com does not\n designate 178.72.21.4 as permitted sender) receiver=protection.outlook.com;\n client-ip=178.72.21.4; helo=localhost.localdomain.com;", "From": "Oleksandr Kolomeiets <okl-plv@napatech.com>", "To": "thomas@monjalon.net, aman.deep.singh@intel.com, yuying.zhang@intel.com,\n orika@nvidia.com, mko-plv@napatech.com, ckm@napatech.com", "Cc": "dev@dpdk.org", "Subject": "[PATCH v2] app/testpmd: support updating flow rule actions", "Date": "Thu, 1 Feb 2024 10:59:07 +0100", "Message-ID": "<20240201095907.3564286-1-okl-plv@napatech.com>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<20240126132142.2649250-1-okl-plv@napatech.com>", "References": "<20240126132142.2649250-1-okl-plv@napatech.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-EOPAttributedMessage": "0", "X-MS-PublicTrafficType": "Email", "X-MS-TrafficTypeDiagnostic": "DU2PEPF00028D0C:EE_|DU0P190MB1907:EE_", "Content-Type": "text/plain", "X-MS-Office365-Filtering-Correlation-Id": "798ac624-6b9c-48ef-966a-08dc230c618b", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n ytVn7vEbLYf1YuVEqp2VOtkNSg/YF55VSjUXAvW7JcWBDFll+x3U90tv0xql4lE25GpS13DtbJtZ5KxSzVKN9GOvabvLdgXZZ+qXMWbEHubvuU2saQMqdu7ci5WDG2sxhtT+eOcwc8XXN2i7jzlc9WMSVIspOrltTMn+LI5WY/Ui7GtZrtthPPvSRuKlbMnKFdNHQLnIq3RcF4fN6SQLAPQD+RDH2tT+VcqUAciKeKJfd39kzv/V+dD6PyJbqdaGjarmfHKoy4vYCZDdPU6E6zHYDNPJnbbV91hwGPyMYDdoaC5apYOWBVvgZvuUJD2g6hgavlbxzpEXgWbEhb4MPj61c0AYduDSmX2+p/2AF2uSSYDe+yw/DYk4oKWIDADclNVWVhy3epEP41hdpjUv65a1ge2biv81kXoZzfTT5lgFMVAeesD2LwBqu2Pf45GP/gCp/PmCdOtYEzwh0ErLZdSUTJrYsyPARj+TpihJ4TNB34QSWvhAZSAzGwbb0jBfr8wS5ZTM1qPkX6qlvHbByWoKuN4nn5v42efYmPqDWidZ4+cwc5dNUGkVnkdgTid2kSoWj/F2mbnT01VaVJiDA/aCoueMb6Yv9FrRb0XTWrUFSsbwZGnvEgmrVlx7T4R3qtH/WQ+NHk89WKcnKnxvshqdLkEyJNqidsGSsjPVKo7p/fCFJ2u/izEvjH/xNvvFM1/UmEwOwua1Wz0w2I6GuDol/tyDDvA7KG12prGz12ZGguvByb9uSOVRGr24O4DwLbXW31DRL1V2BVHNZPz2FA==", "X-Forefront-Antispam-Report": "CIP:178.72.21.4; CTRY:DK; LANG:en; SCL:1; SRV:;\n IPV:NLI; SFV:NSPM; H:localhost.localdomain.com; PTR:InfoDomainNonexistent;\n CAT:NONE;\n SFS:(13230031)(6069001)(346002)(136003)(39840400004)(376002)(396003)(230922051799003)(186009)(82310400011)(64100799003)(451199024)(1800799012)(36840700001)(46966006)(9316004)(2616005)(6636002)(36756003)(6486002)(478600001)(26005)(118246002)(2906002)(4326008)(8936002)(5660300002)(70206006)(86362001)(8676002)(36736006)(316002)(36860700001)(47076005)(70586007)(356005)(81166007)(6666004)(6512007)(83380400001)(6506007)(1076003)(336012)(41300700001)(40480700001)(956004)(36900700001)(15866825006);\n DIR:OUT; SFP:1102;", "X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount": "1", "X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0": "\n iG4m5OvbHbgn+17ZPW3tf+8mYLCj/V7Uz8y3Z9CboIXkEL6zzvzh36DMDYeSllWXGO3VOLhvlo1a2M0PA8SBQ8IjJF8teBeyEHJ6nvBoadygQlXc74yVE66SgfAC2t2QIRpaVggM+dcUieuWfZBpnrklN+JRiUp4E5O7CNYk7noSg0KvVy7inKBayKy4PVTKCI3Da89J/ix6kaXiG7j3mDPfKO69zLZX0N3bBlAXnsSZy/AVGxpaQnva3cqfjVqogO+BJ6yIh5KvJV8ai1tuLY1J2K0dcCAbZSH8rc9AHri9IAWkd+1ufG8bIPGqPR0iEF2XgSaC4tMNfZJTIgWQyU0K46LLqKryVqWFxU4eWpGxapl+ti1XziIbVKRN1Mds14Mm8RZXpJTEAv67yXZPv1ZMpmHX+nfMD590iV9Ngy+3o81Bde8pklBadzDBOr27RWw3ilFvnEgWQUa7ZapD2fZ8cSod1oiifzOF+cnqYJiHp/0Cpu8jCE7BoIzEeVyshZgv7+cUinRiURSSXXcIQtByN5e2a5Be/RoaUtQPOgcfrk+pGOzO1voH9/wuoFnWWLYlZlD2Kt5RLCl305H9ZtgotmokMHXOE/EXeUTDe6y/inxlAWpwJz7TknXRWUnc/74oSPZhCS6G8V0syOjx9w==", "X-OriginatorOrg": "napatech.com", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "01 Feb 2024 09:58:45.9916 (UTC)", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n 798ac624-6b9c-48ef-966a-08dc230c618b", "X-MS-Exchange-CrossTenant-Id": "c4540d0b-728a-4233-9da5-9ea30c7ec3ed", "X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp": "\n TenantId=c4540d0b-728a-4233-9da5-9ea30c7ec3ed; Ip=[178.72.21.4];\n Helo=[localhost.localdomain.com]", "X-MS-Exchange-CrossTenant-AuthSource": "\n DU2PEPF00028D0C.eurprd03.prod.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Anonymous", "X-MS-Exchange-CrossTenant-FromEntityHeader": "HybridOnPrem", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "DU0P190MB1907", "X-BESS-ID": "1706781563-303288-12393-4500-1", "X-BESS-VER": "2019.1_20240130.2130", "X-BESS-Apparent-Source-IP": "104.47.12.51", "X-BESS-Parts": "H4sIAAAAAAACA4uuVkqtKFGyUioBkjpK+cVKVpYWRpZAVgZQMMUoxTg1LdEsMT\n XZNMkozSI1LSXNMMnCJDHR3NTA0sBCqTYWALk9jp1BAAAA", "X-BESS-Outbound-Spam-Score": "0.00", "X-BESS-Outbound-Spam-Report": "Code version 3.2,\n rules version 3.2.2.253908 [from\n cloudscan17-55.eu-central-1b.ess.aws.cudaops.com]\n Rule breakdown below\n pts rule name description\n ---- ---------------------- --------------------------------\n 0.00 MAILTO_TO_SPAM_ADDR META: Includes a link to a likely spammer email\n 0.00 BSF_BESS_OUTBOUND META: BESS Outbound\n 0.00 BSF_SC0_MISMATCH_TO META: Envelope rcpt doesn't match header", "X-BESS-Outbound-Spam-Status": "SCORE=0.00 using account:ESS113687 scores of\n KILL_LEVEL=7.0 tests=MAILTO_TO_SPAM_ADDR, BSF_BESS_OUTBOUND,\n BSF_SC0_MISMATCH_TO", "X-BESS-BRTS-Status": "1", "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": "\"flow actions_update\" updates a flow rule specified by a rule ID with a\nnew action list by making a call to \"rte_flow_actions_update()\":\n\n flow actions_update {port_id} {rule_id}\n actions {action} [/ {action} [...]] / end [user_id]\n\nCreating, updating and destroying a flow rule:\n\n testpmd> flow create 0 group 1 pattern eth / end actions drop / end\n Flow rule #0 created\n testpmd> flow actions_update 0 0 actions queue index 1 / end\n Flow rule #0 updated with new actions\n testpmd> flow destroy 0 rule 0\n Flow rule #0 destroyed\n\nSigned-off-by: Oleksandr Kolomeiets <okl-plv@napatech.com>\nReviewed-by: Mykola Kostenok <mko-plv@napatech.com>\nReviewed-by: Christian Koue Muf <ckm@napatech.com>\n---\nv2:\n* No changes.\n---\n .mailmap | 3 ++\n app/test-pmd/cmdline.c | 4 ++\n app/test-pmd/cmdline_flow.c | 34 +++++++++++++++-\n app/test-pmd/config.c | 43 ++++++++++++++++++++\n app/test-pmd/testpmd.h | 3 ++\n doc/guides/testpmd_app_ug/testpmd_funcs.rst | 44 +++++++++++++++++++++\n 6 files changed, 130 insertions(+), 1 deletion(-)", "diff": "diff --git a/.mailmap b/.mailmap\nindex 6011526cae..44da31c1c2 100644\n--- a/.mailmap\n+++ b/.mailmap\n@@ -232,6 +232,7 @@ Chintu Hetam <rometoroam@gmail.com>\n Choonho Son <choonho.son@gmail.com>\n Chris Metcalf <cmetcalf@mellanox.com>\n Christian Ehrhardt <christian.ehrhardt@canonical.com>\n+Christian Koue Muf <ckm@napatech.com>\n Christian Maciocco <christian.maciocco@intel.com>\n Christophe Fontaine <cfontain@redhat.com>\n Christophe Grosse <christophe.grosse@6wind.com>\n@@ -986,6 +987,7 @@ Mukesh Dua <mukesh.dua81@gmail.com>\n Murphy Yang <murphyx.yang@intel.com>\n Murthy NSSR <nidadavolu.murthy@caviumnetworks.com>\n Muthurajan Jayakumar <muthurajan.jayakumar@intel.com>\n+Mykola Kostenok <mko-plv@napatech.com>\n Nachiketa Prachanda <nprachan@brocade.com> <nprachan@vyatta.att-mail.com>\n Nagadheeraj Rottela <rnagadheeraj@marvell.com>\n Naga Harish K S V <s.v.naga.harish.k@intel.com>\n@@ -1040,6 +1042,7 @@ Odi Assli <odia@nvidia.com>\n Ognjen Joldzic <ognjen.joldzic@gmail.com>\n Ola Liljedahl <ola.liljedahl@arm.com>\n Oleg Polyakov <olegp123@walla.co.il>\n+Oleksandr Kolomeiets <okl-plv@napatech.com>\n Olga Shern <olgas@nvidia.com> <olgas@mellanox.com>\n Olivier Gournet <ogournet@corp.free.fr>\n Olivier Matz <olivier.matz@6wind.com>\ndiff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c\nindex f704319771..8249e4eb92 100644\n--- a/app/test-pmd/cmdline.c\n+++ b/app/test-pmd/cmdline.c\n@@ -852,6 +852,10 @@ static void cmd_help_long_parsed(void *parsed_result,\n \t\t\t\"flow destroy {port_id} rule {rule_id} [...]\\n\"\n \t\t\t\" Destroy specific flow rules.\\n\\n\"\n \n+\t\t\t\"flow actions_update {port_id} {rule_id}\"\n+\t\t\t\" actions {action} [/ {action} [...]] / end [user_id]\\n\"\n+\t\t\t\" Update a flow rule with new actions.\\n\\n\"\n+\n \t\t\t\"flow flush {port_id}\\n\"\n \t\t\t\" Destroy all flow rules.\\n\\n\"\n \ndiff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c\nindex 359c187b3c..0af8d13121 100644\n--- a/app/test-pmd/cmdline_flow.c\n+++ b/app/test-pmd/cmdline_flow.c\n@@ -91,6 +91,7 @@ enum index {\n \tVALIDATE,\n \tCREATE,\n \tDESTROY,\n+\tACTIONS_UPDATE,\n \tFLUSH,\n \tDUMP,\n \tQUERY,\n@@ -250,6 +251,7 @@ enum index {\n \tVC_TUNNEL_SET,\n \tVC_TUNNEL_MATCH,\n \tVC_USER_ID,\n+\tVC_IS_USER_ID,\n \n \t/* Dump arguments */\n \tDUMP_ALL,\n@@ -3084,6 +3086,7 @@ static const struct token token_list[] = {\n \t\t\t VALIDATE,\n \t\t\t CREATE,\n \t\t\t DESTROY,\n+\t\t\t ACTIONS_UPDATE,\n \t\t\t FLUSH,\n \t\t\t DUMP,\n \t\t\t LIST,\n@@ -3888,6 +3891,17 @@ static const struct token token_list[] = {\n \t\t.args = ARGS(ARGS_ENTRY(struct buffer, port)),\n \t\t.call = parse_destroy,\n \t},\n+\t[ACTIONS_UPDATE] = {\n+\t\t.name = \"actions_update\",\n+\t\t.help = \"update a flow rule with new actions\",\n+\t\t.next = NEXT(NEXT_ENTRY(VC_IS_USER_ID, END),\n+\t\t\t NEXT_ENTRY(ACTIONS),\n+\t\t\t NEXT_ENTRY(COMMON_RULE_ID),\n+\t\t\t NEXT_ENTRY(COMMON_PORT_ID)),\n+\t\t.args = ARGS(ARGS_ENTRY(struct buffer, args.vc.rule_id),\n+\t\t\t ARGS_ENTRY(struct buffer, port)),\n+\t\t.call = parse_vc,\n+\t},\n \t[FLUSH] = {\n \t\t.name = \"flush\",\n \t\t.help = \"destroy all flow rules\",\n@@ -4134,6 +4148,11 @@ static const struct token token_list[] = {\n \t\t.args = ARGS(ARGS_ENTRY(struct buffer, args.vc.user_id)),\n \t\t.call = parse_vc,\n \t},\n+\t[VC_IS_USER_ID] = {\n+\t\t.name = \"user_id\",\n+\t\t.help = \"rule identifier is user-id\",\n+\t\t.call = parse_vc,\n+\t},\n \t/* Validate/create pattern. */\n \t[ITEM_PATTERN] = {\n \t\t.name = \"pattern\",\n@@ -8083,8 +8102,13 @@ parse_vc(struct context *ctx, const struct token *token,\n \tif (!out->command) {\n \t\tif (ctx->curr != VALIDATE && ctx->curr != CREATE &&\n \t\t ctx->curr != PATTERN_TEMPLATE_CREATE &&\n-\t\t ctx->curr != ACTIONS_TEMPLATE_CREATE)\n+\t\t ctx->curr != ACTIONS_TEMPLATE_CREATE &&\n+\t\t ctx->curr != ACTIONS_UPDATE)\n \t\t\treturn -1;\n+\t\tif (ctx->curr == ACTIONS_UPDATE)\n+\t\t\tout->args.vc.pattern =\n+\t\t\t\t(void *)RTE_ALIGN_CEIL((uintptr_t)(out + 1),\n+\t\t\t\t\t\t sizeof(double));\n \t\tif (sizeof(*out) > size)\n \t\t\treturn -1;\n \t\tout->command = ctx->curr;\n@@ -8156,6 +8180,9 @@ parse_vc(struct context *ctx, const struct token *token,\n \t\tctx->object = out->args.vc.actions;\n \t\tctx->objmask = NULL;\n \t\treturn len;\n+\tcase VC_IS_USER_ID:\n+\t\tout->args.vc.user_id = true;\n+\t\treturn len;\n \tdefault:\n \t\tif (!token->priv)\n \t\t\treturn -1;\n@@ -12713,6 +12740,11 @@ cmd_flow_parsed(const struct buffer *in)\n \t\t\t\t in->args.destroy.rule,\n \t\t\t\t in->args.destroy.is_user_id);\n \t\tbreak;\n+\tcase ACTIONS_UPDATE:\n+\t\tport_flow_actions_update(in->port, in->args.vc.rule_id,\n+\t\t\t\t\t in->args.vc.actions,\n+\t\t\t\t\t in->args.vc.user_id);\n+\t\tbreak;\n \tcase FLUSH:\n \t\tport_flow_flush(in->port);\n \t\tbreak;\ndiff --git a/app/test-pmd/config.c b/app/test-pmd/config.c\nindex cad7537bc6..09d44908b0 100644\n--- a/app/test-pmd/config.c\n+++ b/app/test-pmd/config.c\n@@ -3696,6 +3696,49 @@ port_flow_destroy(portid_t port_id, uint32_t n, const uint64_t *rule,\n \treturn ret;\n }\n \n+/** Update a flow rule with new actions. */\n+int\n+port_flow_actions_update(portid_t port_id, uint32_t rule_id,\n+\t\t\t const struct rte_flow_action *actions, bool is_user_id)\n+{\n+\tstruct rte_port *port;\n+\tstruct port_flow **flow_list;\n+\n+\tif (port_id_is_invalid(port_id, ENABLED_WARN) ||\n+\t port_id == (portid_t)RTE_PORT_ALL)\n+\t\treturn -EINVAL;\n+\tport = &ports[port_id];\n+\tflow_list = &port->flow_list;\n+\twhile (*flow_list) {\n+\t\tstruct port_flow *flow = *flow_list;\n+\t\tstruct rte_flow_error error;\n+\n+\t\tif (rule_id != (is_user_id ? flow->user_id : flow->id)) {\n+\t\t\tflow_list = &flow->next;\n+\t\t\tcontinue;\n+\t\t}\n+\t\t/*\n+\t\t * Poisoning to make sure PMDs update it in case\n+\t\t * of error.\n+\t\t */\n+\t\tmemset(&error, 0x33, sizeof(error));\n+\t\tif (rte_flow_actions_update(port_id, flow->flow, actions,\n+\t\t\t\t\t &error))\n+\t\t\treturn port_flow_complain(&error);\n+\t\tif (is_user_id)\n+\t\t\tprintf(\"Flow rule #%\"PRIu64\" updated with new actions,\"\n+\t\t\t \" user-id 0x%\"PRIx64\"\\n\",\n+\t\t\t flow->id, flow->user_id);\n+\t\telse\n+\t\t\tprintf(\"Flow rule #%\"PRIu64\n+\t\t\t \" updated with new actions\\n\",\n+\t\t\t flow->id);\n+\t\treturn 0;\n+\t}\n+\tprintf(\"Failed to find flow %\"PRIu32\"\\n\", rule_id);\n+\treturn -EINVAL;\n+}\n+\n /** Remove all flow rules. */\n int\n port_flow_flush(portid_t port_id)\ndiff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h\nindex 9b10a9ea1c..7ff76658e7 100644\n--- a/app/test-pmd/testpmd.h\n+++ b/app/test-pmd/testpmd.h\n@@ -1034,6 +1034,9 @@ void update_age_action_context(const struct rte_flow_action *actions,\n int mcast_addr_pool_destroy(portid_t port_id);\n int port_flow_destroy(portid_t port_id, uint32_t n, const uint64_t *rule,\n \t\t bool is_user_id);\n+int port_flow_actions_update(portid_t port_id, uint32_t rule,\n+\t\t\t const struct rte_flow_action *actions,\n+\t\t\t bool is_user_id);\n int port_flow_flush(portid_t port_id);\n int port_flow_dump(portid_t port_id, bool dump_all,\n \t\t\tuint64_t rule, const char *file_name,\ndiff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst\nindex ab18a80b30..274cb96724 100644\n--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst\n+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst\n@@ -3046,6 +3046,11 @@ following sections.\n \n flow destroy {port_id} rule {rule_id} [...] [user_id]\n \n+- Update a flow rule with new actions::\n+\n+ flow actions_update {port_id} {rule_id}\n+ actions {action} [/ {action} [...]] / end [user_id]\n+\n - Destroy all flow rules::\n \n flow flush {port_id}\n@@ -4235,6 +4240,45 @@ Non-existent rule IDs are ignored::\n Flow rule #0 destroyed\n testpmd>\n \n+Updating flow rules with new actions\n+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n+\n+``flow actions_update`` updates a flow rule specified by a rule ID with a new\n+action list by making a call to ``rte_flow_actions_update()``::\n+\n+ flow actions_update {port_id} {rule_id}\n+ actions {action} [/ {action} [...]] / end [user_id]\n+\n+If successful, it will show::\n+\n+ Flow rule #[...] updated with new actions\n+\n+Or if ``user_id`` flag is provided::\n+\n+ Flow rule #[...] updated with new actions, user-id [...]\n+\n+If a flow rule can not be found::\n+\n+ Failed to find flow [...]\n+\n+Otherwise it will show the usual error message of the form::\n+\n+ Caught error type [...] ([...]): [...]\n+\n+Optional ``user_id`` is a flag that signifies the rule ID is the one provided\n+by the user at creation.\n+\n+Action list is identical to the one described for the ``flow create``.\n+\n+Creating, updating and destroying a flow rule::\n+\n+ testpmd> flow create 0 group 1 pattern eth / end actions drop / end\n+ Flow rule #0 created\n+ testpmd> flow actions_update 0 0 actions queue index 1 / end\n+ Flow rule #0 updated with new actions\n+ testpmd> flow destroy 0 rule 0\n+ Flow rule #0 destroyed\n+\n Enqueueing destruction of flow rules\n ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n \n", "prefixes": [ "v2" ] }