From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v17 0/5] User-space Ethtool Date: Fri, 17 Jul 2015 00:09:01 +0200 Message-ID: <1922905.J0eqSKxbIu@xps13> References: <1432946276-9424-1-git-send-email-liang-min.wang@intel.com> <3483104.M20H0g0coM@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Wang, Liang-min" Return-path: Received: from mail-wi0-f173.google.com (mail-wi0-f173.google.com [209.85.212.173]) by dpdk.org (Postfix) with ESMTP id 0507D5A58 for ; Fri, 17 Jul 2015 00:10:14 +0200 (CEST) Received: by widic2 with SMTP id ic2so26011003wid.0 for ; Thu, 16 Jul 2015 15:10:13 -0700 (PDT) In-Reply-To: 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" 2015-07-16 21:55, Wang, Liang-min: > Thomas, > Do you want me to create a separate patch just include the example/l2fwd-ethtool? Yes > Do you also mean besides the identified Makefiles, you see more rework needs to be done, > or I just need to fix Makefile issue? If just Makefile issue, I could try to make another attempt tomorrow? After checking the build there is probably more review to do. Let's take more time to have something clean and maybe more complete in 2.2. This patchset is your first contribution to DPDK and is already a nice achievement. The new API must now be implemented in more drivers to be effective. > > -----Original Message----- > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > Sent: Thursday, July 16, 2015 5:48 PM > > To: Wang, Liang-min > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v17 0/5] User-space Ethtool > > > > 2015-07-16 09:25, Liang-Min Larry Wang: > > > This implementation is designed to provide a familar interface for > > applications that rely on kernel-space driver to support ethtool_op and > > net_device_op for device management. The initial implementation focuses > > on ops that can be implemented through existing netdev APIs. More ops will > > be supported in latter release. > > > > Applied without example which needs more work, thanks