Re: [dpdk-dev] [PATCH v2 0/4] Remove PCI and VDEV dependency from eventdev library

["Gaëtan Rivet" <gaetan.rivet@6wind.com>]

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 <inttypes.h>
#include <rte_debug.h>

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.

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