DPDK patches and discussions
 help / color / mirror / Atom feed
From: Shreyansh Jain <shreyansh.jain@nxp.com>
To: "Gaëtan Rivet" <gaetan.rivet@6wind.com>
Cc: <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v2 05/14] pci: introduce PCI lib and bus
Date: Mon, 18 Sep 2017 17:48:59 +0530	[thread overview]
Message-ID: <1ed11883-d6bb-05c1-aa14-92046bf8bd49@nxp.com> (raw)
In-Reply-To: <20170918115151.GS21444@bidouze.vm.6wind.com>

On Monday 18 September 2017 05:21 PM, Gaëtan Rivet wrote:
> Hey,
> 
> On Mon, Sep 18, 2017 at 05:23:23PM +0530, Shreyansh Jain wrote:
>> Hello Gaetan,
>>
>> On Monday 18 September 2017 03:01 PM, Gaetan Rivet wrote:
>>> The PCI lib defines the types and methods allowing to use PCI elements.
>>>
>>> The PCI bus implements a bus driver for PCI devices by constructing
>>> rte_bus elements using the PCI lib.
>>>
>>> Move the relevant code out of the EAL to their expected place.
>>>
>>> Signed-off-by: Gaetan Rivet <gaetan.rivet@6wind.com>
>>> ---
>>>   config/common_base                              |  10 +
>>>   drivers/bus/Makefile                            |   2 +
>>>   drivers/bus/pci/Makefile                        |  59 ++
>>>   drivers/bus/pci/bsd/Makefile                    |  32 ++
>>>   drivers/bus/pci/bsd/rte_pci.c                   | 670 ++++++++++++++++++++++
>>>   drivers/bus/pci/include/rte_bus_pci.h           | 387 +++++++++++++
>>>   drivers/bus/pci/linux/Makefile                  |  37 ++
>>>   drivers/bus/pci/linux/rte_pci.c                 | 722 ++++++++++++++++++++++++
>>>   drivers/bus/pci/linux/rte_pci_init.h            |  97 ++++
>>>   drivers/bus/pci/linux/rte_pci_uio.c             | 567 +++++++++++++++++++
>>>   drivers/bus/pci/linux/rte_pci_vfio.c            | 674 ++++++++++++++++++++++
>>>   drivers/bus/pci/linux/rte_vfio_mp_sync.c        | 424 ++++++++++++++
>>>   drivers/bus/pci/private.h                       | 173 ++++++
>>>   drivers/bus/pci/rte_bus_pci_version.map         |  21 +
>>>   drivers/bus/pci/rte_pci_common.c                | 542 ++++++++++++++++++
>>>   drivers/bus/pci/rte_pci_common_uio.c            | 234 ++++++++
>>>   lib/Makefile                                    |   2 +
>>>   lib/librte_eal/bsdapp/eal/Makefile              |   3 -
>>>   lib/librte_eal/bsdapp/eal/eal_pci.c             | 670 ----------------------
>>>   lib/librte_eal/bsdapp/eal/rte_eal_version.map   |  15 -
>>>   lib/librte_eal/common/Makefile                  |   2 +-
>>>   lib/librte_eal/common/eal_common_pci.c          | 580 -------------------
>>>   lib/librte_eal/common/eal_common_pci_uio.c      | 233 --------
>>>   lib/librte_eal/common/include/rte_pci.h         | 598 --------------------
>>>   lib/librte_eal/linuxapp/eal/Makefile            |  10 -
>>>   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 |  15 -
>>>   lib/librte_ether/rte_ethdev.h                   |   2 -
>>>   lib/librte_pci/Makefile                         |  48 ++
>>>   lib/librte_pci/include/rte_pci.h                | 279 +++++++++
>>>   lib/librte_pci/rte_pci.c                        |  92 +++
>>>   lib/librte_pci/rte_pci_version.map              |   8 +
>>>   mk/rte.app.mk                                   |   3 +
>>>   37 files changed, 5084 insertions(+), 4611 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
>>>   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
>>>
>>
>> <lot of snip here...>
>>
>>> +#endif /* _PCI_PRIVATE_H_ */
>>> diff --git a/drivers/bus/pci/rte_bus_pci_version.map b/drivers/bus/pci/rte_bus_pci_version.map
>>> new file mode 100644
>>> index 0000000..eca49e9
>>> --- /dev/null
>>> +++ b/drivers/bus/pci/rte_bus_pci_version.map
>>> @@ -0,0 +1,21 @@
>>> +DPDK_17.08 {
>>
>> You might want to bump this to 17.11.
>>
> 
> Thanks, fixing this.
> 
>>> +	global:
>>> +
>>> +	rte_pci_detach;
>>> +	rte_pci_dump;
>>> +	rte_pci_ioport_map;
>>> +	rte_pci_ioport_read;
>>> +	rte_pci_ioport_unmap;
>>> +	rte_pci_ioport_write;
>>> +	rte_pci_map_device;
>>> +	rte_pci_probe;
>>> +	rte_pci_probe_one;
>>> +	rte_pci_read_config;
>>> +	rte_pci_register;
>>> +	rte_pci_scan;
>>> +	rte_pci_unmap_device;
>>> +	rte_pci_unregister;
>>> +	rte_pci_write_config;
>>> +
>>> +	local: *;
>>> +};
>>
>> This is huuuge patch :( and I am not yet through it (most of it is movement
>> so I doubt anything major would be problem here).
>> Just the above comment in case you are spinning a new series.
>>
>>
> 
> Thanks for reading the patch.
> 
> Yes, most of it is moving the code as-is to a new location.
> I tried to reduce it, but at some point it does not really make sense
> anymore.

Agree. It is difficult to manage such large movement with a compile-able 
patch series. I think compilation success takes priority over size in 
such cases so as not to break bisect.

Here, I found that "eal: remove references to PCI" onwards, the 
compilation was breaking. You have already mentioned about this in the 
cover letter, it seems.

> 
> I think the important thing to look for here is the build system,
> dependency graph and the division of the PCI API between the lib and the
> bus driver.

Agree. I am trying to look through this.

> 
> I divided it along the lines of the rte_pci_device being defined or not.
> Anything using an rte_pci_device going to the bus and everything
> else going to the lib.

Yes, I agree with this in principle.
rte_xxx_device and rte_xxx_driver originate from the xxx_bus. Bus should 
be responsible for defining (scan) xxx_devices and attaching (probe) to 
xxx_driver. Which in turn means that all APIs dealing with xxx_device 
and xxx_driver should be pivoted in the Bus layer.

Though, I think here the complexity is also of crypto devices. You have 
already reached out to Declan through cover letter - probably his (and 
other crypto experts) opinion would matter here.

My opinion would be to have

> 
> I'm mostly worried about this divide. Having the rte_pci_device defined
> seems mostly of the responsibility of the bus driver, in my opinion. I'd
> like to hear others'.
> 

+1 from side for this principle for divide.

  reply	other threads:[~2017-09-18 12:08 UTC|newest]

Thread overview: 156+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-25  9:04 [dpdk-dev] [PATCH 00/13] Move PCI away from the EAL 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 [this message]
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
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=1ed11883-d6bb-05c1-aa14-92046bf8bd49@nxp.com \
    --to=shreyansh.jain@nxp.com \
    --cc=dev@dpdk.org \
    --cc=gaetan.rivet@6wind.com \
    /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).