Re: [ovs-dev] [PATCH v3 net-next] openvswitch: fix vport-netdev unregister

2013-10-15 Thread Alexei Starovoitov
On Tue, Oct 15, 2013 at 9:53 AM, Alexei Starovoitov wrote: > On Tue, Oct 15, 2013 at 8:31 AM, Jesse Gross wrote: >> On Sun, Oct 13, 2013 at 8:50 PM, Alexei Starovoitov >> wrote: >>> diff --git a/net/openvswitch/dp_notify.c b/net/openvswitch/dp_notify.c >>> index c323567..ffa429a 100644 >>> ---

Re: [ovs-dev] [PATCH v3 net-next] openvswitch: fix vport-netdev unregister

2013-10-15 Thread Alexei Starovoitov
On Tue, Oct 15, 2013 at 8:31 AM, Jesse Gross wrote: > On Sun, Oct 13, 2013 at 8:50 PM, Alexei Starovoitov wrote: >> diff --git a/net/openvswitch/dp_notify.c b/net/openvswitch/dp_notify.c >> index c323567..ffa429a 100644 >> --- a/net/openvswitch/dp_notify.c >> +++ b/net/openvswitch/dp_notify.c >>

Re: [ovs-dev] [PATCH v3 net-next] openvswitch: fix vport-netdev unregister

2013-10-15 Thread Jesse Gross
On Sun, Oct 13, 2013 at 8:50 PM, Alexei Starovoitov wrote: > diff --git a/net/openvswitch/dp_notify.c b/net/openvswitch/dp_notify.c > index c323567..ffa429a 100644 > --- a/net/openvswitch/dp_notify.c > +++ b/net/openvswitch/dp_notify.c > @@ -59,15 +59,9 @@ void ovs_dp_notify_wq(struct work_struct

[ovs-dev] [PATCH v3 net-next] openvswitch: fix vport-netdev unregister

2013-10-13 Thread Alexei Starovoitov
The combination of two commits: commit 8e4e1713e4 ("openvswitch: Simplify datapath locking.") commit 2537b4dd0a ("openvswitch:: link upper device for port devices") introduced a bug where upper_dev wasn't unlinked upon netdev_unregister notification The following steps: modprobe openvswitch