Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/44762/?format=api
http://patchwork.dpdk.org/api/patches/44762/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/1537108670-11380-3-git-send-email-dekelp@mellanox.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": "<1537108670-11380-3-git-send-email-dekelp@mellanox.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1537108670-11380-3-git-send-email-dekelp@mellanox.com", "date": "2018-09-16T14:37:49", "name": "[v2,2/3] app/testpmd: support metadata as flow rule criteria", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "5219969fcb93853a7fa40682dfaa87e0388b209a", "submitter": { "id": 1082, "url": "http://patchwork.dpdk.org/api/people/1082/?format=api", "name": "Dekel Peled", "email": "dekelp@mellanox.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/1537108670-11380-3-git-send-email-dekelp@mellanox.com/mbox/", "series": [ { "id": 1338, "url": "http://patchwork.dpdk.org/api/series/1338/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=1338", "date": "2018-09-16T14:37:49", "name": null, "version": 2, "mbox": "http://patchwork.dpdk.org/series/1338/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/44762/comments/", "check": "fail", "checks": "http://patchwork.dpdk.org/api/patches/44762/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 B442934F3;\n\tSun, 16 Sep 2018 16:39:17 +0200 (CEST)", "from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129])\n\tby dpdk.org (Postfix) with ESMTP id 394BF532C\n\tfor <dev@dpdk.org>; Sun, 16 Sep 2018 16:39:16 +0200 (CEST)", "from Internal Mail-Server by MTLPINE1 (envelope-from\n\tdekelp@mellanox.com)\n\twith ESMTPS (AES256-SHA encrypted); 16 Sep 2018 16:43:32 +0200", "from mtl-vdi-280.wap.labs.mlnx. (mtl-vdi-280.wap.labs.mlnx\n\t[10.128.130.87])\n\tby labmailer.mlnx (8.13.8/8.13.8) with ESMTP id w8GEd0hK031896;\n\tSun, 16 Sep 2018 17:39:12 +0300" ], "From": "Dekel Peled <dekelp@mellanox.com>", "To": "wenzhuo.lu@intel.com, jingjing.wu@intel.com, bernard.iremonger@intel.com,\n\tdev@dpdk.org, olivier.matz@6wind.com, adrien.mazarguil@6wind.com,\n\tthomas@monjalon.net, ferruh.yigit@intel.com, arybchenko@solarflare.com", "Cc": "shahafs@mellanox.com, orika@mellanox.com", "Date": "Sun, 16 Sep 2018 17:37:49 +0300", "Message-Id": "<1537108670-11380-3-git-send-email-dekelp@mellanox.com>", "X-Mailer": "git-send-email 1.7.1", "In-Reply-To": "<1537104835-9034-1-git-send-email-dekelp@mellanox.com>", "References": "<1537104835-9034-1-git-send-email-dekelp@mellanox.com>", "Subject": "[dpdk-dev] [PATCH v2 2/3] app/testpmd: support metadata as flow\n\trule criteria", "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": "As described in [1], this series adds option to set metadata value as\nmatch pattern when creating a new flow rule.\n\nThis patch introduces additional options in testpmd commands.\nNew item type \"meta\" \"data\", new offload flag \"match_metadata\".\n\n[1] \"ethdev: support metadata as flow rule criteria\"\n\nSigned-off-by: Dekel Peled <dekelp@mellanox.com>\n---\nV2:\n* Fix some checkpatch coding style issues (wrongly sent).\n---\n\n app/test-pmd/cmdline.c | 14 ++++++++------\n app/test-pmd/cmdline_flow.c | 25 +++++++++++++++++++++++++\n app/test-pmd/config.c | 1 +\n app/test-pmd/testpmd.c | 4 ++++\n doc/guides/testpmd_app_ug/testpmd_funcs.rst | 5 ++---\n 5 files changed, 40 insertions(+), 9 deletions(-)", "diff": "diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c\nindex 589121d..4559d59 100644\n--- a/app/test-pmd/cmdline.c\n+++ b/app/test-pmd/cmdline.c\n@@ -17403,7 +17403,8 @@ struct cmd_config_per_port_tx_offload_result {\n \t\t\t \"sctp_cksum#tcp_tso#udp_tso#outer_ipv4_cksum#\"\n \t\t\t \"qinq_insert#vxlan_tnl_tso#gre_tnl_tso#\"\n \t\t\t \"ipip_tnl_tso#geneve_tnl_tso#macsec_insert#\"\n-\t\t\t \"mt_lockfree#multi_segs#mbuf_fast_free#security\");\n+\t\t\t \"mt_lockfree#multi_segs#mbuf_fast_free#security#\"\n+\t\t\t \"match_metadata\");\n cmdline_parse_token_string_t cmd_config_per_port_tx_offload_result_on_off =\n \tTOKEN_STRING_INITIALIZER\n \t\t(struct cmd_config_per_port_tx_offload_result,\n@@ -17484,8 +17485,8 @@ struct cmd_config_per_port_tx_offload_result {\n \t\t \"sctp_cksum|tcp_tso|udp_tso|outer_ipv4_cksum|\"\n \t\t \"qinq_insert|vxlan_tnl_tso|gre_tnl_tso|\"\n \t\t \"ipip_tnl_tso|geneve_tnl_tso|macsec_insert|\"\n-\t\t \"mt_lockfree|multi_segs|mbuf_fast_free|security \"\n-\t\t \"on|off\",\n+\t\t \"mt_lockfree|multi_segs|mbuf_fast_free|security|\"\n+\t\t \"match_metadata on|off\",\n \t.tokens = {\n \t\t(void *)&cmd_config_per_port_tx_offload_result_port,\n \t\t(void *)&cmd_config_per_port_tx_offload_result_config,\n@@ -17535,7 +17536,8 @@ struct cmd_config_per_queue_tx_offload_result {\n \t\t\t \"sctp_cksum#tcp_tso#udp_tso#outer_ipv4_cksum#\"\n \t\t\t \"qinq_insert#vxlan_tnl_tso#gre_tnl_tso#\"\n \t\t\t \"ipip_tnl_tso#geneve_tnl_tso#macsec_insert#\"\n-\t\t\t \"mt_lockfree#multi_segs#mbuf_fast_free#security\");\n+\t\t\t \"mt_lockfree#multi_segs#mbuf_fast_free#security#\"\n+\t\t\t \"match_metadata\");\n cmdline_parse_token_string_t cmd_config_per_queue_tx_offload_result_on_off =\n \tTOKEN_STRING_INITIALIZER\n \t\t(struct cmd_config_per_queue_tx_offload_result,\n@@ -17588,8 +17590,8 @@ struct cmd_config_per_queue_tx_offload_result {\n \t\t \"sctp_cksum|tcp_tso|udp_tso|outer_ipv4_cksum|\"\n \t\t \"qinq_insert|vxlan_tnl_tso|gre_tnl_tso|\"\n \t\t \"ipip_tnl_tso|geneve_tnl_tso|macsec_insert|\"\n-\t\t \"mt_lockfree|multi_segs|mbuf_fast_free|security \"\n-\t\t \"on|off\",\n+\t\t \"mt_lockfree|multi_segs|mbuf_fast_free|security|\"\n+\t\t \"match_metadata on|off\",\n \t.tokens = {\n \t\t(void *)&cmd_config_per_queue_tx_offload_result_port,\n \t\t(void *)&cmd_config_per_queue_tx_offload_result_port_id,\ndiff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c\nindex f926060..f6ca6b4 100644\n--- a/app/test-pmd/cmdline_flow.c\n+++ b/app/test-pmd/cmdline_flow.c\n@@ -178,6 +178,8 @@ enum index {\n \tITEM_ICMP6_ND_OPT_SLA_ETH_SLA,\n \tITEM_ICMP6_ND_OPT_TLA_ETH,\n \tITEM_ICMP6_ND_OPT_TLA_ETH_TLA,\n+\tITEM_META,\n+\tITEM_META_DATA,\n \n \t/* Validate/create actions. */\n \tACTIONS,\n@@ -564,6 +566,7 @@ struct parse_action_priv {\n \tITEM_ICMP6_ND_OPT,\n \tITEM_ICMP6_ND_OPT_SLA_ETH,\n \tITEM_ICMP6_ND_OPT_TLA_ETH,\n+\tITEM_META,\n \tZERO,\n };\n \n@@ -784,6 +787,12 @@ struct parse_action_priv {\n \tZERO,\n };\n \n+static const enum index item_meta[] = {\n+\tITEM_META_DATA,\n+\tITEM_NEXT,\n+\tZERO,\n+};\n+\n static const enum index next_action[] = {\n \tACTION_END,\n \tACTION_VOID,\n@@ -1985,6 +1994,22 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *,\n \t\t.args = ARGS(ARGS_ENTRY_HTON\n \t\t\t (struct rte_flow_item_icmp6_nd_opt_tla_eth, tla)),\n \t},\n+\t[ITEM_META] = {\n+\t\t.name = \"meta\",\n+\t\t.help = \"match metadata header\",\n+\t\t.priv = PRIV_ITEM(META, sizeof(struct rte_flow_item_meta)),\n+\t\t.next = NEXT(item_meta),\n+\t\t.call = parse_vc,\n+\t},\n+\t[ITEM_META_DATA] = {\n+\t\t.name = \"data\",\n+\t\t.help = \"metadata value\",\n+\t\t.next = NEXT(item_meta, NEXT_ENTRY(UNSIGNED), item_param),\n+\t\t.args = ARGS(ARGS_ENTRY_MASK_HTON(struct rte_flow_item_meta,\n+\t\t\t\t\t\t\tdata,\n+\t\t\t\t\t\t\t\"\\xff\\xff\\xff\\xff\"\n+\t\t\t\t\t\t\t\"\\xff\\xff\\xff\\xff\")),\n+\t},\n \n \t/* Validate/create actions. */\n \t[ACTIONS] = {\ndiff --git a/app/test-pmd/config.c b/app/test-pmd/config.c\nindex 14ccd68..70b2cce 100644\n--- a/app/test-pmd/config.c\n+++ b/app/test-pmd/config.c\n@@ -1034,6 +1034,7 @@ void print_valid_ports(void)\n \t\t sizeof(struct rte_flow_item_icmp6_nd_opt_sla_eth)),\n \tMK_FLOW_ITEM(ICMP6_ND_OPT_TLA_ETH,\n \t\t sizeof(struct rte_flow_item_icmp6_nd_opt_tla_eth)),\n+\tMK_FLOW_ITEM(META, sizeof(struct rte_flow_item_meta)),\n };\n \n /** Pattern item specification types. */\ndiff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c\nindex ee48db2..969eb7c 100644\n--- a/app/test-pmd/testpmd.c\n+++ b/app/test-pmd/testpmd.c\n@@ -747,6 +747,10 @@ static void eth_dev_event_callback(char *device_name,\n \t\t DEV_TX_OFFLOAD_MBUF_FAST_FREE))\n \t\t\tport->dev_conf.txmode.offloads &=\n \t\t\t\t~DEV_TX_OFFLOAD_MBUF_FAST_FREE;\n+\t\tif (!(port->dev_info.tx_offload_capa &\n+\t\t\tDEV_TX_OFFLOAD_MATCH_METADATA))\n+\t\t\tport->dev_conf.txmode.offloads &=\n+\t\t\t\t~DEV_TX_OFFLOAD_MATCH_METADATA;\n \t\tif (numa_support) {\n \t\t\tif (port_numa[pid] != NUMA_NO_CONFIG)\n \t\t\t\tport_per_socket[port_numa[pid]]++;\ndiff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst\nindex dde205a..7d86692 100644\n--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst\n+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst\n@@ -3511,10 +3511,9 @@ This section lists supported pattern items and their attributes, if any.\n \n - ``sla {MAC-48}``: source Ethernet LLA.\n \n-- ``icmp6_nd_opt_sla_eth``: match ICMPv6 neighbor discovery target Ethernet\n- link-layer address option.\n+- ``meta``: match application specific metadata.\n \n- - ``tla {MAC-48}``: target Ethernet LLA.\n+ - ``data``: metadata value.\n \n Actions list\n ^^^^^^^^^^^^\n", "prefixes": [ "v2", "2/3" ] }{ "id": 44762, "url": "