From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id BE1512C31 for ; Fri, 3 Mar 2017 11:57:38 +0100 (CET) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 03 Mar 2017 02:57:37 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.35,236,1484035200"; d="scan'208";a="231855567" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.255.205.116]) ([10.255.205.116]) by fmsmga004.fm.intel.com with ESMTP; 03 Mar 2017 02:57:33 -0800 To: Shrikrishna Khare , yongwang@vmware.com References: <1488059989-6930-1-git-send-email-skhare@vmware.com> Cc: dev@dpdk.org From: Ferruh Yigit Message-ID: <8019d737-e839-b5e9-53ef-3717db6ab5b3@intel.com> Date: Fri, 3 Mar 2017 10:57:32 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <1488059989-6930-1-git-send-email-skhare@vmware.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit 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: Fri, 03 Mar 2017 10:57:39 -0000 On 2/25/2017 9:59 PM, Shrikrishna Khare wrote: > vmxnet3 emulation has recently added several new features which includes > support for new commands the driver can issue to emulation, change in > descriptor fields etc. This patch series extends the vmxnet3 driver to > leverage these new features. > > Compatibility is maintained using existing vmxnet3 versioning mechanism as > follows: > - new features added to vmxnet3 emulation are associated with new vmxnet3 > version viz. vmxnet3 version 3. > - emulation advertises all the versions it supports to the driver. > - during initialization, vmxnet3 driver picks the highest version number > supported by both the emulation and the driver and configures emulation > to run at that version. > > In particular, following changes are introduced: > > Patch 1: > Trivial cleanup in preparation of version 3 changes. > > Patch 2: > This patch introduces generalized command interface which allows > for easily adding new commands that vmxnet3 driver can issue to the > emulation. Further patches in this series make use of this facility. > > Patch 3: > Transmit data ring buffer is used to copy packet headers or small > packets. It is a fixed size buffer. This patch extends the driver to > allow variable sized transmit data ring buffer. > > Patch 4: > This patch introduces receive data ring buffer - a set of small sized > buffers that are always mapped by the emulation. This avoids memory > mapping/unmapping overhead for small packets. > > Patch 5: > This patch adds reserved commands. > > Patch 6: > In vmxnet3 version 3, the emulation added support for the vmxnet3 driver > to communicate information about the memory regions the driver will use > for rx/tx buffers. This patch exposes related commands to the driver. The > driver is also extended to make use of this feaeture. > > Patch 7: > With all vmxnet3 version 3 changes incorporated in the vmxnet3 driver, > with this patch, the driver can configure emulation to run at vmxnet3 > version 3. > > 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. 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(-) >