DPDK patches and discussions
 help / color / mirror / Atom feed
From: "Gaëtan Rivet" <gaetan.rivet@6wind.com>
To: "Doherty, Declan" <declan.doherty@intel.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v2 00/14] Move PCI away from the EAL
Date: Wed, 11 Oct 2017 16:32:13 +0200	[thread overview]
Message-ID: <20171011143213.GF31685@bidouze.vm.6wind.com> (raw)
In-Reply-To: <98afeb56-69b4-dcff-df5b-897d279c5072@intel.com>

On Wed, Oct 11, 2017 at 03:19:31PM +0100, Doherty, Declan wrote:
> On 18/09/2017 10:31 AM, Gaetan Rivet wrote:
> >Hi all,
> >
> >Here is a new version of the PCI bus move out of the EAL.
> >
> >The EAL PCI implementation is divided in two parts:
> >
> >   - librte_pci: library offering helpers to handle PCI objects
> >   - librte_bus_pci: bus driver for PCI devices
> >
> >This allows other libraries / tools to use PCI elements (location, mappings,
> >parsing operations, etc) without forcing a dependency on a bus driver.
> >
> >The latter should not have to export helpers that others might need. It
> >is focused on defining the rte_pci_device, rte_pci_driver objects and
> >their handling.
> >
> >The cryptodev library has hard dependencies on rte_pci_devices (used by
> >generic probe function). Other similar libs (ether and eventdev) avoided
> >the issue by inlining such functions and expecting users to include the
> >relevant headers once the PCI bus has already been built.
> >
> >@Declan:
> >I proposed a solution that would avoid inlining those functions,
> >which does not feel right. Let me know what you think of it or if you
> >think of a better solution. I think it would be best to have cryptodev
> >completely independent from PCI / vdev as far as the lib in concerned
> >(the vdev bus will move as well).
> >
> 
> 
> Hey Gaetan, apologies for the delay in getting back to you on this, I had
> been looking at this but got sidelined onto other issues before usersapce
> and I'm only getting back to it now. I think that while your solution works
> it just highlights the dependency which probably shouldn't be there between
> the cryptodev library and PCI devices. I've had a look, and the functions in
> the cryptodev which you moved don't really provide that much useful
> functionality. I done some testing and completely removed them and just
> update the QAT PMD which is the only crypto PMD which was using them and it
> seems much cleaner to me. I'll push a patch for this change later today and
> it will allow you to drop the patch "cryptodev: move PCI specific helpers to
> drivers/crypto" from this set.
> 
> Regards
> Declan
> 
> 
> 

Hi Declan,

All right, seems good from my PoV. I will rebase onto your patch once
it is sent.

