From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gaetan.rivet@6wind.com>
Received: from mail-wm0-f50.google.com (mail-wm0-f50.google.com [74.125.82.50])
 by dpdk.org (Postfix) with ESMTP id DC37A3254
 for <dev@dpdk.org>; Tue,  5 Sep 2017 10:20:54 +0200 (CEST)
Received: by mail-wm0-f50.google.com with SMTP id 187so15183735wmn.1
 for <dev@dpdk.org>; Tue, 05 Sep 2017 01:20:54 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=6wind-com.20150623.gappssmtp.com; s=20150623;
 h=date:from:to:cc:subject:message-id:references:mime-version
 :content-disposition:content-transfer-encoding:in-reply-to
 :user-agent; bh=S/MbEpIzAEMPA52H2lCcq/mVPoBEW/ycUIrO5ayXAxs=;
 b=nmg6Zb4aOTvOJOC8QHf32FthxNCutpY+ONzu4L2gKiAugZxR43DZe6TVEKSKH4duDW
 9pc4LVOjmY+Z1Wt3MzNkSua/hLxWnBtS5U00SoyOBokT6b87Vr2LrC0JnkmLY1pN9aJN
 BR3Fp9qRXhB2s6q2xDhMNFh8T2+qXvHkC2DzVWOUSGU0zYmYovD2M+CEb0cCU49DXt6a
 7BZfyKbvsB67rR7adxyV86edmwJB5bNcaqZnEdBWSzaLk455C4tz9s2kGTWWpYipb3sr
 UFuwuXQO6I5THvNyEzSNcWZXxM6zWeUDo9katowbNsiIJrc8CO0iiqfsDn045Cl/4ibN
 WhfQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20161025;
 h=x-gm-message-state:date:from:to:cc:subject:message-id:references
 :mime-version:content-disposition:content-transfer-encoding
 :in-reply-to:user-agent;
 bh=S/MbEpIzAEMPA52H2lCcq/mVPoBEW/ycUIrO5ayXAxs=;
 b=MeFvYND2J0Ngy3+phYec0UYdg5pRzdMvtfrv7AsD85DhSOD9bqpvgU0xd5qJr6nS31
 +KLY667gYNcTBM16d8rDeHnu+/+zBVAdBr9E/Bb82PDufmhT37y/N7Nrb8Jwn8JJfBdO
 3zq32DI4h3Fco9ZXhcxUvfjLUoNKs759hI06uMY4iSy1/lmztFgCdOBy8GGqeLGOInDu
 S+8V7bGUyaALdkpNUk2YNywl68NkV+F32Td6SRXnpAxuzNXmaIt2UoJxViJ0pPmtzVA1
 yfuWu/MhpAgEimqLVDAmXqfFzCusOMqOUA0j2xkvMsFv52noYmkwMAAtTOdj3yDQncjn
 q45g==
X-Gm-Message-State: AHPjjUg+0Iosx0D1RY/tKbsgNonuWCFrodnsgxMR6YrxYoQtnpKX1WcB
 bq4OqtPM8Vt+sWjh
X-Google-Smtp-Source: ADKCNb7R5vblNGyCrFZo5KkvUh75CBj3BqT6J+epyY4kZEciZ+Sthyr1DukQEulbb/2HbCMqUtgn6g==
X-Received: by 10.28.101.85 with SMTP id z82mr1856095wmb.32.1504599654305;
 Tue, 05 Sep 2017 01:20:54 -0700 (PDT)
Received: from bidouze.vm.6wind.com (host.78.145.23.62.rev.coltfrance.com.
 [62.23.145.78])
 by smtp.gmail.com with ESMTPSA id q64sm123898wmg.42.2017.09.05.01.20.53
 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);
 Tue, 05 Sep 2017 01:20:53 -0700 (PDT)
Date: Tue, 5 Sep 2017 10:20:43 +0200
From: =?iso-8859-1?Q?Ga=EBtan?= Rivet <gaetan.rivet@6wind.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Cc: Jan Blunck <jblunck@infradead.org>, dev@dpdk.org
Message-ID: <20170905082043.GE21444@bidouze.vm.6wind.com>
References: <20170711232512.54641-1-jblunck@infradead.org>
 <20170714211213.34436-1-jblunck@infradead.org>
 <0ef4eaf8-a356-e229-2b93-31eadca3725e@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <0ef4eaf8-a356-e229-2b93-31eadca3725e@intel.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Subject: Re: [dpdk-dev] [PATCH v2 00/15] devargs fixes
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
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>
X-List-Received-Date: Tue, 05 Sep 2017 08:20:55 -0000

Hi Ferruh,

On Mon, Sep 04, 2017 at 05:04:57PM +0100, Ferruh Yigit wrote:
> On 7/14/2017 10:11 PM, Jan Blunck wrote:
> > The changes to enum rte_devtype that got merged into 17.08-rc1 are breaking
> > API without prior notice. This series is reworking the rte_devargs changes
> > in a way hopefully compliant to the new failover PMD and still keeping API
> > compatible with earlier releases.
> 
> This patchset seems target 17.08, but 17.08 already released and some of
> the patches in this patchset seems included in the release.
> 
> Patchset needs to be rebased on top of latest HEAD.
> 

The relevant fixes in this patchset were included. Other "fixes" that
tried to impose a certain API and one way of doing things were not.

These evolutions should have been proposed within the proposal window.
I am currently working on a series addressing a few of those elements.

> > 
> > The introduced changes to 17.08-rc1 are trading the tightly coupling of
> > struct rte_devargs to the PCI and vdev bus against the struct rte_bus.
> > The changes proposed in this series decouple struct rte_devargs from
> > the new dependencies.
> > 
> > Changes since v1:
> > - explicitly pass busname to rte_eal_devargs_parse() and validate it
> > - better explain why changes are done
> > 
> > Jan Blunck (15):
> >   Revert "devargs: make device types generic"
> >   devargs: fix unittest
> >   devargs: extend unittest
> >   devargs: deprecate enum rte_devtype based functions
> >   pci: use scan_mode configuration
> >   bus: add configuration interface for buses
> >   devargs: use bus configuration interface to set scanning mode
> >   devargs: use existing functions in rte_eal_devargs_parse()
> >   devargs: add busname string field
> >   devargs: use busname
> >   pci: use busname
> >   vdev: use busname
> >   devargs: pass busname argument when parsing
> >   devargs: remove type field
> >   devargs: remove bus field
> 
> <...>
> 

-- 
Gaëtan Rivet
6WIND