From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id B4132378B for ; Sun, 9 Oct 2016 08:41:58 +0200 (CEST) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga105.jf.intel.com with ESMTP; 08 Oct 2016 23:41:57 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,316,1473145200"; d="scan'208";a="1062378616" Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.67.162]) by orsmga002.jf.intel.com with ESMTP; 08 Oct 2016 23:41:56 -0700 Date: Sun, 9 Oct 2016 14:42:49 +0800 From: Yuanhan Liu To: Shreyansh Jain Cc: David Marchand , thomas.monjalon@6wind.com, dev@dpdk.org, huawei.xie@intel.com Message-ID: <20161009064249.GO1597@yliu-dev.sh.intel.com> References: <1475845393-25670-1-git-send-email-david.marchand@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH] net/virtio: add missing driver name X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 09 Oct 2016 06:41:59 -0000 On Fri, Oct 07, 2016 at 06:57:41PM +0530, Shreyansh Jain wrote: > On Friday 07 October 2016 06:33 PM, David Marchand wrote: > >The driver name has been lost with the eal rework. > >Restore it. > > > >Fixes: c830cb295411 ("drivers: use PCI registration macro") > > > >Signed-off-by: David Marchand > >--- > > drivers/net/virtio/virtio_ethdev.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > >diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c > >index b4dfc0a..809ebf7 100644 > >--- a/drivers/net/virtio/virtio_ethdev.c > >+++ b/drivers/net/virtio/virtio_ethdev.c > >@@ -1304,6 +1304,9 @@ eth_virtio_dev_uninit(struct rte_eth_dev *eth_dev) > > > > static struct eth_driver rte_virtio_pmd = { > > .pci_drv = { > >+ .driver = { > >+ .name = "net_virtio", > >+ }, > > .id_table = pci_id_virtio_map, > > .drv_flags = RTE_PCI_DRV_DETACHABLE, > > .probe = rte_eth_dev_pci_probe, > > > > Indeed. For MLX4/5 and Virtio, DRIVER_REGISTER_* is not being used which > takes this 'name'. In this case, it was missed. > > Acked-by: Shreyansh Jain Applied to dpdk-next-virtio. Thanks. --yliu