From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH v2 2/3] l2fwd: keep alive sample application Date: Tue, 27 Oct 2015 10:10:23 +0000 Message-ID: <20151027101023.GA9176@bricha3-MOBL3> References: <1443603881-4700-1-git-send-email-remy.horton@intel.com> <1443603881-4700-3-git-send-email-remy.horton@intel.com> <1A27633A6DA49C4A92FCD5D4312DBF536A5D9327@IRSMSX109.ger.corp.intel.com> <562F3A69.1040306@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "dev@dpdk.org" To: Remy Horton Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 8E3E05A65 for ; Tue, 27 Oct 2015 11:10:27 +0100 (CET) Content-Disposition: inline In-Reply-To: <562F3A69.1040306@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Oct 27, 2015 at 08:48:41AM +0000, Remy Horton wrote: > > > On 23/10/2015 15:23, Tahhan, Maryam wrote: > > >> # binary name > >>-APP = l2fwd > >>+APP = l2fwd-keepalive > >Hi Remy, > >Looks great overall, just a couple of comments > >For backward compatibility the rename could be a nightmare? What do you think? > > True. Undecided whether to change the Makefile(s) so that both keepalive and > non-keepalive l2fwd are built, or to simply clone all of l2fwd.. > My opinion is that l2fwd should be kept as simple as possible, so I'd suggest cloning to provide a separate keepalive example. /Bruce