From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shreyansh Jain Subject: Re: [PATCH v9 22/25] eal/pci: inherit rte_driver by rte_pci_driver Date: Fri, 16 Sep 2016 10:17:58 +0530 Message-ID: References: <1466510566-9240-1-git-send-email-shreyansh.jain@nxp.com> <1473257297-7221-1-git-send-email-shreyansh.jain@nxp.com> <1473257297-7221-23-git-send-email-shreyansh.jain@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Cc: , Jan Viktorin To: Ferruh Yigit , Return-path: Received: from NAM02-CY1-obe.outbound.protection.outlook.com (mail-cys01nam02on0081.outbound.protection.outlook.com [104.47.37.81]) by dpdk.org (Postfix) with ESMTP id 9CD3A689B for ; Fri, 16 Sep 2016 06:47:36 +0200 (CEST) In-Reply-To: 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" On Thursday 08 September 2016 07:55 PM, Ferruh Yigit wrote: > On 9/7/2016 3:08 PM, Shreyansh Jain wrote: >> Remove the 'name' member from rte_pci_driver and move to generic rte_driver. >> >> Most of the PMD drivers were initially using DRIVER_REGISTER_PCI(..) >> as well as assigning a name to eth_driver.pci_drv.name member. >> In this patch, only the original DRIVER_REGISTER_PCI(..) name has been >> populated into the rte_driver.name member - assignments through eth_driver >> has been removed. >> >> Signed-off-by: Jan Viktorin >> Signed-off-by: Shreyansh Jain > > .... > >> diff --git a/lib/librte_eal/linuxapp/eal/rte_eal_version.map b/lib/librte_eal/linuxapp/eal/rte_eal_version.map >> index c0bd391..b8bfd4b 100644 >> --- a/lib/librte_eal/linuxapp/eal/rte_eal_version.map >> +++ b/lib/librte_eal/linuxapp/eal/rte_eal_version.map >> @@ -172,4 +172,7 @@ DPDK_16.11 { >> >> rte_eal_dev_attach; >> rte_eal_dev_detach; >> + rte_eal_vdrv_register; >> + rte_eal_vdrv_unregister; >> + >> } DPDK_16.07; > > This needs to be part of patch 15, where these functions implemented. > > Missing these in .map files cause patch 17,18,19,20,21 fail to compile > for shared lib config. > Updated the map file in Patch 16 of v10 posted. Thanks for comment. -- - Shreyansh