14/07/2017 23:11, Jan Blunck: > 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. > > 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
Most of these changes does not seem to fix a bug or an API break. Before -rc1, we can take them for the sake of cleaning. But after -rc1, we must take only fixes. If I understand well, the urgent fix to make in -rc2 is to revert an API break without prior notice. Which change is it? rte_devtype? The plan is to fix hotplug/devargs while integrating failsafe in -rc2. After releasing -rc2, we must agree on the planned clean-up and be sure that the right deprecation notices are in 17.08. Then we will integrate all these patches in August for 17.11.
