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
