From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ferruh.yigit@intel.com>
Received: from mga05.intel.com (mga05.intel.com [192.55.52.43])
 by dpdk.org (Postfix) with ESMTP id C5A7C7CCC
 for <dev@dpdk.org>; Mon,  4 Sep 2017 18:05:13 +0200 (CEST)
Received: from orsmga004.jf.intel.com ([10.7.209.38])
 by fmsmga105.fm.intel.com with ESMTP; 04 Sep 2017 09:05:06 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.41,475,1498546800"; d="scan'208";a="125446477"
Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.57])
 ([10.237.220.57])
 by orsmga004.jf.intel.com with ESMTP; 04 Sep 2017 09:04:59 -0700
To: Jan Blunck <jblunck@infradead.org>, dev@dpdk.org
References: <20170711232512.54641-1-jblunck@infradead.org>
 <20170714211213.34436-1-jblunck@infradead.org>
From: Ferruh Yigit <ferruh.yigit@intel.com>
Message-ID: <0ef4eaf8-a356-e229-2b93-31eadca3725e@intel.com>
Date: Mon, 4 Sep 2017 17:04:57 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.3.0
MIME-Version: 1.0
In-Reply-To: <20170714211213.34436-1-jblunck@infradead.org>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
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: Mon, 04 Sep 2017 16:05:14 -0000

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 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

<...>