From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id AD37129D2 for ; Wed, 23 Jan 2019 18:20:44 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 23 Jan 2019 09:20:43 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,512,1539673200"; d="scan'208";a="140684408" Received: from irsmsx103.ger.corp.intel.com ([163.33.3.157]) by fmsmga001.fm.intel.com with ESMTP; 23 Jan 2019 09:20:42 -0800 Received: from irsmsx105.ger.corp.intel.com ([169.254.7.116]) by IRSMSX103.ger.corp.intel.com ([169.254.3.60]) with mapi id 14.03.0415.000; Wed, 23 Jan 2019 17:20:41 +0000 From: "Ananyev, Konstantin" To: Thomas Monjalon , "Yigit, Ferruh" CC: Ravi Kerur , dpdk-dev , "Mcnamara, John" Thread-Topic: [dpdk-dev] [PATCH v6 0/3] examples/l3fwd: merge l3fwd-acl code into l3fwd Thread-Index: AQHUszlLa02nzV1TckmyzleQeK7em6W9FqMAgAABK0A= Date: Wed, 23 Jan 2019 17:20:40 +0000 Message-ID: <2601191342CEEE43887BDE71AB977258010D907202@irsmsx105.ger.corp.intel.com> References: <1489008729-28784-2-git-send-email-rkerur@gmail.com> <1489179491-9401-1-git-send-email-rkerur@gmail.com> <26db3c19-9321-ebd0-c732-54a05ed15a54@intel.com> <2510395.XNuST6rEQr@xps> In-Reply-To: <2510395.XNuST6rEQr@xps> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMWI3ZmRkOTgtM2U0Ni00NGYyLTgzMjYtZmZjN2RiZmMxZmZjIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiQWxPVTZHaHF5MVZtSEs0VDE3VmpWQ2NtaWozenpyK3FKNlczSlNLcHl0N0crck50Q1RTamltejhpK3JnY1wvbVUifQ== x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v6 0/3] examples/l3fwd: merge l3fwd-acl code into l3fwd 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: Wed, 23 Jan 2019 17:20:45 -0000 > -----Original Message----- > From: Thomas Monjalon [mailto:thomas@monjalon.net] > Sent: Wednesday, January 23, 2019 5:11 PM > To: Yigit, Ferruh ; Ananyev, Konstantin > Cc: Ravi Kerur ; dpdk-dev ; Mcnamara, Joh= n > Subject: Re: [dpdk-dev] [PATCH v6 0/3] examples/l3fwd: merge l3fwd-acl co= de into l3fwd >=20 > 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 o= f 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/ >=20 > This work was going in the right direction. >=20 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= . Konstantin=20