From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Gatto Subject: [PATCH] virtio: remove obsolete comment Date: Fri, 2 Sep 2016 17:05:21 +0200 Message-ID: <20160902150521.680-1-matthias.gatto@outscale.com> Cc: Matthias Gatto , huawei.xie@intel.com, yuanhan.liu@linux.intel.com To: dev@dpdk.org Return-path: Received: from dv-adm01.admin (ows-185-21-195-233.eu-west-1.compute.outscale.com [185.21.195.233]) by dpdk.org (Postfix) with ESMTP id D6D6B4CE7 for ; Fri, 2 Sep 2016 17:05:30 +0200 (CEST) List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" As new_device and destroy_device use an int instead of a "struct virtio_net *", The comment about setting VIRTIO_DEV_RUNNING doesn't make sense anymore, plus If I've correctly understand the code, the drivers take care of setting the flag before calling the callbacks, so I guess that this comment is obsolet and I've remove it. Signed-off-by: Matthias Gatto --- lib/librte_vhost/rte_virtio_net.h | 4 ---- 1 file changed, 4 deletions(-) diff --git a/lib/librte_vhost/rte_virtio_net.h b/lib/librte_vhost/rte_virtio_net.h index 9caa622..a88aecd 100644 --- a/lib/librte_vhost/rte_virtio_net.h +++ b/lib/librte_vhost/rte_virtio_net.h @@ -59,10 +59,6 @@ enum {VIRTIO_RXQ, VIRTIO_TXQ, VIRTIO_QNUM}; /** * Device and vring operations. - * - * Make sure to set VIRTIO_DEV_RUNNING to the device flags in new_device and - * remove it in destroy_device. - * */ struct virtio_net_device_ops { int (*new_device)(int vid); /**< Add device. */ -- 2.9.3