From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH v6 0/3] examples/l3fwd: merge l3fwd-acl code
 into l3fwd
Date: Wed, 23 Jan 2019 19:36:09 +0000
Message-ID: <cafdfdfb-c321-9a93-8a40-a9f4f65b8e64@intel.com>
References: <1489008729-28784-2-git-send-email-rkerur@gmail.com>
 <2510395.XNuST6rEQr@xps>
 <2601191342CEEE43887BDE71AB977258010D907202@irsmsx105.ger.corp.intel.com>
 <2922307.MMPdoxV78S@xps>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Cc: Ravi Kerur <rkerur@gmail.com>, dpdk-dev <dev@dpdk.org>,
 "Mcnamara, John" <john.mcnamara@intel.com>
To: Thomas Monjalon <thomas@monjalon.net>,
 "Ananyev, Konstantin" <konstantin.ananyev@intel.com>,
 Stephen Hemminger <stephen@networkplumber.org>
Return-path: <dev-bounces@dpdk.org>
Received: from mga07.intel.com (mga07.intel.com [134.134.136.100])
 by dpdk.org (Postfix) with ESMTP id 77D102B87
 for <dev@dpdk.org>; Wed, 23 Jan 2019 20:36:12 +0100 (CET)
In-Reply-To: <2922307.MMPdoxV78S@xps>
Content-Language: en-US
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On 1/23/2019 5:26 PM, Thomas Monjalon wrote:
> 23/01/2019 18:20, Ananyev, Konstantin:
>> From: Thomas Monjalon [mailto:thomas@monjalon.net]
>>> 23/01/2019 17:32, Ferruh Yigit:
>>>> On 3/10/2017 8:58 PM, rkerur at gmail.com (Ravi Kerur) wrote:
>>>>> This patchset merges l3fwd-acl and l3fwd code into common directory.
>>>>> Adds config file read option to build LPM and EM tables.
>>>>>
>>>>> Ravi Kerur (3):
>>>>>   examples/l3fwd: merge l3fwd-acl code into l3fwd
>>>>>   examples/l3fwd: add config file support for lpm
>>>>>   examples/l3fwd: add config file support for exact
>>>>
>>>> Hi Ravi,
>>>>
>>>> These l3fwd patches are in patchwork for a long time, I am updating the patchset
>>>> as rejected, if it is still relevant please send a new version on top of latest
>>>> repo.
>>>>
>>>> Sorry for any inconvenience caused.
>>>>
>>>> For reference patches:
>>>> https://patches.dpdk.org/patch/21696/
>>>> https://patches.dpdk.org/patch/21695/
>>>> https://patches.dpdk.org/patch/21697/
>>>>
>>>> doc one:
>>>> https://patches.dpdk.org/patch/24211/
>>>
>>> This work was going in the right direction.
>>>
>>
>> Totally agree.
>>
>>> Konstantin, as the maintainer of the ACL library,
>>> do you think it is worth to keep this example as standalone or merged?
>>
>> My vote is definitely for merging.
>> That would give us single l3fwd app with 3 different routing methods
>> (lpm, hash, acl) selectable at run-time, plus routing tables in config file.
> 
> OK, so we just need to find a volunteer.

There was a "Nice to have - Future" section in Roadmap webpage [1], does it help
putting there?

Also we talked about GSOC recently, can this be an item for it?


[1]
http://core.dpdk.org/roadmap/#future

> Ravi started the work a long time ago and did not receive enough review.
> I'm afraid he's not available anymore.
> 
>