On Sat, 12 Oct 2013 at 01:12 GMT, Alexei Starovoitov <[email protected]> wrote:
> @@ -87,7 +81,11 @@ static int dp_device_event(struct notifier_block *unused, 
> unsigned long event,
>       if (!vport)
>               return NOTIFY_DONE;
>  
> -     if (event == NETDEV_UNREGISTER) {
> +     if (event == NETDEV_UNREGISTER && dev->priv_flags & IFF_OVS_DATAPATH) {
> +             /* upper_dev_unlink and decrement promisc immediately */
> +             ovs_netdev_detach_dev(vport);
> +
> +             /* schedule vport destroy, dev_put and genl notification */

ovs_netdev_get_vport() already checks IFF_OVS_DATAPATH flag before this 'if'.

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to