<div dir="ltr">Thanks for review, pushed to master and branch-2.2.<div><br></div><div>Backport for 2.1 and 2.0 is here:</div><div><a href="http://openvswitch.org/pipermail/dev/2014-May/039761.html">http://openvswitch.org/pipermail/dev/2014-May/039761.html</a><br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 3 May 2014 02:52, Ben Pfaff <span dir="ltr"><<a href="mailto:blp@nicira.com" target="_blank">blp@nicira.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On Fri, May 02, 2014 at 01:13:42PM +1200, Joe Stringer wrote:<br>
> netdev_open() would previously increment a netdev's refcount without<br>
> holding a lock for it. This commit shifts the locking to protect it.<br>
><br>
> Found by inspection.<br>
><br>
> Signed-off-by: Joe Stringer <<a href="mailto:joestringer@nicira.com">joestringer@nicira.com</a>><br>
<br>
</div>Should we backport this?<br>
<br>
Acked-by: Ben Pfaff <<a href="mailto:blp@nicira.com">blp@nicira.com</a>><br>
</blockquote></div><br></div>