From patchwork Sun Sep 16 16:53:43 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ori Kam X-Patchwork-Id: 44766 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D09A35B26; Sun, 16 Sep 2018 18:56:38 +0200 (CEST) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-eopbgr00089.outbound.protection.outlook.com [40.107.0.89]) by dpdk.org (Postfix) with ESMTP id 824BE5B1E for ; Sun, 16 Sep 2018 18:56:37 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=JGN00lqg8V7hbmRGEKG1d0GC+NaP2Bf4B41+ZcsGPHQ=; b=yEpjC/PFozFOgN1mkR02Rqb9LX5UbOxjAM5E6RspWeaoyLHCXWMggTs+U58vOpdVzx18FYhQANEWwRkkxCsELI8jzjH1A4DU467ueEtJodyIKRYtKEbJC4ycM0IXsg8Fw0hVWLewkviPA9I/cQHdk6UP4Tmg1UDAUhhItf8eweA= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=orika@mellanox.com; Received: from mellanox.com (37.142.13.130) by HE1PR05MB3434.eurprd05.prod.outlook.com (2603:10a6:7:33::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1143.15; Sun, 16 Sep 2018 16:56:33 +0000 From: Ori Kam To: arybchenko@solarflare.com, ferruh.yigit@intel.com, stephen@networkplumber.org, adrien.mazarguil@6wind.com Cc: dev@dpdk.org, dekelp@mellanox.com, thomas@monjalon.net, nelio.laranjeiro@6wind.com, yskoh@mellanox.com, orika@mellanox.com Date: Sun, 16 Sep 2018 16:53:43 +0000 Message-Id: <1537116824-191205-3-git-send-email-orika@mellanox.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1537116824-191205-1-git-send-email-orika@mellanox.com> References: <1537116824-191205-1-git-send-email-orika@mellanox.com> MIME-Version: 1.0 X-Originating-IP: [37.142.13.130] X-ClientProxiedBy: DB6P193CA0021.EURP193.PROD.OUTLOOK.COM (2603:10a6:6:29::31) To HE1PR05MB3434.eurprd05.prod.outlook.com (2603:10a6:7:33::24) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 99759dbc-494a-4028-086e-08d61bf55cd8 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989137)(4534165)(4627221)(201703031133081)(201702281549075)(8990107)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:HE1PR05MB3434; X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB3434; 3:gX3bwoZzREnGcjMR2f3L0WIN2URbciu0iVhwSL87MYTrH8UhVk/22sn2/Ed/jotymw0jH9AZkXfE15oLbl7N1qsE+1GPbRFJIBmq+eKVrLtb5UXk45OLuZ1QTXT2/GcEMgQYjbx0QnHsuePWAjJX84XxQCV+5lWUznS18v0AknhqBwjRxSpLkC34hx1AA5fsbEYNz3H1xYAQQwTq5XHzYFDr2X4h8rM/JRC5jq/9SWpt3pxtebFPMN0laNdZLg6i; 25:IyJAXBVKt+/tN6yfn9IxTec2iACDFfuggLReyyy3H6r1ZwudLw23Hyhpn6eoMEifEuNZEwyCxKGJQ9qVv+mnOzKSFV//dB6PC3upSiBG+uC+SADhU/q6se+iwg0PNxT29XvucMzYECcUeWdJ0xAngtekrnX3qGN+8uara8UN5NZHBJ6eAsA/DQ/17sgCsVDtbRA3uY7kQLuLKPFEI4IyS64fsmPVW4gYfl3AsMFPdYlzK1ayLq/lOOADZx25v0/sNh9m05pf8tZkyZtWr0MFvT2oaIP58Rk2lJOb72SqWZ6T+yW6Z+lb857iVnwNs1OcFXHGNihmXj3AZk+HbYKSNA==; 31:r39o1pG/gDxJA5WYAXTUs4qOVYyOdN5iqPCr1fqNCtw5rjKFHFCPuxl8PqAkQgtvqzVhj6vYnRZ1kXz2k+GGu/TEapU4V61BSYk4bAPCk+uOnQ/iLXDExkqV5jPaNQUvs8wbMbS2Or+qZQgcSOtdBEbo1DNh+VtskwC7tI9GVTq4n3UP+twrMW7iy2JLkR4o/QGJTsbc2PV52O+thRoB6zx1GdWYyK26xJzVk1M/wnM= X-MS-TrafficTypeDiagnostic: HE1PR05MB3434: X-LD-Processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB3434; 20:eyfuuaR1fWduVtRKDAGnX+O4RbmKtj2SjjCTkqnkAP3SmV0gykuUrNPVchTmLQ/ofgw2UVeK7ShyzeT5aldbIEabItdE7aCAap0Addc6L4ogctQ3JCchlLIfbjX5aB51NNr+IBEJCVfNRkEYqUuYZeSsj58YwoOrCvIRdIa6pi1PIze7XKNdbjcQwwtp9b7/oFN9QX1238FA3GyC4Xa6FQDvhvQKyoNM+iBgg0xqCigoTu8SMizuNPitCiyFelzggW13u5EkfIOsyENFnadBS7hmGtIATKvYkgCg1KiPMolHg71LJRzqW/0Iw+yahxPU9ebWDuqTEUIrFN9C5umO4VUrHehx5D/0NaiddCrCVAlt9SKB3XC6tVYRiqF1wjYe56TyV+PbPIl7CxYZgi9RJqBWttqJrzitU3A4K82h8k4vWUbtei3z0zm78LP5IcTr6en4i6sNe9TDUceai7hNXxFgHCdq2cZdASVh85ctl20oKtwzy4plB1yAn8NjlhWF; 4:RdQX1T21n1OmAi4Pm5Bj2qD4qrNlZOt0mjOG0zSZ4SJspvOkCMY0K4oidNvJ3qd29FGU5RPeRjUp9EbQHhCmf3OpGGb5gNcVrtzrf7S+g79/6GRtPPBgZ7ZAkGUVZr5PeMf4msky4vhUzFuyJYWPySQ/kbdW8LFXyjN80Upnv6vMlEyupVGjfxR1EOHtM/0qyw5G4Nuyz5MOHux7qm4bPfm907cuZZZqN6K6ae4Tt2/ZM5uUkY4KgU2G/EYXSXnlSsiGitPPUtMSI7nZwEm2uQ== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231355)(944501410)(52105095)(10201501046)(3002001)(93006095)(93001095)(6055026)(149027)(150027)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(20161123560045)(20161123562045)(20161123564045)(201708071742011)(7699050); SRVR:HE1PR05MB3434; BCL:0; PCL:0; RULEID:; SRVR:HE1PR05MB3434; X-Forefront-PRVS: 079756C6B9 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(136003)(346002)(366004)(396003)(39860400002)(376002)(189003)(199004)(16586007)(47776003)(81156014)(6666003)(50226002)(8936002)(25786009)(5660300001)(68736007)(81166006)(8676002)(4326008)(106356001)(105586002)(2906002)(4720700003)(69596002)(316002)(21086003)(305945005)(8886007)(7736002)(66066001)(33026002)(107886003)(956004)(3846002)(476003)(48376002)(51416003)(478600001)(97736004)(7696005)(52116002)(55016002)(53936002)(14444005)(26005)(36756003)(186003)(16526019)(386003)(50466002)(76176011)(2616005)(86362001)(6116002)(11346002)(446003)(486006); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR05MB3434; H:mellanox.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; Received-SPF: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; HE1PR05MB3434; 23:vhH1RDpxHhEw6W8EyvANolnn+LGRoTXIjODtU9aZG?= FL9iFyt99i/oYgeikmZLw+RGGpeqRCQH7zCmyzhoQX6F2knfNUg5vY2W/7SX+Fj0XTMIiaFQc8DXr/ysU3sA0KSICQ/G0jIzmBSKbD9wZQXYIjnrDzWLe8ANkJWPfaFaemnXoN1snSEYDalC4ZN339MXXKAevMflKo6L4HSx2+J2LFx45eKaGOUWCiTrSuhnPT8ltUXSko6Vzxe6LDj+7NUlFNSTIQExw3/WoKetzQXmJZpI53pL0Yad2/arJNdmvKgECoKVZyNcJgw2p3haSNfrBaVQ/BncjSkPwk6BaghuZnanQWqkFUNXUxcIdQqdN+G5GGAxQ5/903IpNlbCgbvhFuBRbZlzCOkY7wwz8vJJ6tlwOeZ8G4sx600OGyX+z+oVkhHnHbWIs9CAeBLjQ+P7OXnzV2hf4cC9p5rD2jInehSJdaALLEhbIM80JG2/jIg3H3i3a5XxyMDv7oo64siAdBkNiDMB7Dh/EOA52fnyvVeHLQRxyCIXCLOKdH5IAgsn8WZ37SofWJSG0OumYatnHOQPXsYlvYbQdmed8D5++x1HVBbwMH4W2ztQfmm8iad+R/jBSgGNBxJ+A6YCu9Iao7S6qO+Dff7D6b7h3Ap+ZyVQnyez13jeVVxlwC2VwWBq4EOm8xz9RrmSi7H3MaSzN045xDdF6LMETlNNfm+C72kDorAifzDNAoTzpX7z+uT+NSJjfTV0MfCHPrBRDc+ezDYiSkY87HaOsgXN+zD4tMZ6/51yekTujHMDXTFD/Stb9kUwhplyFoYegfUOeplz4zTSUyg3f9VmXn0LJ+GFBA27h3ZUJ6EsHXXPl+lVFfZN2FR+Y52NEEKQaF95d0kv3lcmC0pq+3AJk7GUILSzLaL6YCu7y/2PdPoilwHpNUN8ujjkKmAJJzrDGIuU4/PZ2tXwoQ3FjdxCt8PdkcKeejyB06D7UZLH24GLlC0jVYdx2N3aPGquxAAmDXJo62Rm3Kgt/dvCYYNMlv+BstXFejOhtCMzb/4Rbon2N/9c/oMb7F3krzCbylyn5sVsrYxuYZSIeCFYigCgoIZzQCDyMWCZXeMwnK16drK9q7zlftK8M6vF6TKDABZhN3xyTHMwA70lFlN7oOeTHcWHBR9e9W5ckxN95FJyw2sa+AxTjtEvXER0mFQElOfPG/VqHytKXfyaQorWvoJjR6hElkOaDBY7L0zovRJcVYMPWkO7eQ1qukEBgkpiAqqw0Weq0Nj6/X+FCaYuVwMQU/9dpYHgg== X-Microsoft-Antispam-Message-Info: kcnrZ9JYAS5V3vArD+KpV4mYw/i5KEew6PTdHofrPx5v2kNxwdxY/kpYVl5ixsSoUMALsZKx/h/9uFj99aVCkugrIf2i9COUiA3KWrDpY/gxR2FoP7JXdB2NyhXK0fPKFjZqMk36Dpv5r4lS+ACVHnV5B4kiKAeS4dWnYMHjSzt2299XYgmUbz9ORFxUhHZOcFA09pukVPNvRPR4DIe5s7LTBYevakD3iJbzwjbTL8edVjDZT61T2UrfNiMnO9Ptbca0UehxS19Sw79cpAvYszRSsKQoOaE/ev3h2pE4vrayQIySGjal4ijVpiB8Yvjrvnb+SYpw5xMvlpZthxH8jmxc+fPxA8xIMmR8rWm1xRg= X-Microsoft-Exchange-Diagnostics: 1; HE1PR05MB3434; 6:sk82tArYbn07OS/HmhA548eGp4J4OXnqHYAVNSpxMh9h68+JTjcqBsrcqMioYFOnb1Av3N7p58Dixyq4GSAZyr/TF4ilvjav7NYw8LShiznMNP+v07xWu5ZsJplE2ORBgJrtwA5a8jJyQODsqF9IZwqyMHN0CffzpRb3fMgoi1afAaJXkqOQ9IVzCo94W1/+PSyCeadrzroXl9VLaefiWZUK2cxUoRHu5BGcE9ia043Qu26jed7XT+dBlsb2uSkd0cxzEoQ+Vm0CpNuCITKTrNqtq7+fCMy1Od/bPz8OI6p06poECaVGYbK43L7hl+hiUjr8DevIHCqbzNBRuoFiFP1pWzVWkKCZk12FTyjnVVbt0x34xjUNIyqpGEfFoE/j/7De03P3udpsG41oD8iRRpyI49X04dDCylxZCTpiCHJstIrOpVRD13JuheedOgyZ4FrCgJxzfEsXEoZ51lFP4A==; 5:t9tlqum7GfKWc/+oDpOeP+i5ckVAAzpbZiEdXn6VjTRNScVl/a9LczdQcuaXGAf/8n5ll4jA3qHeESSLCw7JqZ6HI+WKLrep+bz4KuBsayFyDKkeIyG+2VNHW8XrggK5GUQZE3cSWUMXTHWVK1Sc061qTE31ojlInNm3fCNUWf4=; 7:0qzzwHqObBOzoB6KaXxAx7a9GZ/RW5sOSZ7kq7vQIvb9F254bhhobZZFj/TOxUD2b3SEsTaczM3khxXDvMF0fmDRX4tNm6nceSyd7dCfikhUy/MObGTq7JKXYUX79IzvEJB5iVx/bL37i79ngnCmFLezJmF9maTAv2U7Nft+LbXT7MDZNM2QafTcK8WhyqzXiq9Mvv9hWF3qf6mCMLWUkG2Cr58J6oQ9zqVAn+KaIz/Qov3R+YB6ddft5306Z1/s SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Sep 2018 16:56:33.8572 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 99759dbc-494a-4028-086e-08d61bf55cd8 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR05MB3434 Subject: [dpdk-dev] [PATCH 2/3] ethdev: convert testpmd encap commands to new API X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Currently there are 2 encapsulation commands in testpmd one for VXLAN and one for NVGRE, both of those commands are using the old rte encap command. This commit update the commands to work with the new tunnel encap actions. The reason that we have different encapsulation commands, one for VXLAN and one for NVGRE is the ease of use in testpmd, both commands are using the same rte flow action for tunnel encap. Signed-off-by: Ori Kam --- app/test-pmd/cmdline_flow.c | 294 +++++++++++++++++++++----------------------- 1 file changed, 137 insertions(+), 157 deletions(-) diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c index f926060..349e822 100644 --- a/app/test-pmd/cmdline_flow.c +++ b/app/test-pmd/cmdline_flow.c @@ -262,37 +262,13 @@ struct action_rss_data { uint16_t queue[ACTION_RSS_QUEUE_NUM]; }; -/** Maximum number of items in struct rte_flow_action_vxlan_encap. */ -#define ACTION_VXLAN_ENCAP_ITEMS_NUM 6 - -/** Storage for struct rte_flow_action_vxlan_encap including external data. */ -struct action_vxlan_encap_data { - struct rte_flow_action_vxlan_encap conf; - struct rte_flow_item items[ACTION_VXLAN_ENCAP_ITEMS_NUM]; - struct rte_flow_item_eth item_eth; - struct rte_flow_item_vlan item_vlan; - union { - struct rte_flow_item_ipv4 item_ipv4; - struct rte_flow_item_ipv6 item_ipv6; - }; - struct rte_flow_item_udp item_udp; - struct rte_flow_item_vxlan item_vxlan; -}; +/** Maximum buffer size for the encap data. */ +#define ACTION_TUNNEL_ENCAP_MAX_BUFFER_SIZE 64 -/** Maximum number of items in struct rte_flow_action_nvgre_encap. */ -#define ACTION_NVGRE_ENCAP_ITEMS_NUM 5 - -/** Storage for struct rte_flow_action_nvgre_encap including external data. */ -struct action_nvgre_encap_data { - struct rte_flow_action_nvgre_encap conf; - struct rte_flow_item items[ACTION_NVGRE_ENCAP_ITEMS_NUM]; - struct rte_flow_item_eth item_eth; - struct rte_flow_item_vlan item_vlan; - union { - struct rte_flow_item_ipv4 item_ipv4; - struct rte_flow_item_ipv6 item_ipv6; - }; - struct rte_flow_item_nvgre item_nvgre; +/** Storage for struct rte_flow_action_tunnel_encap including external data. */ +struct action_tunnel_encap_data { + struct rte_flow_action_tunnel_encap conf; + uint8_t buf[ACTION_TUNNEL_ENCAP_MAX_BUFFER_SIZE]; }; /** Maximum number of subsequent tokens and arguments on the stack. */ @@ -2438,8 +2414,8 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, .name = "vxlan_encap", .help = "VXLAN encapsulation, uses configuration set by \"set" " vxlan\"", - .priv = PRIV_ACTION(VXLAN_ENCAP, - sizeof(struct action_vxlan_encap_data)), + .priv = PRIV_ACTION(TUNNEL_ENCAP, + sizeof(struct action_tunnel_encap_data)), .next = NEXT(NEXT_ENTRY(ACTION_NEXT)), .call = parse_vc_action_vxlan_encap, }, @@ -2448,7 +2424,7 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, .help = "Performs a decapsulation action by stripping all" " headers of the VXLAN tunnel network overlay from the" " matched flow.", - .priv = PRIV_ACTION(VXLAN_DECAP, 0), + .priv = PRIV_ACTION(TUNNEL_DECAP, 0), .next = NEXT(NEXT_ENTRY(ACTION_NEXT)), .call = parse_vc, }, @@ -2456,8 +2432,8 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, .name = "nvgre_encap", .help = "NVGRE encapsulation, uses configuration set by \"set" " nvgre\"", - .priv = PRIV_ACTION(NVGRE_ENCAP, - sizeof(struct action_nvgre_encap_data)), + .priv = PRIV_ACTION(TUNNEL_ENCAP, + sizeof(struct action_tunnel_encap_data)), .next = NEXT(NEXT_ENTRY(ACTION_NEXT)), .call = parse_vc_action_nvgre_encap, }, @@ -2466,7 +2442,7 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, .help = "Performs a decapsulation action by stripping all" " headers of the NVGRE tunnel network overlay from the" " matched flow.", - .priv = PRIV_ACTION(NVGRE_DECAP, 0), + .priv = PRIV_ACTION(TUNNEL_DECAP, 0), .next = NEXT(NEXT_ENTRY(ACTION_NEXT)), .call = parse_vc, }, @@ -3034,6 +3010,9 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, return len; } +/** IP next protocol UDP. */ +#define IP_PROTO_UDP 0x11 + /** Parse VXLAN encap action. */ static int parse_vc_action_vxlan_encap(struct context *ctx, const struct token *token, @@ -3042,7 +3021,32 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, { struct buffer *out = buf; struct rte_flow_action *action; - struct action_vxlan_encap_data *action_vxlan_encap_data; + struct action_tunnel_encap_data *action_vxlan_encap_data; + struct rte_flow_item_eth eth = { .type = 0, }; + struct rte_flow_item_vlan vlan = { + .tci = vxlan_encap_conf.vlan_tci, + .inner_type = 0, + }; + struct rte_flow_item_ipv4 ipv4 = { + .hdr = { + .src_addr = vxlan_encap_conf.ipv4_src, + .dst_addr = vxlan_encap_conf.ipv4_dst, + .next_proto_id = IP_PROTO_UDP, + }, + }; + struct rte_flow_item_ipv6 ipv6 = { + .hdr = { + .proto = IP_PROTO_UDP, + }, + }; + struct rte_flow_item_udp udp = { + .hdr = { + .src_port = vxlan_encap_conf.udp_src, + .dst_port = vxlan_encap_conf.udp_dst, + }, + }; + struct rte_flow_item_vxlan vxlan = { .flags = 0, }; + uint8_t *header; int ret; ret = parse_vc(ctx, token, str, len, buf, size); @@ -3057,83 +3061,58 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, /* Point to selected object. */ ctx->object = out->args.vc.data; ctx->objmask = NULL; - /* Set up default configuration. */ + /* Copy the headers to the buffer. */ action_vxlan_encap_data = ctx->object; - *action_vxlan_encap_data = (struct action_vxlan_encap_data){ - .conf = (struct rte_flow_action_vxlan_encap){ - .definition = action_vxlan_encap_data->items, - }, - .items = { - { - .type = RTE_FLOW_ITEM_TYPE_ETH, - .spec = &action_vxlan_encap_data->item_eth, - .mask = &rte_flow_item_eth_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_VLAN, - .spec = &action_vxlan_encap_data->item_vlan, - .mask = &rte_flow_item_vlan_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_IPV4, - .spec = &action_vxlan_encap_data->item_ipv4, - .mask = &rte_flow_item_ipv4_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_UDP, - .spec = &action_vxlan_encap_data->item_udp, - .mask = &rte_flow_item_udp_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_VXLAN, - .spec = &action_vxlan_encap_data->item_vxlan, - .mask = &rte_flow_item_vxlan_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_END, - }, - }, - .item_eth.type = 0, - .item_vlan = { - .tci = vxlan_encap_conf.vlan_tci, - .inner_type = 0, - }, - .item_ipv4.hdr = { - .src_addr = vxlan_encap_conf.ipv4_src, - .dst_addr = vxlan_encap_conf.ipv4_dst, + *action_vxlan_encap_data = (struct action_tunnel_encap_data) { + .conf = (struct rte_flow_action_tunnel_encap){ + .buf = action_vxlan_encap_data->buf, }, - .item_udp.hdr = { - .src_port = vxlan_encap_conf.udp_src, - .dst_port = vxlan_encap_conf.udp_dst, - }, - .item_vxlan.flags = 0, + .buf = {}, }; - memcpy(action_vxlan_encap_data->item_eth.dst.addr_bytes, + header = action_vxlan_encap_data->buf; + if (vxlan_encap_conf.select_vlan) + eth.type = rte_cpu_to_be_16(ETHER_TYPE_VLAN); + else if (vxlan_encap_conf.select_ipv4) + eth.type = rte_cpu_to_be_16(ETHER_TYPE_IPv4); + else + eth.type = rte_cpu_to_be_16(ETHER_TYPE_IPv6); + memcpy(eth.dst.addr_bytes, vxlan_encap_conf.eth_dst, ETHER_ADDR_LEN); - memcpy(action_vxlan_encap_data->item_eth.src.addr_bytes, + memcpy(eth.src.addr_bytes, vxlan_encap_conf.eth_src, ETHER_ADDR_LEN); - if (!vxlan_encap_conf.select_ipv4) { - memcpy(&action_vxlan_encap_data->item_ipv6.hdr.src_addr, + memcpy(header, ð, sizeof(eth)); + header += sizeof(eth); + if (vxlan_encap_conf.select_vlan) { + if (vxlan_encap_conf.select_ipv4) + vlan.inner_type = rte_cpu_to_be_16(ETHER_TYPE_IPv4); + else + vlan.inner_type = rte_cpu_to_be_16(ETHER_TYPE_IPv6); + memcpy(header, &vlan, sizeof(vlan)); + header += sizeof(vlan); + } + if (vxlan_encap_conf.select_ipv4) { + memcpy(header, &ipv4, sizeof(ipv4)); + header += sizeof(ipv4); + } else { + memcpy(&ipv6.hdr.src_addr, &vxlan_encap_conf.ipv6_src, sizeof(vxlan_encap_conf.ipv6_src)); - memcpy(&action_vxlan_encap_data->item_ipv6.hdr.dst_addr, + memcpy(&ipv6.hdr.dst_addr, &vxlan_encap_conf.ipv6_dst, sizeof(vxlan_encap_conf.ipv6_dst)); - action_vxlan_encap_data->items[2] = (struct rte_flow_item){ - .type = RTE_FLOW_ITEM_TYPE_IPV6, - .spec = &action_vxlan_encap_data->item_ipv6, - .mask = &rte_flow_item_ipv6_mask, - }; + memcpy(header, &ipv6, sizeof(ipv6)); + header += sizeof(ipv6); } - if (!vxlan_encap_conf.select_vlan) - action_vxlan_encap_data->items[1].type = - RTE_FLOW_ITEM_TYPE_VOID; - memcpy(action_vxlan_encap_data->item_vxlan.vni, vxlan_encap_conf.vni, - RTE_DIM(vxlan_encap_conf.vni)); + memcpy(header, &udp, sizeof(udp)); + header += sizeof(udp); + memcpy(vxlan.vni, vxlan_encap_conf.vni, RTE_DIM(vxlan_encap_conf.vni)); + memcpy(header, &vxlan, sizeof(vxlan)); + header += sizeof(vxlan); + action_vxlan_encap_data->conf.size = header - + action_vxlan_encap_data->buf; action->conf = &action_vxlan_encap_data->conf; return ret; } - /** Parse NVGRE encap action. */ static int parse_vc_action_nvgre_encap(struct context *ctx, const struct token *token, @@ -3142,7 +3121,26 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, { struct buffer *out = buf; struct rte_flow_action *action; - struct action_nvgre_encap_data *action_nvgre_encap_data; + struct action_tunnel_encap_data *action_nvgre_encap_data; + struct rte_flow_item_eth eth = { .type = 0, }; + struct rte_flow_item_vlan vlan = { + .tci = nvgre_encap_conf.vlan_tci, + .inner_type = 0, + }; + struct rte_flow_item_ipv4 ipv4 = { + .hdr = { + .src_addr = nvgre_encap_conf.ipv4_src, + .dst_addr = nvgre_encap_conf.ipv4_dst, + .next_proto_id = IP_PROTO_UDP, + }, + }; + struct rte_flow_item_ipv6 ipv6 = { + .hdr = { + .proto = IP_PROTO_UDP, + }, + }; + struct rte_flow_item_nvgre nvgre = { .flow_id = 0, }; + uint8_t *header; int ret; ret = parse_vc(ctx, token, str, len, buf, size); @@ -3157,74 +3155,56 @@ static int comp_vc_action_rss_queue(struct context *, const struct token *, /* Point to selected object. */ ctx->object = out->args.vc.data; ctx->objmask = NULL; - /* Set up default configuration. */ + /* Copy the headers to the buffer. */ action_nvgre_encap_data = ctx->object; - *action_nvgre_encap_data = (struct action_nvgre_encap_data){ - .conf = (struct rte_flow_action_nvgre_encap){ - .definition = action_nvgre_encap_data->items, - }, - .items = { - { - .type = RTE_FLOW_ITEM_TYPE_ETH, - .spec = &action_nvgre_encap_data->item_eth, - .mask = &rte_flow_item_eth_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_VLAN, - .spec = &action_nvgre_encap_data->item_vlan, - .mask = &rte_flow_item_vlan_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_IPV4, - .spec = &action_nvgre_encap_data->item_ipv4, - .mask = &rte_flow_item_ipv4_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_NVGRE, - .spec = &action_nvgre_encap_data->item_nvgre, - .mask = &rte_flow_item_nvgre_mask, - }, - { - .type = RTE_FLOW_ITEM_TYPE_END, - }, - }, - .item_eth.type = 0, - .item_vlan = { - .tci = nvgre_encap_conf.vlan_tci, - .inner_type = 0, - }, - .item_ipv4.hdr = { - .src_addr = nvgre_encap_conf.ipv4_src, - .dst_addr = nvgre_encap_conf.ipv4_dst, + *action_nvgre_encap_data = (struct action_tunnel_encap_data) { + .conf = (struct rte_flow_action_tunnel_encap){ + .buf = action_nvgre_encap_data->buf, }, - .item_nvgre.flow_id = 0, + .buf = {}, }; - memcpy(action_nvgre_encap_data->item_eth.dst.addr_bytes, + header = action_nvgre_encap_data->buf; + if (nvgre_encap_conf.select_vlan) + eth.type = rte_cpu_to_be_16(ETHER_TYPE_VLAN); + else if (nvgre_encap_conf.select_ipv4) + eth.type = rte_cpu_to_be_16(ETHER_TYPE_IPv4); + else + eth.type = rte_cpu_to_be_16(ETHER_TYPE_IPv6); + memcpy(eth.dst.addr_bytes, nvgre_encap_conf.eth_dst, ETHER_ADDR_LEN); - memcpy(action_nvgre_encap_data->item_eth.src.addr_bytes, + memcpy(eth.src.addr_bytes, nvgre_encap_conf.eth_src, ETHER_ADDR_LEN); - if (!nvgre_encap_conf.select_ipv4) { - memcpy(&action_nvgre_encap_data->item_ipv6.hdr.src_addr, + memcpy(header, ð, sizeof(eth)); + header += sizeof(eth); + if (nvgre_encap_conf.select_vlan) { + if (nvgre_encap_conf.select_ipv4) + vlan.inner_type = rte_cpu_to_be_16(ETHER_TYPE_IPv4); + else + vlan.inner_type = rte_cpu_to_be_16(ETHER_TYPE_IPv6); + memcpy(header, &vlan, sizeof(vlan)); + header += sizeof(vlan); + } + if (nvgre_encap_conf.select_ipv4) { + memcpy(header, &ipv4, sizeof(ipv4)); + header += sizeof(ipv4); + } else { + memcpy(&ipv6.hdr.src_addr, &nvgre_encap_conf.ipv6_src, sizeof(nvgre_encap_conf.ipv6_src)); - memcpy(&action_nvgre_encap_data->item_ipv6.hdr.dst_addr, + memcpy(&ipv6.hdr.dst_addr, &nvgre_encap_conf.ipv6_dst, sizeof(nvgre_encap_conf.ipv6_dst)); - action_nvgre_encap_data->items[2] = (struct rte_flow_item){ - .type = RTE_FLOW_ITEM_TYPE_IPV6, - .spec = &action_nvgre_encap_data->item_ipv6, - .mask = &rte_flow_item_ipv6_mask, - }; + memcpy(header, &ipv6, sizeof(ipv6)); + header += sizeof(ipv6); } - if (!nvgre_encap_conf.select_vlan) - action_nvgre_encap_data->items[1].type = - RTE_FLOW_ITEM_TYPE_VOID; - memcpy(action_nvgre_encap_data->item_nvgre.tni, nvgre_encap_conf.tni, - RTE_DIM(nvgre_encap_conf.tni)); + memcpy(nvgre.tni, nvgre_encap_conf.tni, RTE_DIM(nvgre_encap_conf.tni)); + memcpy(header, &nvgre, sizeof(nvgre)); + header += sizeof(nvgre); + action_nvgre_encap_data->conf.size = header - + action_nvgre_encap_data->buf; action->conf = &action_nvgre_encap_data->conf; return ret; } - /** Parse tokens for destroy command. */ static int parse_destroy(struct context *ctx, const struct token *token,