From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wj0-f171.google.com (mail-wj0-f171.google.com [209.85.210.171]) by dpdk.org (Postfix) with ESMTP id 649352B9C for ; Tue, 3 Jan 2017 15:08:10 +0100 (CET) Received: by mail-wj0-f171.google.com with SMTP id tq7so208737428wjb.0 for ; Tue, 03 Jan 2017 06:08:10 -0800 (PST) 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:in-reply-to; bh=NElsq3xJjqtawzblATepBk+loN73KMBAh8Ojkh2yfuQ=; b=wBeUft+uzvvzO/fuYge5ySzlArTbykOPpLE0K0Qjlqh54pTlDyJ8PlfPnwn/w7Q6YJ cMl3ZLUbLCvpfZEXF1ROnjn86p1eWASA3rgVQE0rq1iUMlMbADbMbdbG5eMPr1c4ZDzt PVxVcwIRF16adXOt/K5SC5JLM1v567K7G/TSOybFj2JkI8mVXSdrXy1de1f/97i9QH+u 2oks5FOA98lbvbFj2Uzv7e5IR3h/EiKq4woz8IEfuWWlxfapU04BBNinO5i2PdrCq0ZK TsgNCOZrSMNHTq9SfpW0+5MUb9Y9XqMi/JfceIo0wOeYn6O2pp3jTSh3Rrdg2+Qo7FfT PnNw== 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:in-reply-to; bh=NElsq3xJjqtawzblATepBk+loN73KMBAh8Ojkh2yfuQ=; b=KOZ2+aym5rJahq5H1xmyNPV1w0nzcunQCAO1h1jE1HhtYmc/RTXudLWOhLrXXIhau1 eTgvrHdnXTdTU4yY49eJmAlmL4nCTOGWmL+nG2hYPtLgTPqKNoNc3D8jXYiUcAFfMwrl 35AyW2z9zs9jnTbyNyPNv9Wd4B6pxJ9GDUOa2hCqFV8Igsf3cF0I6rt1N19XJVsnnbj8 JxXR00rubeZfdoo/LicJVwCEBRrBD32p5RIoXwEGxHxcmBKCBvY9pIXkOeYWiDqp9bu7 5zGEvVTiEzNzVDCKi3tnf8DfMqT0OPMQ8jGUjvChPnCv7XDuktcCLPXwESUN41UBdB4h /spw== X-Gm-Message-State: AIkVDXJNKISt5zqPFBaGQL7M3LkMkxZE5yYTxJPqhLkNAxn4KIA7rQ/5p3Y6kTl25TkiwSQJ X-Received: by 10.194.158.100 with SMTP id wt4mr66399111wjb.148.1483452490098; Tue, 03 Jan 2017 06:08:10 -0800 (PST) Received: from 6wind.com (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id g73sm89927116wme.16.2017.01.03.06.08.08 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 03 Jan 2017 06:08:09 -0800 (PST) Date: Tue, 3 Jan 2017 15:07:59 +0100 From: Adrien Mazarguil To: Wei Zhao Cc: dev@dpdk.org, Wenzhuo Lu Message-ID: <20170103140759.GA12822@6wind.com> References: <1483084390-53159-1-git-send-email-wei.zhao1@intel.com> <1483084390-53159-15-git-send-email-wei.zhao1@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1483084390-53159-15-git-send-email-wei.zhao1@intel.com> Subject: Re: [dpdk-dev] [PATCH v2 14/18] net/ixgbe: parse L2 tunnel filter 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, 03 Jan 2017 14:08:10 -0000 Hi Wei, On Fri, Dec 30, 2016 at 03:53:06PM +0800, Wei Zhao wrote: > check if the rule is a L2 tunnel rule, and get the L2 tunnel info. > > Signed-off-by: Wei Zhao > Signed-off-by: Wenzhuo Lu > > --- > > v2: > --add new error set function > --change return value type of parser function > --- > drivers/net/ixgbe/ixgbe_ethdev.c | 269 +++++++++++++++++++++++++++++++++++---- > lib/librte_ether/rte_flow.h | 32 +++++ > 2 files changed, 273 insertions(+), 28 deletions(-) [...] > diff --git a/lib/librte_ether/rte_flow.h b/lib/librte_ether/rte_flow.h > index 98084ac..e9e6220 100644 > --- a/lib/librte_ether/rte_flow.h > +++ b/lib/librte_ether/rte_flow.h > @@ -268,6 +268,13 @@ enum rte_flow_item_type { > * See struct rte_flow_item_vxlan. > */ > RTE_FLOW_ITEM_TYPE_VXLAN, > + > + /** > + * Matches a E_TAG header. > + * > + * See struct rte_flow_item_e_tag. > + */ > + RTE_FLOW_ITEM_TYPE_E_TAG, > }; > > /** > @@ -454,6 +461,31 @@ struct rte_flow_item_vxlan { > }; > > /** > + * RTE_FLOW_ITEM_TYPE_E_TAG. > + * > + * Matches a E-tag header. > + */ > +struct rte_flow_item_e_tag { > + struct ether_addr dst; /**< Destination MAC. */ > + struct ether_addr src; /**< Source MAC. */ > + uint16_t e_tag_ethertype; /**< E-tag EtherType, 0x893F. */ > + uint16_t e_pcp:3; /**< E-PCP */ > + uint16_t dei:1; /**< DEI */ > + uint16_t in_e_cid_base:12; /**< Ingress E-CID base */ > + uint16_t rsv:2; /**< reserved */ > + uint16_t grp:2; /**< GRP */ > + uint16_t e_cid_base:12; /**< E-CID base */ > + uint16_t in_e_cid_ext:8; /**< Ingress E-CID extend */ > + uint16_t e_cid_ext:8; /**< E-CID extend */ > + uint16_t type; /**< MAC type. */ > + unsigned int tags; /**< Number of 802.1Q/ad tags defined. */ > + struct { > + uint16_t tpid; /**< Tag protocol identifier. */ > + uint16_t tci; /**< Tag control information. */ > + } tag[]; /**< 802.1Q/ad tag definitions, outermost first. */ > +}; [...] See my previous reply [1], this definition is not endian-safe and comprises protocols defined as independent items (namely ETH and VLAN). Here is an untested suggestion: struct rte_flow_item_e_tag { uint16_t tpid; /**< Tag protocol identifier (0x893F). */ /** E-Tag control information (E-TCI). */ uint16_t epcp_edei_in_ecid_b; /**< E-PCP (3b), E-DEI (1b), ingress E-CID base (12b). */ uint16_t rsvd_grp_ecid_b; /**< Reserved (2b), GRP (2b), E-CID base (12b). */ uint8_t in_ecid_e; /**< Ingress E-CID ext. */ uint8_t ecid_e; /**< E-CID ext. */ }; Applications are responsibile for breaking down and filling individual fields properly. Ethernet header would be provided as its own item as shown in this testpmd flow command example: flow create 0 ingress pattern eth / e_tag in_ecid_base is 42 / end actions drop / end Note, all multibyte values are in network order like other protocol header definitions. [1] http://dpdk.org/ml/archives/dev/2016-December/053181.html Message ID: 20161223081310.GH10340@6wind.com -- Adrien Mazarguil 6WIND