From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Duszynski Subject: Re: [PATCH 5/8] net/mrvl: add classifier support Date: Thu, 8 Mar 2018 14:23:07 +0100 Message-ID: <20180308132307.GA22443@sh> References: <1519222460-14605-1-git-send-email-tdu@semihalf.com> <1519222460-14605-6-git-send-email-tdu@semihalf.com> <80a8f381-3580-cc2a-b3ba-b52407c6e0d9@intel.com> <20180307111606.GA13388@sh> <8cc0e48e-217b-74ae-a469-00ea05853f80@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Cc: Tomasz Duszynski , dev@dpdk.org, mw@semihalf.com, dima@marvell.com, nsamsono@marvell.com, jck@semihalf.com To: Ferruh Yigit Return-path: Received: from mail-lf0-f66.google.com (mail-lf0-f66.google.com [209.85.215.66]) by dpdk.org (Postfix) with ESMTP id E8ADF4CBD for ; Thu, 8 Mar 2018 14:23:09 +0100 (CET) Received: by mail-lf0-f66.google.com with SMTP id h127-v6so8242011lfg.12 for ; Thu, 08 Mar 2018 05:23:09 -0800 (PST) Content-Disposition: inline In-Reply-To: <8cc0e48e-217b-74ae-a469-00ea05853f80@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, Mar 07, 2018 at 11:24:11AM +0000, Ferruh Yigit wrote: > On 3/7/2018 11:16 AM, Tomasz Duszynski wrote: > > On Wed, Mar 07, 2018 at 11:07:14AM +0000, Ferruh Yigit wrote: > >> On 2/21/2018 2:14 PM, Tomasz Duszynski wrote: > >>> Add classifier configuration support via rte_flow api. > >>> > >>> Signed-off-by: Natalie Samsonov > >>> Signed-off-by: Tomasz Duszynski > >>> --- > >>> doc/guides/nics/mrvl.rst | 168 +++ > >>> drivers/net/mrvl/Makefile | 1 + > >>> drivers/net/mrvl/mrvl_ethdev.c | 59 + > >>> drivers/net/mrvl/mrvl_ethdev.h | 10 + > >>> drivers/net/mrvl/mrvl_flow.c | 2787 ++++++++++++++++++++++++++++++= ++++++++++ > >>> 5 files changed, 3025 insertions(+) > >>> create mode 100644 drivers/net/mrvl/mrvl_flow.c > >> > >> <...> > >> > >>> diff --git a/drivers/net/mrvl/mrvl_flow.c b/drivers/net/mrvl/mrvl_flo= w.c > >>> new file mode 100644 > >>> index 0000000..a2c25e6 > >>> --- /dev/null > >>> +++ b/drivers/net/mrvl/mrvl_flow.c > >>> @@ -0,0 +1,2787 @@ > >>> +/*- > >>> + * BSD LICENSE > >>> + * > >>> + * Copyright(c) 2018 Marvell International Ltd. > >>> + * Copyright(c) 2018 Semihalf. > >>> + * All rights reserved. > >>> + * > >>> + * Redistribution and use in source and binary forms, with or with= out > >>> + * modification, are permitted provided that the following conditi= ons > >>> + * are met: > >>> + * > >>> + * * Redistributions of source code must retain the above copyri= ght > >>> + * notice, this list of conditions and the following disclaime= r. > >>> + * * Redistributions in binary form must reproduce the above cop= yright > >>> + * notice, this list of conditions and the following disclaime= r in > >>> + * the documentation and/or other materials provided with the > >>> + * distribution. > >>> + * * Neither the name of the copyright holder nor the names of i= ts > >>> + * contributors may be used to endorse or promote products der= ived > >>> + * from this software without specific prior written permissio= n. > >>> + * > >>> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBU= TORS > >>> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT N= OT > >>> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNE= SS FOR > >>> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPY= RIGHT > >>> + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCID= ENTAL, > >>> + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT > >>> + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS O= F USE, > >>> + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND = ON ANY > >>> + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR = TORT > >>> + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF T= HE USE > >>> + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DA= MAGE. > >>> + */ > >> > >> Can you please use SPDX licensing tags for new files? > >> > >> And marvell PMD seems not switched to SPDX tags yet, can you please pl= an the switch? > > > > SPDX conversion patches are waiting for submission. Once Marvell > > gives final approval they will be pushed out. > > Good, thank you. Will it be possible to send this set with SPDX license? OK, so let's postpone this until SPDX license conversion gets fully resolved. > > > > >> > >> Thanks, > >> ferruh > > > > -- > > - Tomasz Duszy=C5=84ski > > > -- - Tomasz Duszy=C5=84ski