Re: [ovs-dev] [PATCH 2/3] netdev: Safely increment refcount in netdev_open().

2014-05-04 Thread Joe Stringer
Thanks for review, pushed to master and branch-2.2. Backport for 2.1 and 2.0 is here: http://openvswitch.org/pipermail/dev/2014-May/039761.html On 3 May 2014 02:52, Ben Pfaff wrote: > On Fri, May 02, 2014 at 01:13:42PM +1200, Joe Stringer wrote: > > netdev_open() would previously increment a n

Re: [ovs-dev] [PATCH 2/3] netdev: Safely increment refcount in netdev_open().

2014-05-02 Thread Ben Pfaff
On Fri, May 02, 2014 at 01:13:42PM +1200, Joe Stringer wrote: > netdev_open() would previously increment a netdev's refcount without > holding a lock for it. This commit shifts the locking to protect it. > > Found by inspection. > > Signed-off-by: Joe Stringer Should we backport this? Acked-by

Re: [ovs-dev] [PATCH 2/3] netdev: Safely increment refcount in netdev_open().

2014-05-01 Thread YAMAMOTO Takashi
> netdev_open() would previously increment a netdev's refcount without > holding a lock for it. This commit shifts the locking to protect it. > > Found by inspection. > > Signed-off-by: Joe Stringer Reviewed-by: YAMAMOTO Takashi ___ dev mailing list

[ovs-dev] [PATCH 2/3] netdev: Safely increment refcount in netdev_open().

2014-05-01 Thread Joe Stringer
netdev_open() would previously increment a netdev's refcount without holding a lock for it. This commit shifts the locking to protect it. Found by inspection. Signed-off-by: Joe Stringer --- lib/netdev.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/netdev.c b