From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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 ; Tue, 5 Sep 2017 10:20:54 +0200 (CEST) Received: by mail-wm0-f50.google.com with SMTP id 187so15183735wmn.1 for ; 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 To: Ferruh Yigit Cc: Jan Blunck , 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-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