From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v2 0/4] Remove PCI and VDEV dependency from eventdev library Date: Thu, 8 Jun 2017 22:35:48 +0530 Message-ID: <20170608170547.GA15360@jerin> References: <20170606141046.32103-1-jerin.jacob@caviumnetworks.com> <20170607084333.6338-1-jerin.jacob@caviumnetworks.com> <20170607092726.GO18840@bidouze.vm.6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Cc: dev@dpdk.org, bruce.richardson@intel.com, harry.van.haaren@intel.com, hemant.agrawal@nxp.com, gage.eads@intel.com, nipun.gupta@nxp.com, narender.vangati@intel.com, nikhil.rao@intel.com To: =?iso-8859-1?Q?Ga=EBtan?= Rivet Return-path: Received: from NAM02-BL2-obe.outbound.protection.outlook.com (mail-bl2nam02on0088.outbound.protection.outlook.com [104.47.38.88]) by dpdk.org (Postfix) with ESMTP id 3BED42BB9 for ; Thu, 8 Jun 2017 19:06:31 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20170607092726.GO18840@bidouze.vm.6wind.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" -----Original Message----- > Date: Wed, 7 Jun 2017 11:27:26 +0200 > From: Gaëtan Rivet > To: Jerin Jacob > Cc: dev@dpdk.org, bruce.richardson@intel.com, harry.van.haaren@intel.com, > hemant.agrawal@nxp.com, gage.eads@intel.com, nipun.gupta@nxp.com, > narender.vangati@intel.com, nikhil.rao@intel.com > Subject: Re: [dpdk-dev] [PATCH v2 0/4] Remove PCI and VDEV dependency from > eventdev library > User-Agent: Mutt/1.5.23 (2014-03-12) > > Hi Jerin, > > On Wed, Jun 07, 2017 at 02:13:29PM +0530, Jerin Jacob wrote: > > v3: > > Removed the bus dependency by spliting lib/librte_eventdev/rte_eventdev_pmd_pci.h > > and lib/librte_eventdev/rte_eventdev_pmd_vdev.h > > > > It works with the PCI bus now, good job. > Small problem however: to compile drivers/event/sw/sw_evdev.c, two includes were > missing: > > #include > #include > > I added both to rte_eventdev_pmd_vdev.h to quickly fix this and test the > compilation otherwise, but maybe this is only necessary for sw_evdev.c. I tested x86_64-native-linuxapp-gcc + dpdk.org master on following versions of gcc. I couldn't see the compilation issue. Anyway, I will send the next version with suggested header file changes. gcc version 4.9.2 20150212 (Red Hat 4.9.2-6) (GCC) gcc version 4.8.3 20140911 (Red Hat 4.8.3-7) (GCC) gcc version 7.1.1 20170528 (GCC) > > Thanks > > > Jerin Jacob (4): > > eventdev: remove PCI dependency from generic data structures > > eventdev: restructure event PMD release function > > eventdev: make PCI probe and remove functions optional > > eventdev: make vdev init and uninit functions optional > > > > drivers/event/octeontx/ssovf_evdev.h | 2 +- > > drivers/event/octeontx/ssovf_worker.h | 1 + > > drivers/event/skeleton/skeleton_eventdev.c | 33 +++--- > > drivers/event/skeleton/skeleton_eventdev.h | 3 +- > > drivers/event/sw/sw_evdev.h | 2 +- > > lib/librte_eventdev/Makefile | 2 + > > lib/librte_eventdev/rte_eventdev.c | 148 ------------------------- > > lib/librte_eventdev/rte_eventdev.h | 2 - > > lib/librte_eventdev/rte_eventdev_pmd.h | 101 +----------------- > > lib/librte_eventdev/rte_eventdev_pmd_pci.h | 160 ++++++++++++++++++++++++++++ > > lib/librte_eventdev/rte_eventdev_pmd_vdev.h | 133 +++++++++++++++++++++++ > > 11 files changed, 323 insertions(+), 264 deletions(-) > > create mode 100644 lib/librte_eventdev/rte_eventdev_pmd_pci.h > > create mode 100644 lib/librte_eventdev/rte_eventdev_pmd_vdev.h > > > > -- > > 2.13.0 > > > > -- > Gaëtan Rivet > 6WIND