From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tiwei Bie Subject: Re: [PATCH v4 0/5] vhost: generalize buffer vectors Date: Mon, 9 Jul 2018 15:01:34 +0800 Message-ID: <20180709070133.GA5852@debian> References: <20180706070449.1946-1-maxime.coquelin@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Cc: zhihong.wang@intel.com, dev@dpdk.org To: Maxime Coquelin Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id F13902BF5 for ; Mon, 9 Jul 2018 09:01:48 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20180706070449.1946-1-maxime.coquelin@redhat.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Jul 06, 2018 at 09:04:44AM +0200, Maxime Coquelin wrote: > This series is again preliminray work to ease packed ring > layout integration. > > Main changes are using vector buffres also in the dequeue > path, and perform IOVA to HVA translation at vectors fill > time. > > I still have to run more benchmarks, but PVP benchmarks does > not show performance changes. > > Good thing is that it saves ~140 further lines. > > Changes since v3: > ================= > - Fix dequeue_zero_copy last_used_idx update (Tiwei) > - Remove "vhost: make gpa to hpa failure an error" patch (Tiwei) > > Changes since v2: > ================= > - check vec_id doesn't overflow (Tiwei) > - Fix perm parameters passed to fill_vec_buf (Tiwei) > - Remove extra space in variable assignation (Tiwei) > > > Maxime Coquelin (5): > vhost: use shadow used ring in dequeue path > vhost: use buffer vectors in dequeue path > vhost: improve prefetching in dequeue path > vhost: prefetch first descriptor in dequeue path > vhost: improve prefetching in enqueue path > > lib/librte_vhost/vhost.h | 1 + > lib/librte_vhost/virtio_net.c | 517 ++++++++++++++++-------------------------- > 2 files changed, 193 insertions(+), 325 deletions(-) > > -- > 2.14.4 > Applied to dpdk-next-virtio/master, thanks.