[ovs-dev] [PATCH 2/2] datapath: Destroy internal devices before freeing datapath.

[Ben Pfaff]

On Thu, Feb 24, 2011 at 04:23:07PM -0800, Jesse Gross wrote:
> When destroying vports we account for two types of synchronization
> mechanisms: RTNL and RCU.  However, it is possible to call into
> network device methods with just a device reference without either
> of these.  These device methods can use the datapath data structures
> but we don't wait for all of the references to go away before freeing
> the datapath.  The actual wait happens in rtnl_unlock(), so by moving
> up that call we can avoid the possibility of use after free with
> internal devices.
> 
> Signed-off-by: Jesse Gross <jesse at nicira.com>

Acked-by: Ben Pfaff <blp at nicira.com>