From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f193.google.com (mail-wr0-f193.google.com [209.85.128.193]) by dpdk.org (Postfix) with ESMTP id 03C6623C for ; Tue, 24 Apr 2018 10:20:18 +0200 (CEST) Received: by mail-wr0-f193.google.com with SMTP id h3-v6so48174643wrh.5 for ; Tue, 24 Apr 2018 01:20:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=Xwbs4nssT5PLxjUgX2M2mkP/9SfcnoyxLNKJ+tiPOsY=; b=0LFXYsAvOWltf9CkvgUiilmG8q9l61n2x7Kkuqj5KgU5TUmnjgkeAswscH3ak/T4/C 9ZwKUyrbVtu5IC9wXvgsqy/O1mKKCLTQZnNC4+ep8xkl/c0R6iuBwX5Omwc76VBzidqI xp8x2iFAO/Ohys1e8G0ivB+UCSVx9UiKULwq69rFsHgjTamABuOybp0O3iRiAzPhMBtt vSmndpfhyMIoiSlPuAOeAC0M937CkzbL5wUr2/JgriL/lVOamScA7yU+JGzKM6sFRTKh VCQ1mfOlCiuHHSRAvX6vH8MX6b18ufhT0xtym4315z1BpYwOLpK9V/ooU79qOGyCzkC9 xIEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=Xwbs4nssT5PLxjUgX2M2mkP/9SfcnoyxLNKJ+tiPOsY=; b=e+Cp5WQrNd5O+cNR/jymzNoIsBBXwSuIgS4FtokCPcjj2NMIJh5WxTsWlx3VlAXP6D lqVE3vip234sPe9pa3CBl6/UaZrAHLzszv9sSF+PNHkUM2cagOHmUDHJ5qnhRouu0kQG sJBfjDXCY3bg3N7H2EBekkc5Q2TLDR+kOouifrSpkouU4vXdu+STeD9A2SmVzAYTShDv u0sFW4MK4ARJUE01olczk/fNqm2rpkzjKc0Eg25jzVgjCeYJx4w4QAKM5liM8ezaI8bM AYTTQ8Lr4aj9mYWQCdltnliycH+ZfTbFB55e0kgf7xpPKId8jqsbEk097kWyCLxAuCSh rm7g== X-Gm-Message-State: ALQs6tCFvoPMpFO5cYSbLKc2jzMECMIC+hhDcmhw+AThdFJGjKNQwzsX z2wdzgwIerxGoaJhkt3I8kUB X-Google-Smtp-Source: AIpwx49h/+fcb2SK7pL3c6NlOIkQKfKSXIBcaIGaDQAoqMblkDD788QwbnYNPTAwSsT3b+5caFZEPg== X-Received: by 2002:adf:87ab:: with SMTP id b40-v6mr20107454wrb.156.1524558018736; Tue, 24 Apr 2018 01:20:18 -0700 (PDT) Received: from laranjeiro-vm.dev.6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id c124sm10726843wmd.36.2018.04.24.01.20.17 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 24 Apr 2018 01:20:18 -0700 (PDT) Date: Tue, 24 Apr 2018 10:21:05 +0200 From: =?iso-8859-1?Q?N=E9lio?= Laranjeiro To: "Xueming(Steven) Li" Cc: Shahaf Shuler , "dev@dpdk.org" Message-ID: <20180424082105.htxbgiwl5i3shnv5@laranjeiro-vm.dev.6wind.com> References: <20180420122340.113348-1-xuemingl@mellanox.com> <20180423123310.32991-3-xuemingl@mellanox.com> <20180423125553.xr2mbqd7pfqvil4j@laranjeiro-vm.dev.6wind.com> <20180423134614.ssgcy73idz52x4f6@laranjeiro-vm.dev.6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [dpdk-dev] [PATCH v6 02/11] net/mlx5: support GRE tunnel flow 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: , X-List-Received-Date: Tue, 24 Apr 2018 08:20:20 -0000 On Tue, Apr 24, 2018 at 07:40:24AM +0000, Xueming(Steven) Li wrote: > > > > -----Original Message----- > > From: Nélio Laranjeiro > > Sent: Monday, April 23, 2018 9:46 PM > > To: Xueming(Steven) Li > > Cc: Shahaf Shuler ; dev@dpdk.org > > Subject: Re: [PATCH v6 02/11] net/mlx5: support GRE tunnel flow > > > > On Mon, Apr 23, 2018 at 01:32:23PM +0000, Xueming(Steven) Li wrote: > > > Hi Nelio, > > > > > > > -----Original Message----- > > > > From: Nélio Laranjeiro > > > > Sent: Monday, April 23, 2018 8:56 PM > > > > To: Xueming(Steven) Li > > > > Cc: Shahaf Shuler ; dev@dpdk.org > > > > Subject: Re: [PATCH v6 02/11] net/mlx5: support GRE tunnel flow > > > > > > > > On Mon, Apr 23, 2018 at 08:33:01PM +0800, Xueming Li wrote: > > > > > Signed-off-by: Xueming Li > > > > > --- > > > > > drivers/net/mlx5/mlx5_flow.c | 101 > > > > > ++++++++++++++++++++++++++++++++++++++++--- > > > > > 1 file changed, 94 insertions(+), 7 deletions(-) > > > > > > > > > > diff --git a/drivers/net/mlx5/mlx5_flow.c > > > > > b/drivers/net/mlx5/mlx5_flow.c index 5402cb148..b365f9868 100644 > > > > > --- a/drivers/net/mlx5/mlx5_flow.c > > > > > +++ b/drivers/net/mlx5/mlx5_flow.c > > > > > @@ -37,6 +37,7 @@ > > > > > /* Internet Protocol versions. */ #define MLX5_IPV4 4 #define > > > > > MLX5_IPV6 6 > > > > > +#define MLX5_GRE 47 > > > > > > > > > > #ifndef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT > > > > > struct ibv_flow_spec_counter_action { @@ -89,6 +90,11 @@ > > > > > mlx5_flow_create_vxlan(const struct rte_flow_item *item, > > > > > const void *default_mask, > > > > > struct mlx5_flow_data *data); > > > > > > > > > > +static int > > > > > +mlx5_flow_create_gre(const struct rte_flow_item *item, > > > > > + const void *default_mask, > > > > > + struct mlx5_flow_data *data); > > > > > + > > > > > struct mlx5_flow_parse; > > > > > > > > > > static void > > > > > @@ -231,6 +237,10 @@ struct rte_flow { > > > > > __VA_ARGS__, RTE_FLOW_ITEM_TYPE_END, \ > > > > > } > > > > > > > > > > +#define IS_TUNNEL(type) ( \ > > > > > + (type) == RTE_FLOW_ITEM_TYPE_VXLAN || \ > > > > > + (type) == RTE_FLOW_ITEM_TYPE_GRE) > > > > > + > > > > > /** Structure to generate a simple graph of layers supported by > > > > > the NIC. */ struct mlx5_flow_items { > > > > > /** List of possible actions for these items. */ @@ -284,7 > > > > > +294,8 @@ static const enum rte_flow_action_type valid_actions[] = > > > > > { static const struct mlx5_flow_items mlx5_flow_items[] = { > > > > > [RTE_FLOW_ITEM_TYPE_END] = { > > > > > .items = ITEMS(RTE_FLOW_ITEM_TYPE_ETH, > > > > > - RTE_FLOW_ITEM_TYPE_VXLAN), > > > > > + RTE_FLOW_ITEM_TYPE_VXLAN, > > > > > + RTE_FLOW_ITEM_TYPE_GRE), > > > > > }, > > > > > [RTE_FLOW_ITEM_TYPE_ETH] = { > > > > > .items = ITEMS(RTE_FLOW_ITEM_TYPE_VLAN, @@ -316,7 +327,8 @@ > > > > > static const struct mlx5_flow_items mlx5_flow_items[] = { > > > > > }, > > > > > [RTE_FLOW_ITEM_TYPE_IPV4] = { > > > > > .items = ITEMS(RTE_FLOW_ITEM_TYPE_UDP, > > > > > - RTE_FLOW_ITEM_TYPE_TCP), > > > > > + RTE_FLOW_ITEM_TYPE_TCP, > > > > > + RTE_FLOW_ITEM_TYPE_GRE), > > > > > .actions = valid_actions, > > > > > .mask = &(const struct rte_flow_item_ipv4){ > > > > > .hdr = { > > > > > @@ -333,7 +345,8 @@ static const struct mlx5_flow_items mlx5_flow_items[] = { > > > > > }, > > > > > [RTE_FLOW_ITEM_TYPE_IPV6] = { > > > > > .items = ITEMS(RTE_FLOW_ITEM_TYPE_UDP, > > > > > - RTE_FLOW_ITEM_TYPE_TCP), > > > > > + RTE_FLOW_ITEM_TYPE_TCP, > > > > > + RTE_FLOW_ITEM_TYPE_GRE), > > > > > .actions = valid_actions, > > > > > .mask = &(const struct rte_flow_item_ipv6){ > > > > > .hdr = { > > > > > @@ -386,6 +399,19 @@ static const struct mlx5_flow_items mlx5_flow_items[] = { > > > > > .convert = mlx5_flow_create_tcp, > > > > > .dst_sz = sizeof(struct ibv_flow_spec_tcp_udp), > > > > > }, > > > > > + [RTE_FLOW_ITEM_TYPE_GRE] = { > > > > > + .items = ITEMS(RTE_FLOW_ITEM_TYPE_ETH, > > > > > + RTE_FLOW_ITEM_TYPE_IPV4, > > > > > + RTE_FLOW_ITEM_TYPE_IPV6), > > > > > + .actions = valid_actions, > > > > > + .mask = &(const struct rte_flow_item_gre){ > > > > > + .protocol = -1, > > > > > + }, > > > > > + .default_mask = &rte_flow_item_gre_mask, > > > > > + .mask_sz = sizeof(struct rte_flow_item_gre), > > > > > + .convert = mlx5_flow_create_gre, > > > > > + .dst_sz = sizeof(struct ibv_flow_spec_tunnel), > > > > > + }, > > > > > [RTE_FLOW_ITEM_TYPE_VXLAN] = { > > > > > .items = ITEMS(RTE_FLOW_ITEM_TYPE_ETH), > > > > > .actions = valid_actions, > > > > > @@ -401,7 +427,7 @@ static const struct mlx5_flow_items > > > > > mlx5_flow_items[] = { > > > > > > > > > > /** Structure to pass to the conversion function. */ struct > > > > > mlx5_flow_parse { > > > > > - uint32_t inner; /**< Set once VXLAN is encountered. */ > > > > > + uint32_t inner; /**< Verbs value, set once tunnel is > > > > > +encountered. */ > > > > > uint32_t create:1; > > > > > /**< Whether resources should remain after a validate. */ > > > > > uint32_t drop:1; /**< Target is a drop queue. */ @@ -829,13 > > > > > +855,13 @@ mlx5_flow_convert_items_validate(const struct rte_flow_item items[], > > > > > cur_item->mask_sz); > > > > > if (ret) > > > > > goto exit_item_not_supported; > > > > > - if (items->type == RTE_FLOW_ITEM_TYPE_VXLAN) { > > > > > + if (IS_TUNNEL(items->type)) { > > > > > if (parser->inner) { > > > > > rte_flow_error_set(error, ENOTSUP, > > > > > RTE_FLOW_ERROR_TYPE_ITEM, > > > > > items, > > > > > - "cannot recognize multiple" > > > > > - " VXLAN encapsulations"); > > > > > + "Cannot recognize multiple" > > > > > + " tunnel encapsulations."); > > > > > return -rte_errno; > > > > > } > > > > > parser->inner = IBV_FLOW_SPEC_INNER; @@ -1641,6 +1667,67 @@ > > > > > mlx5_flow_create_vxlan(const struct rte_flow_item *item, } > > > > > > > > > > /** > > > > > + * Convert GRE item to Verbs specification. > > > > > + * > > > > > + * @param item[in] > > > > > + * Item specification. > > > > > + * @param default_mask[in] > > > > > + * Default bit-masks to use when item->mask is not provided. > > > > > + * @param data[in, out] > > > > > + * User structure. > > > > > + * > > > > > + * @return > > > > > + * 0 on success, a negative errno value otherwise and rte_errno is set. > > > > > + */ > > > > > +static int > > > > > +mlx5_flow_create_gre(const struct rte_flow_item *item __rte_unused, > > > > > + const void *default_mask __rte_unused, > > > > > + struct mlx5_flow_data *data) { > > > > > + struct mlx5_flow_parse *parser = data->parser; > > > > > + unsigned int size = sizeof(struct ibv_flow_spec_tunnel); > > > > > + struct ibv_flow_spec_tunnel tunnel = { > > > > > + .type = parser->inner | IBV_FLOW_SPEC_VXLAN_TUNNEL, > > > > > + .size = size, > > > > > + }; > > > > > + struct ibv_flow_spec_ipv4_ext *ipv4; > > > > > + struct ibv_flow_spec_ipv6 *ipv6; > > > > > + unsigned int i; > > > > > + > > > > > + parser->inner = IBV_FLOW_SPEC_INNER; > > > > > + /* Update encapsulation IP layer protocol. */ > > > > > + for (i = 0; i != hash_rxq_init_n; ++i) { > > > > > + if (!parser->queue[i].ibv_attr) > > > > > + continue; > > > > > + if (parser->out_layer == HASH_RXQ_IPV4) { > > > > > + ipv4 = (void *)((uintptr_t)parser->queue[i].ibv_attr + > > > > > + parser->queue[i].offset - > > > > > + sizeof(struct ibv_flow_spec_ipv4_ext)); > > > > > + if (ipv4->mask.proto && ipv4->val.proto != MLX5_GRE) > > > > > + break; > > > > > + ipv4->val.proto = MLX5_GRE; > > > > > + ipv4->mask.proto = 0xff; > > > > > + } else if (parser->out_layer == HASH_RXQ_IPV6) { > > > > > + ipv6 = (void *)((uintptr_t)parser->queue[i].ibv_attr + > > > > > + parser->queue[i].offset - > > > > > + sizeof(struct ibv_flow_spec_ipv6)); > > > > > + if (ipv6->mask.next_hdr && > > > > > + ipv6->val.next_hdr != MLX5_GRE) > > > > > + break; > > > > > + ipv6->val.next_hdr = MLX5_GRE; > > > > > + ipv6->mask.next_hdr = 0xff; > > > > > + } > > > > > + } > > > > > + if (i != hash_rxq_init_n) > > > > > + return rte_flow_error_set(data->error, EINVAL, > > > > > + RTE_FLOW_ERROR_TYPE_ITEM, > > > > > + item, > > > > > + "IP protocol of GRE must be 47"); > > > > > + mlx5_flow_create_copy(parser, &tunnel, size); > > > > > + return 0; > > > > > +} > > > > > > > > There is something strange, item is not unused as it is at least used in the rte_flow_error_set(). > > > > > > A new issue introduced when adding GRE protocol check. > > > If you finished this patchset review, I'll upload a new version to remove it. > > > > > > > > > > > In the other series you are pushing, there is no new > > > > RTE_FLOW_ITEM_GRE and in the current code there is also no RTE_FLOW_ITEM_GRE. > > > > > > > > I don't see how this code can match the missing item, what am I missing? > > > > > > Are you looking for RTE_FLOW_ITEM_TYPE_GRE? > > > > Yes > > RTE_FLOW_ITEM_TYPE_GRE has been defined in rte_flow.h, please check. Ok I've just missed it. > > > > > > > > > > > > +/** > > > > > * Convert mark/flag action to Verbs specification. > > > > > * > > > > > * @param parser > > > > > -- > > > > > 2.13.3 > > > > > > > > Thanks, > > > > > > > > -- > > > > Nélio Laranjeiro > > > > 6WIND > > > > -- > > Nélio Laranjeiro > > 6WIND -- Nélio Laranjeiro 6WIND