From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH 1/4] net/virtio: fix the control vq support Date: Wed, 23 Jan 2019 23:03:51 +0100 Message-ID: <6b9f587a-dd1d-07bf-8ffa-5528045c9800@redhat.com> References: <20190122170143.5650-1-tiwei.bie@intel.com> <20190122170143.5650-2-tiwei.bie@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit To: Tiwei Bie , zhihong.wang@intel.com, dev@dpdk.org Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id B68952B87 for ; Wed, 23 Jan 2019 23:03:55 +0100 (CET) In-Reply-To: <20190122170143.5650-2-tiwei.bie@intel.com> Content-Language: en-US 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 1/22/19 6:01 PM, Tiwei Bie wrote: > This patch mainly fixed below issues in the packed ring based > control vq support in virtio driver: > > 1. When parsing the used descriptors, we have to track the > number of descs that we need to skip; > 2. vq->vq_free_cnt was decreased twice for a same desc; > > Meanwhile, make the function name consistent with other parts. > > Fixes: ec194c2f1895 ("net/virtio: support packed queue in send command") > Fixes: a4270ea4ff79 ("net/virtio: check head desc with correct wrap counter") > > Signed-off-by: Tiwei Bie > --- > drivers/net/virtio/virtio_ethdev.c | 62 ++++++++++++++---------------- > drivers/net/virtio/virtqueue.h | 12 +----- > 2 files changed, 31 insertions(+), 43 deletions(-) > Reviewed-by: Maxime Coquelin Thanks, Maxime