From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas.monjalon@6wind.com>
Subject: Re: [PATCH 00/13] Introducing EAL Bus-Device-Driver Model
Date: Thu, 08 Dec 2016 08:21:21 +0100
Message-ID: <1743313.gQvgAgeimG@xps13>
References: <1480846288-2517-1-git-send-email-shreyansh.jain@nxp.com>
 <1524529.OpOKghfniY@xps13> <a0b14c5f-ad9d-dc08-90f3-284226ce714a@nxp.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: David Marchand <david.marchand@6wind.com>, dev@dpdk.org
To: Shreyansh Jain <shreyansh.jain@nxp.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mail-wj0-f178.google.com (mail-wj0-f178.google.com
 [209.85.210.178]) by dpdk.org (Postfix) with ESMTP id 767902A66
 for <dev@dpdk.org>; Thu,  8 Dec 2016 08:21:23 +0100 (CET)
Received: by mail-wj0-f178.google.com with SMTP id v7so381015080wjy.2
 for <dev@dpdk.org>; Wed, 07 Dec 2016 23:21:23 -0800 (PST)
In-Reply-To: <a0b14c5f-ad9d-dc08-90f3-284226ce714a@nxp.com>
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>

2016-12-08 10:34, Shreyansh Jain:
> On Wednesday 07 December 2016 06:54 PM, Thomas Monjalon wrote:
> > 2016-12-07 18:40, Shreyansh Jain:
> >> Is there anything specific that you are looking for in patchset v2?
> >> I was thinking of:
> >> 0. fixing BSD compilation issue reported by CI
> >> 1. improving the test_pci.c
> >> 2. hotplugging
> >> 3. trying to move PCI to drives/bus/pci/linux/* and resolving how
> >> drivers link to it, and how EAL resources like devargs are consumed.
> >
> > I am concerned about the time needed for all these changes.
> > Please let's make sure that the basic parts are well done and pushed, first.
> > That's why I suggest to postpone 1, 2 and 3 to next release if possible.
> > The priority is to have a clean bus model,
> > and if time permits, integrate the NXP driver.
> >
> 
> I am absolutely OK with this.
> 
> Just a confirmation, though. Is it OK to have a broken test_pci.c (1) in 
> a series? It is not getting compiled by default, but it would definitely 
> not work if compiled (symbols are missing).

No, that's why I said "if possible" :).
I thought 1. was about improvements. But if it does not compile, it is
a must.