From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas.monjalon@6wind.com>
Subject: Re: [PATCH v13 0/7] rte_bus_pci
Date: Fri, 14 Apr 2017 14:16:28 +0200
Message-ID: <6574511.32WtNN6P9h@xps13>
References: <cover.1491577715.git.gaetan.rivet@6wind.com>
 <cover.1491906224.git.gaetan.rivet@6wind.com>
 <20170412090931.72cc2e64@xeon-e3>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev@dpdk.org, Stephen Hemminger <stephen@networkplumber.org>
To: Gaetan Rivet <gaetan.rivet@6wind.com>, shreyansh.jain@nxp.com,
 Jan Blunck <jblunck@infradead.org>
Return-path: <dev-bounces@dpdk.org>
Received: from mail-wr0-f176.google.com (mail-wr0-f176.google.com
 [209.85.128.176]) by dpdk.org (Postfix) with ESMTP id 676BC5599
 for <dev@dpdk.org>; Fri, 14 Apr 2017 14:16:30 +0200 (CEST)
Received: by mail-wr0-f176.google.com with SMTP id z109so50214826wrb.1
 for <dev@dpdk.org>; Fri, 14 Apr 2017 05:16:30 -0700 (PDT)
In-Reply-To: <20170412090931.72cc2e64@xeon-e3>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

2017-04-12 09:09, Stephen Hemminger:
> On Tue, 11 Apr 2017 13:07:27 +0200
> Gaetan Rivet <gaetan.rivet@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@networkplumber.org>

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