[ovs-dev] [PATCH] netdev-dpdk: Fix dev attached flag.

[Ilya Maximets]

On 08.11.2019 9:55, Zhike Wang wrote:
> If the dev was already probed correctly, the dev attached flag
> should be set to true, or resource would leak during destruct.

We're not detaching devices that wasn't attached by us.

If device is already probed in DPDK than it means that most likely it
was attached on dpdk_eal_init(), because user explicitly added this
device in EAL arguments with 'dpdk-extra' knob.

Previously there was non-detachable devices and we protected ourselves
from trying to detach them.  I beleive (hope) that in modern DPDK
all the devices are detachable, but still.  If user added devices
to EAL arguments than he/she wanted them to exist whole time while
OVS is running and we're not eligible to detach these devices.

If you really want to detach them, use special command
'ovs-appctl netdev-dpdk/detach' that was designed for this exact case.

Best regards, Ilya Maximets.