From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3 0/7] ethdev: add flow API object converter Date: Wed, 03 Oct 2018 22:31:33 +0200 Message-ID: <2339995.YGgu5CYcxV@xps> References: <20180803132032.29038-1-adrien.mazarguil@6wind.com> <20180831085337.21419-1-adrien.mazarguil@6wind.com> <20180831113259.u5rceja7zpx5i27w@laranjeiro-vm.dev.6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Cc: dev@dpdk.org, =?ISO-8859-1?Q?N=E9lio?= Laranjeiro , Adrien Mazarguil To: Ferruh Yigit Return-path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id C4EC41B137 for ; Wed, 3 Oct 2018 22:31:35 +0200 (CEST) In-Reply-To: <20180831113259.u5rceja7zpx5i27w@laranjeiro-vm.dev.6wind.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" 31/08/2018 13:32, N=E9lio Laranjeiro: > On Fri, Aug 31, 2018 at 11:00:57AM +0200, Adrien Mazarguil wrote: > > This is a follow up to the "Flow API helpers enhancements" series submi= tted > > almost a year ago [1]. The new title is due to the reduced scope of this > > version. > >=20 > > rte_flow_conv() is a flexible replacement to rte_flow_copy(), itself a > > temporary solution pending something better [2]. It replaces a lot of > > duplicated code found in testpmd and removes some of the maintenance bu= rden > > that developers tend to forget (me included) when modifying pattern > > items or actions (updating app/test-pmd/config.c to be clear). > >=20 > > This series was unearthed in order to complete the implementation of > > RTE_FLOW_ACTION_TYPE_ENCAP_(VXLAN|NVGRE) in testpmd [3] without having = to > > duplicate existing code once again. >=20 > Acked-by: Nelio Laranjeiro No more comment after one month. =46erruh, I think it should go into next-net.