On 7/24/2018 3:34 PM, Chas Williams wrote:
I think this adds another race. In bond_ethdev_slave_link_status_change_monitor(), it checks to see if bonding is started, if not it exits. So you need to have dev_started = 1 set before you enable this callback.
Indeed, I will have another look at this. Thanks for reviewing! Can you also have a look at this one? https://patches.dpdk.org/patch/43187/ I think I addressed all your previous feedback.