> >v2:
> >
> >   + Made rte_eal_using_phys_addrs common to both linux and bsd interfaces.
> >   + Added documentation of EAL API changes in release note.
> >   + Fixed a few rebase-related mistakes.
> >   + Fixed parallel build race condition reported by Luca Boccassi.
> >   + Grouped together commits breaking compilation:
> >
> >     -> pci: introduce PCI lib and bus
> >     -> lib: include rte_bus_pci
> >     -> drivers: include rte_bus_pci
> >     -> test: include rte_bus_pci
> >     -> app/testpmd: include rte_bus_pci
> >     -> cryptodev: move PCI specific helpers to drivers/crypto
> >
> >   Until all of them have been applied, compilation is broken.
> >   I am currently wondering whether merging some of them might
> >   be sensible.
> >
> >   + Not included in this series:
> >
> >     Several filesystem-related functions are currently
> >     private to the EAL and directly linked. This is not good,
> >     but the solution seems to be to have a new lib offering an FS abstraction.
> >     This seems an overreach for this patchset and should probably come in a
> >     second step.
> >
> >Gaetan Rivet (14):
> >   eal: expose rte_eal_using_phys_addrs
> >   ethdev: remove useless PCI dependency
> >   bus: properly include rte_debug
> >   eal: remove references to PCI
> >   pci: introduce PCI lib and bus
> >   lib: include rte_bus_pci
> >   drivers: include rte_bus_pci
> >   test: include rte_bus_pci
> >   app/testpmd: include rte_bus_pci
> >   cryptodev: move PCI specific helpers to drivers/crypto
> >   pci: avoid inlining functions
> >   pci: avoid over-complicated macro
> >   pci: deprecate misnamed functions
> >   doc: add notes on EAL PCI API update
> >
> >  app/test-pmd/testpmd.h                           |   1 +
> >  config/common_base                               |  10 +
> >  doc/guides/rel_notes/deprecation.rst             |  10 +
> >  doc/guides/rel_notes/release_17_11.rst           |  28 +
> >  drivers/Makefile                                 |   2 +-
> >  drivers/bus/Makefile                             |   2 +
> >  drivers/bus/pci/Makefile                         |  59 ++
> >  drivers/bus/pci/bsd/Makefile                     |  32 +
> >  drivers/bus/pci/bsd/rte_pci.c                    | 671 +++++++++++++++++++++
> >  drivers/bus/pci/include/rte_bus_pci.h            | 387 ++++++++++++
> >  drivers/bus/pci/linux/Makefile                   |  37 ++
> >  drivers/bus/pci/linux/rte_pci.c                  | 723 +++++++++++++++++++++++
> >  drivers/bus/pci/linux/rte_pci_init.h             |  97 +++
> >  drivers/bus/pci/linux/rte_pci_uio.c              | 568 ++++++++++++++++++
> >  drivers/bus/pci/linux/rte_pci_vfio.c             | 675 +++++++++++++++++++++
> >  drivers/bus/pci/linux/rte_vfio_mp_sync.c         | 425 +++++++++++++
> >  drivers/bus/pci/private.h                        | 174 ++++++
> >  drivers/bus/pci/rte_bus_pci_version.map          |  21 +
> >  drivers/bus/pci/rte_pci_common.c                 | 543 +++++++++++++++++
> >  drivers/bus/pci/rte_pci_common_uio.c             | 235 ++++++++
> >  drivers/crypto/Makefile                          |   4 +-
> >  drivers/crypto/pci/Makefile                      |  52 ++
> >  drivers/crypto/pci/rte_cryptodev_pci.c           | 128 ++++
> >  drivers/crypto/pci/rte_cryptodev_pci.h           |  94 +++
> >  drivers/crypto/pci/rte_cryptodev_pci_version.map |   7 +
> >  drivers/crypto/qat/qat_qp.c                      |   1 +
> >  drivers/event/octeontx/ssovf_probe.c             |   1 +
> >  drivers/net/ark/ark_ethdev.c                     |   1 +
> >  drivers/net/avp/avp_ethdev.c                     |   2 +
> >  drivers/net/bnxt/bnxt.h                          |   1 +
> >  drivers/net/bonding/rte_eth_bond_args.c          |   1 +
> >  drivers/net/cxgbe/base/adapter.h                 |   1 +
> >  drivers/net/cxgbe/cxgbe_ethdev.c                 |   1 +
> >  drivers/net/e1000/em_ethdev.c                    |   1 +
> >  drivers/net/e1000/igb_ethdev.c                   |   1 +
> >  drivers/net/e1000/igb_pf.c                       |   1 +
> >  drivers/net/ena/ena_ethdev.h                     |   1 +
> >  drivers/net/enic/base/vnic_dev.h                 |   4 +-
> >  drivers/net/enic/enic_ethdev.c                   |   1 +
> >  drivers/net/enic/enic_main.c                     |   1 +
> >  drivers/net/i40e/i40e_ethdev.c                   |   1 +
> >  drivers/net/i40e/i40e_ethdev_vf.c                |   1 +
> >  drivers/net/ixgbe/ixgbe_ethdev.c                 |   1 +
> >  drivers/net/ixgbe/ixgbe_ethdev.h                 |   1 +
> >  drivers/net/mlx5/mlx5.c                          |   1 +
> >  drivers/net/mlx5/mlx5_ethdev.c                   |   1 +
> >  drivers/net/sfc/sfc.h                            |   1 +
> >  drivers/net/sfc/sfc_ethdev.c                     |   1 +
> >  drivers/net/thunderx/nicvf_ethdev.c              |   1 +
> >  drivers/net/virtio/virtio_ethdev.c               |   1 +
> >  drivers/net/virtio/virtio_pci.h                  |   1 +
> >  drivers/net/vmxnet3/vmxnet3_ethdev.c             |   1 +
> >  lib/Makefile                                     |   2 +
> >  lib/librte_cryptodev/Makefile                    |   1 -
> >  lib/librte_cryptodev/rte_cryptodev_pci.h         |  92 ---
> >  lib/librte_cryptodev/rte_cryptodev_pmd.c         |  94 ---
> >  lib/librte_cryptodev/rte_cryptodev_version.map   |   2 -
> >  lib/librte_eal/bsdapp/eal/Makefile               |   3 -
> >  lib/librte_eal/bsdapp/eal/eal.c                  |   1 -
> >  lib/librte_eal/bsdapp/eal/eal_memory.c           |   6 +
> >  lib/librte_eal/bsdapp/eal/eal_pci.c              | 670 ---------------------
> >  lib/librte_eal/bsdapp/eal/rte_eal_version.map    |  16 +-
> >  lib/librte_eal/common/Makefile                   |   2 +-
> >  lib/librte_eal/common/eal_common_bus.c           |   1 +
> >  lib/librte_eal/common/eal_common_pci.c           | 580 ------------------
> >  lib/librte_eal/common/eal_common_pci_uio.c       | 233 --------
> >  lib/librte_eal/common/eal_private.h              | 143 -----
> >  lib/librte_eal/common/include/rte_memory.h       |  11 +
> >  lib/librte_eal/common/include/rte_pci.h          | 598 -------------------
> >  lib/librte_eal/linuxapp/eal/Makefile             |  10 -
> >  lib/librte_eal/linuxapp/eal/eal.c                |   1 -
> >  lib/librte_eal/linuxapp/eal/eal_interrupts.c     |   1 -
> >  lib/librte_eal/linuxapp/eal/eal_memory.c         |   2 +-
> >  lib/librte_eal/linuxapp/eal/eal_pci.c            | 722 ----------------------
> >  lib/librte_eal/linuxapp/eal/eal_pci_init.h       |  97 ---
> >  lib/librte_eal/linuxapp/eal/eal_pci_uio.c        | 567 ------------------
> >  lib/librte_eal/linuxapp/eal/eal_pci_vfio.c       | 674 ---------------------
> >  lib/librte_eal/linuxapp/eal/eal_vfio_mp_sync.c   | 424 -------------
> >  lib/librte_eal/linuxapp/eal/rte_eal_version.map  |  16 +-
> >  lib/librte_ether/rte_ethdev.c                    |   1 -
> >  lib/librte_ether/rte_ethdev.h                    |   2 -
> >  lib/librte_ether/rte_ethdev_pci.h                |   1 +
> >  lib/librte_eventdev/rte_eventdev_pmd_pci.h       |   1 +
> >  lib/librte_pci/Makefile                          |  48 ++
> >  lib/librte_pci/include/rte_pci.h                 | 285 +++++++++
> >  lib/librte_pci/rte_pci.c                         | 210 +++++++
> >  lib/librte_pci/rte_pci_version.map               |  16 +
> >  mk/rte.app.mk                                    |   3 +
> >  test/test/test_kni.c                             |   1 +
> >  test/test/virtual_pmd.c                          |   1 +
> >  90 files changed, 5603 insertions(+), 4951 deletions(-)
> >  create mode 100644 drivers/bus/pci/Makefile
> >  create mode 100644 drivers/bus/pci/bsd/Makefile
> >  create mode 100644 drivers/bus/pci/bsd/rte_pci.c
> >  create mode 100644 drivers/bus/pci/include/rte_bus_pci.h
> >  create mode 100644 drivers/bus/pci/linux/Makefile
> >  create mode 100644 drivers/bus/pci/linux/rte_pci.c
> >  create mode 100644 drivers/bus/pci/linux/rte_pci_init.h
> >  create mode 100644 drivers/bus/pci/linux/rte_pci_uio.c
> >  create mode 100644 drivers/bus/pci/linux/rte_pci_vfio.c
> >  create mode 100644 drivers/bus/pci/linux/rte_vfio_mp_sync.c
> >  create mode 100644 drivers/bus/pci/private.h
> >  create mode 100644 drivers/bus/pci/rte_bus_pci_version.map
> >  create mode 100644 drivers/bus/pci/rte_pci_common.c
> >  create mode 100644 drivers/bus/pci/rte_pci_common_uio.c
> >  create mode 100644 drivers/crypto/pci/Makefile
> >  create mode 100644 drivers/crypto/pci/rte_cryptodev_pci.c
> >  create mode 100644 drivers/crypto/pci/rte_cryptodev_pci.h
> >  create mode 100644 drivers/crypto/pci/rte_cryptodev_pci_version.map
> >  delete mode 100644 lib/librte_cryptodev/rte_cryptodev_pci.h
> >  delete mode 100644 lib/librte_eal/bsdapp/eal/eal_pci.c
> >  delete mode 100644 lib/librte_eal/common/eal_common_pci.c
> >  delete mode 100644 lib/librte_eal/common/eal_common_pci_uio.c
> >  delete mode 100644 lib/librte_eal/common/include/rte_pci.h
> >  delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci.c
> >  delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci_init.h
> >  delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci_uio.c
> >  delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci_vfio.c
> >  delete mode 100644 lib/librte_eal/linuxapp/eal/eal_vfio_mp_sync.c
> >  create mode 100644 lib/librte_pci/Makefile
> >  create mode 100644 lib/librte_pci/include/rte_pci.h
> >  create mode 100644 lib/librte_pci/rte_pci.c
> >  create mode 100644 lib/librte_pci/rte_pci_version.map
> >
> 

