[dpdk-dev] [PATCH v13 0/7] rte_bus_pci

Thomas Monjalon thomas.monjalon at 6wind.com
Fri Apr 14 14:16:28 CEST 2017


2017-04-12 09:09, Stephen Hemminger:
> On Tue, 11 Apr 2017 13:07:27 +0200
> Gaetan Rivet <gaetan.rivet at 6wind.com> wrote:
> 
> > I have rebased the work done by Shreyansh and Thomas to follow
> > the bus abstraction introduced in v17.02 for the PCI bus
> > implementation.
> > This series is only a partial version of the previous one, because
> > some of the patches have already been integrated, and others still
> > need rework.
> > 
> > Additionally, Jan Blunck sent this RFC:
> > 
> > [RFC] eal: use busname and devargs to attach devices
> > 
> > That has not been considered for this version as it had not yet spawned
> > a discussion.
> > 
> > changes in v13:
> >   - Removed the PCI tests. They cannot follow the bus implementation
> >     anymore, and it is an opportunity to update their original design.
> >   - Changed the rte_pci_match function semantic to match its name.
> >   - Made the rte_pci_match function private.
> >   - Made the rte_bus_list private
> >   - Made rte_eal_pci_add_device,
> >          rte_eal_pci_insert_device,
> >          rte_eal_pci_remove_device,
> >      functions private.
> > 
> > Additional work is needed:
> >   - The PCI bus should be moved to the relevant bus sub-directory.
> >   - The documentation must be updated.
> > 
> > But this version should allow for the PCI bus implementation to be
> > integrated in v17.05-rc2.
> > 
> > Gaetan Rivet (1):
> >   test: remove pci tests
> > 
> > Shreyansh Jain (6):
> >   pci: split match and probe
> >   pci: remove loop over drivers in device detach
> >   pci: add bus driver
> >   pci: use bus driver for scan/probe
> >   pci: use bus driver for attach/detach
> >   doc: remove deprecation notice for rte_bus
> 
> Looks good. Removing dependencies on PCI is part of the roadblocks in supporting Hyper-V/VMBUS
> and Xen netfront properly.
> 
> Acked-by: Stephen Hemminger <stephen at networkplumber.org>

Applied without the attach/detach patch because it requires more work.
Thanks everybody


More information about the dev mailing list