From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 811D01C52 for ; Thu, 3 Mar 2016 11:12:01 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 03 Mar 2016 02:12:00 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,532,1449561600"; d="scan'208";a="900219342" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.130]) ([10.237.220.130]) by orsmga001.jf.intel.com with ESMTP; 03 Mar 2016 02:11:57 -0800 To: Jay Rolette , Stephen Hemminger References: <1453911849-16562-1-git-send-email-ferruh.yigit@intel.com> <56D420E5.9010802@intel.com> <56D42462.3020905@scylladb.com> <1945473.Tiatd2m80T@xps13> <20160301180255.3ef1a17e@xeon-e3> From: Ferruh Yigit X-Enigmail-Draft-Status: N1110 Message-ID: <56D80DED.8040909@intel.com> Date: Thu, 3 Mar 2016 10:11:57 +0000 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: DPDK , Avi Kivity Subject: Re: [dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Mar 2016 10:12:01 -0000 On 3/2/2016 10:18 PM, Jay Rolette wrote: > > On Tue, Mar 1, 2016 at 8:02 PM, Stephen Hemminger > > wrote: > > On Mon, 29 Feb 2016 08:33:25 -0600 > Jay Rolette > > wrote: > > > On Mon, Feb 29, 2016 at 5:06 AM, Thomas Monjalon > > > > wrote: > > > > > Hi, > > > I totally agree with Avi's comments. > > > This topic is really important for the future of DPDK. > > > So I think we must give some time to continue the discussion > > > and have netdev involved in the choices done. > > > As a consequence, these series should not be merged in the > release 16.04. > > > Thanks for continuing the work. > > > > > > > I know you guys are very interested in getting rid of the out-of-tree > > drivers, but please do not block incremental improvements to DPDK > in the > > meantime. Ferruh's patch improves the usability of KNI. Don't > throw out > > good and useful enhancements just because it isn't where you want > to be in > > the end. > > > > I'd like to see these be merged. > > > > Jay > > The code is really not ready. I am okay with cooperative development > but the current code needs to go into a staging type tree. > No compatibility, no ABI guarantees, more of an RFC. > Don't want vendors building products with it then screaming when it > gets rebuilt/reworked/scrapped. > > > That's fair. To be clear, it wasn't my intent for code that wasn't baked > yet to be merged. > > The main point of my comment was that I think it is important not to > halt incremental improvements to existing capabilities (KNI in this > case) just because there are philosophical or directional changes that > the community would like to make longer-term. > > Bird in the hand vs. two in the bush... > There are two different statements, first, code being not ready, I agree a fair point (although there is no argument to that statement, it makes hard to discuss this, I will put aside this), this implies when code is ready it can go in to repo. But not having kernel module, independent from their state against what they are trying to replace is something else. And this won't help on KNI related problems. Thanks, ferruh