29/03/2018 08:13, Shahaf Shuler: > And device attributes like : RTE_ETH_DEV_BONDED_SLAVE and > RTE_ETH_DEV_REPRESENTOR. > I don't think they belong together under the genetic name of dev_flags. > > Moreover, I am not sure the fact device is bonded slave should be exposed to > the application. It should be internal to ethdev and its port iterators.
RTE_ETH_DEV_BONDED_SLAVE flag is used to prevent a manual detach of a slave. I think it is wrong. The bonding PMD should be able to manage any detach at any time, because a real hardware plug-out can happen. So I am in favor of removing RTE_ETH_DEV_BONDED_SLAVE.
