From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EX13-EDG-OU-001.vmware.com (ex13-edg-ou-001.vmware.com [208.91.0.189]) by dpdk.org (Postfix) with ESMTP id 17D1DFB04 for ; Mon, 6 Mar 2017 18:25:13 +0100 (CET) Received: from sc9-mailhost3.vmware.com (10.113.161.73) by EX13-EDG-OU-001.vmware.com (10.113.208.155) with Microsoft SMTP Server id 15.0.1156.6; Mon, 6 Mar 2017 09:24:26 -0800 Received: from shri-linux.eng.vmware.com (shri-linux.eng.vmware.com [10.33.72.16]) by sc9-mailhost3.vmware.com (Postfix) with ESMTP id 3B7A140478; Mon, 6 Mar 2017 09:25:12 -0800 (PST) Date: Mon, 6 Mar 2017 09:25:12 -0800 From: Shrikrishna Khare To: Ferruh Yigit CC: Shrikrishna Khare , , In-Reply-To: <8019d737-e839-b5e9-53ef-3717db6ab5b3@intel.com> Message-ID: References: <1488059989-6930-1-git-send-email-skhare@vmware.com> <8019d737-e839-b5e9-53ef-3717db6ab5b3@intel.com> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Received-SPF: None (EX13-EDG-OU-001.vmware.com: skhare@shri-linux.eng.vmware.com does not designate permitted sender hosts) Subject: Re: [dpdk-dev] [PATCH 0/7] vmxnet3: upgrade to version 3 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: Mon, 06 Mar 2017 17:25:13 -0000 On Fri, 3 Mar 2017, Ferruh Yigit wrote: > > Shrikrishna Khare (7): > > vmxnet3: prepare for version 3 changes > > vmxnet3: introduce generalized command interface to configure the > > device > > vmxnet3: allow variable length transmit data ring buffer > > vmxnet3: add receive data ring support > > vmxnet3: add reserved version 3 command > > vmxnet3: introduce command to register memory region > > vmxnet3: update to version 3 > > Hi Shrikrishna, > > Can you please update release notes with this change with one or two > sentences in next version of the patchset? > > Also there are helper scripts in dpdk: > - dpdk/devtools/check-git-log.sh to check patch titles > - dpdk/devtools/checkpatches.sh (a wrapper to Linux checkpatch.pl) > > Both are generating some warnings, can you please check them? > > For checkpatches script, it is free to ignore CAMELCASE & > LONG_LINE_STRING warnings. Hi Ferruh, Thank you for the review comments. I have addressed them all and sent v2 patches for review. In addition to CAMELCASE, there are few LEADING_SPACE warnings that I have not addressed. This is an existing problem with all the vmxnet3 headers and only fixing it for the lines added by this patch series will be odd. Hope that is OK. Perhaps a separate patch can fix these style issues in vmxnet3 header files all at once. Thanks, Shri > > Thanks, > ferruh > > > > > > > > drivers/net/vmxnet3/base/vmxnet3_defs.h | 85 ++++++++++++++++-- > > drivers/net/vmxnet3/vmxnet3_ethdev.c | 154 +++++++++++++++++++++++++++++++- > > drivers/net/vmxnet3/vmxnet3_ethdev.h | 14 +++ > > drivers/net/vmxnet3/vmxnet3_ring.h | 21 +++++ > > drivers/net/vmxnet3/vmxnet3_rxtx.c | 45 ++++++++-- > > 5 files changed, 304 insertions(+), 15 deletions(-) > > > >