From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Freimann Subject: Re: [PATCH 01/17] net/virtio: vring init for packed queues Date: Wed, 4 Apr 2018 09:48:27 +0200 Message-ID: <20180404074827.d7zqu36kp6d6fpnj@reserved-198-163.str.redhat.com> References: <20180316152120.13199-1-jfreimann@redhat.com> <20180316152120.13199-2-jfreimann@redhat.com> <58c7998d-e6c7-2785-79aa-65d475c9d41b@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Cc: dev@dpdk.org, tiwei.bie@intel.com, yliu@fridaylinux.org, mst@redhat.com To: Maxime Coquelin Return-path: Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by dpdk.org (Postfix) with ESMTP id 2137F1BC55 for ; Wed, 4 Apr 2018 09:48:30 +0200 (CEST) Content-Disposition: inline In-Reply-To: <58c7998d-e6c7-2785-79aa-65d475c9d41b@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 Wed, Apr 04, 2018 at 09:33:20AM +0200, Maxime Coquelin wrote: > > >On 03/16/2018 04:21 PM, Jens Freimann wrote: >>Add and initialize descriptor data structures. >> >>Signed-off-by: Jens Freimann >>--- >> drivers/net/virtio/virtio_ethdev.c | 22 ++++++++++++---------- >> drivers/net/virtio/virtio_pci.h | 8 ++++++++ >> drivers/net/virtio/virtio_ring.h | 25 +++++++++++++++++++++++-- >> drivers/net/virtio/virtqueue.h | 10 ++++++++++ >> 4 files changed, 53 insertions(+), 12 deletions(-) >> >>diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c >>index 884f74a..ce4f53d 100644 >>--- a/drivers/net/virtio/virtio_ethdev.c >>+++ b/drivers/net/virtio/virtio_ethdev.c >>@@ -299,19 +299,21 @@ struct rte_virtio_xstats_name_off { >> PMD_INIT_FUNC_TRACE(); >>- /* >>- * Reinitialise since virtio port might have been stopped and restarted >>- */ > >Are you sure the comment has to be removed? Fixed in v3. Thanks! regards, Jens