On 4/7/20 8:55 AM, Somnath Kotur via dev wrote:
> The 'attached' flag for a netdev_dpdk device is set only when it is
> attached via rte_dev_probe(). However, when the DPDK port probe was
> successfully done as part of vswitchd bring up (if the device was bound to
> igb_uio/vfio-pci prior to bring up), then this flag is not set.
> When netdev_dpdk_process_devargs() finds this already probed device as part
> of 'ovs-vsctl add port' cmd and returns it with a port_id, the
> 'attached' flag for the 'dev' is not set which can lead to improper cleanup
> while destroying the device later on in netdev_dpdk_destruct.
It's not a bug. It's intentional that we're setting 'attached' flag only
for devices that OVS really attached by itself. We're not responsible for
devices that were probed on the initialization stage.
See the following thread with a similar patch for details:
https://patchwork.ozlabs.org/patch/1191742/
We might want to improve some comments around this flag, though.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