-- 
Gaëtan Rivet
6WIND

  reply	other threads:[~2017-10-11 14:32 UTC|newest]

Thread overview: 156+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-25  9:04 [dpdk-dev] [PATCH 00/13] " Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 01/13] eal: expose rte_eal_using_phys_addrs Gaetan Rivet
2017-09-04 13:24   ` Burakov, Anatoly
2017-08-25  9:04 ` [dpdk-dev] [PATCH 02/13] ethdev: remove useless PCI dependency Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 03/13] bus: properly include rte_debug Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 04/13] eal: remove references to PCI Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 05/13] pci: introduce PCI lib and bus Gaetan Rivet
2017-08-25  9:31   ` Luca Boccassi
2017-08-25  9:34     ` Gaëtan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 06/13] pci: avoid inlining functions Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 07/13] pci: avoid over-complicated macro Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 08/13] pci: deprecate misnamed functions Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 09/13] lib: include rte_bus_pci Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 10/13] drivers: " Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 11/13] test: " Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 12/13] app/testpmd: " Gaetan Rivet
2017-08-25  9:04 ` [dpdk-dev] [PATCH 13/13] cryptodev: move PCI specific helpers to drivers/crypto Gaetan Rivet
2017-09-18  9:31 ` [dpdk-dev] [PATCH v2 00/14] Move PCI away from the EAL Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 01/14] eal: expose rte_eal_using_phys_addrs Gaetan Rivet
2017-09-18 10:47     ` Shreyansh Jain
2017-09-18 11:37       ` Gaëtan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 02/14] ethdev: remove useless PCI dependency Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 03/14] bus: properly include rte_debug Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 04/14] eal: remove references to PCI Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 05/14] pci: introduce PCI lib and bus Gaetan Rivet
2017-09-18 11:53     ` Shreyansh Jain
2017-09-18 11:51       ` Gaëtan Rivet
2017-09-18 12:18         ` Shreyansh Jain
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 06/14] lib: include rte_bus_pci Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 07/14] drivers: " Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 08/14] test: " Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 09/14] app/testpmd: " Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 10/14] cryptodev: move PCI specific helpers to drivers/crypto Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 11/14] pci: avoid inlining functions Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 12/14] pci: avoid over-complicated macro Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 13/14] pci: deprecate misnamed functions Gaetan Rivet
2017-09-18  9:31   ` [dpdk-dev] [PATCH v2 14/14] doc: add notes on EAL PCI API update Gaetan Rivet
2017-09-18 11:22     ` Mcnamara, John
2017-09-25 15:23   ` [dpdk-dev] [PATCH v3 00/13] Move PCI away from the EAL Gaetan Rivet
2017-09-25 15:23     ` [dpdk-dev] [PATCH v3 01/13] eal: expose rte_eal_using_phys_addrs Gaetan Rivet
2017-09-25 15:23     ` [dpdk-dev] [PATCH v3 02/13] ethdev: remove useless PCI dependency Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 03/13] bus: properly include rte_debug Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 04/13] pci: introduce PCI lib and bus Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 05/13] lib: include rte_bus_pci Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 06/13] drivers: " Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 07/13] test: " Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 08/13] app/testpmd: " Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 09/13] cryptodev: move PCI specific helpers to drivers/crypto Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 10/13] pci: avoid inlining functions Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 11/13] pci: avoid over-complicated macro Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 12/13] pci: deprecate misnamed functions Gaetan Rivet
2017-09-25 15:24     ` [dpdk-dev] [PATCH v3 13/13] doc: add notes on EAL PCI API update Gaetan Rivet
2017-10-12  8:17     ` [dpdk-dev] [PATCH v4 00/16] Move PCI away from the EAL Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 01/16] eal: include debug header in bus source Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 02/16] ethdev: remove useless PCI dependency Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 03/16] pci: introduce PCI lib and bus Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 04/16] lib: include PCI bus header Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 05/16] drivers: " Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 06/16] test: " Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 07/16] app/testpmd: " Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 08/16] cryptodev: move PCI specific helpers to drivers/crypto Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 09/16] net/bonding: use local prefix for local function Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 10/16] pci: avoid inlining functions Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 11/16] pci: avoid over-complicated macro Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 12/16] pci: deprecate misnamed functions Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 13/16] pci: introduce PCI address parsing function Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 14/16] pci: make specialized parsing functions private Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 15/16] bus/pci: use new PCI addr parsing function Gaetan Rivet
2017-10-12  8:17       ` [dpdk-dev] [PATCH v4 16/16] doc: add notes on EAL PCI API update Gaetan Rivet
2017-10-12 10:45       ` [dpdk-dev] [PATCH v5 00/20] Move PCI away from the EAL Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 01/20] eal: include debug header in bus source Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 02/20] ethdev: remove useless PCI dependency Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 03/20] pci: introduce PCI lib and bus Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 04/20] lib: include PCI bus header Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 05/20] drivers: " Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 06/20] test: " Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 07/20] app/testpmd: " Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 08/20] cryptodev: move PCI specific helpers to drivers/crypto Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 09/20] net/bonding: use local prefix for local function Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 10/20] pci: avoid inlining functions Gaetan Rivet
2017-10-17 18:20           ` Aaron Conole
2017-10-18  8:54             ` Gaëtan Rivet
2017-10-18 14:30               ` Aaron Conole
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 11/20] pci: avoid over-complicated macro Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 12/20] pci: deprecate misnamed functions Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 13/20] pci: introduce PCI address parsing function Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 14/20] pci: make specialized parsing functions private Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 15/20] bus/pci: use new PCI addr parsing function Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 16/20] bus/pci: do not expose private functions Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 17/20] bus/pci: do not expose PCI match function Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 18/20] bus/pci: do not expose IOVA mode getter Gaetan Rivet
2017-10-12 10:54           ` Gaëtan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 19/20] doc: add notes on EAL PCI API update Gaetan Rivet
2017-10-12 10:45         ` [dpdk-dev] [PATCH v5 20/20] bus: rename scan policy as probe policy Gaetan Rivet
2017-10-12 10:55           ` Gaëtan Rivet
2017-10-25 22:38         ` [dpdk-dev] [PATCH v7 00/25] Move PCI away from the EAL Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 01/25] ethdev: remove useless PCI dependency Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 02/25] eal: include debug header in bus source Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 03/25] eal: include stdint in private header Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 04/25] eal: include common header Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 05/25] eal: expose rte_eal_using_phys_addrs Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 06/25] eal: expose internal config elements Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 07/25] eal: expose vfio symbols Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 08/25] vfio: remove useless PCI headers and add vfio one Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 09/25] vfio: check PCI dependency from within PCI code Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 10/25] vfio: move PCI-related functions out of vfio header Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 11/25] pci: avoid inlining functions Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 12/25] pci: avoid over-complicated macro Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 13/25] pci: deprecate misnamed functions Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 14/25] pci: introduce PCI address parsing function Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 15/25] pci: make specialized parsing functions private Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 16/25] pci: use new PCI addr comparison function Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 17/25] pci: use new PCI addr parsing function Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 18/25] pci: do not expose private functions Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 19/25] pci: do not expose PCI match function Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 20/25] pci: do not expose IOVA mode getter Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 21/25] pci: use EAL exposed configuration Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 22/25] net/bonding: use local prefix for local function Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 23/25] pci: introduce PCI lib and bus Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 24/25] doc: add notes on EAL PCI API update Gaetan Rivet
2017-10-25 22:38           ` [dpdk-dev] [PATCH v7 25/25] maintainers: claim maintainership of PCI lib Gaetan Rivet
2017-10-26 10:05           ` [dpdk-dev] [PATCH v8 00/25] Move PCI away from the EAL Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 01/25] ethdev: remove useless PCI dependency Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 02/25] eal: include debug header in bus source Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 03/25] eal: include stdint in private header Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 04/25] eal: include common header Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 05/25] eal: expose rte_eal_using_phys_addrs Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 06/25] eal: expose internal config elements Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 07/25] eal: expose vfio symbols Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 08/25] vfio: remove useless PCI headers Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 09/25] vfio: check PCI dependency from within PCI code Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 10/25] vfio: move PCI-related symbols out of vfio header Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 11/25] pci: avoid inlining functions Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 12/25] pci: avoid over-complicated macro Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 13/25] pci: deprecate misnamed functions Gaetan Rivet
2017-10-26 10:05             ` [dpdk-dev] [PATCH v8 14/25] pci: introduce PCI address parsing function Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 15/25] pci: make specialized parsing functions private Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 16/25] pci: use new PCI addr comparison function Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 17/25] pci: use new PCI addr parsing function Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 18/25] pci: do not expose private functions Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 19/25] pci: do not expose PCI match function Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 20/25] pci: do not expose IOVA mode getter Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 21/25] pci: use EAL exposed configuration Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 22/25] net/bonding: use local prefix for local function Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 23/25] pci: introduce PCI lib and bus Gaetan Rivet
2017-10-26 15:56               ` Thomas Monjalon
2017-10-26 17:44               ` Thomas Monjalon
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 24/25] doc: add notes on EAL PCI API update Gaetan Rivet
2017-10-26 10:06             ` [dpdk-dev] [PATCH v8 25/25] maintainers: claim maintainership of PCI lib Gaetan Rivet
2017-10-26 21:23             ` [dpdk-dev] [PATCH v8 00/25] Move PCI away from the EAL Thomas Monjalon
2017-10-29 16:47             ` Andrew Rybchenko
2017-10-30  5:16               ` Gaëtan Rivet
2017-10-11 14:19   ` [dpdk-dev] [PATCH v2 00/14] " Doherty, Declan
2017-10-11 14:32     ` Gaëtan Rivet [this message]
2017-10-23  8:44       ` De Lara Guarch, Pablo
2017-10-23  8:49         ` Gaëtan Rivet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171011143213.GF31685@bidouze.vm.6wind.com \
    --to=gaetan.rivet@6wind.com \
    --cc=declan.doherty@intel.com \
    --cc=dev@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).