On Mon, Mar 30, 2015 at 4:57 AM, Thomas Graf <tg...@suug.ch> wrote:
> Return module reference before invoking the respective vport
> ->destroy() function. This is needed as ovs_vport_del() is not
> invoked inside an RCU read side critical section so the kfree
> can occur immediately before returning to ovs_vport_del().
>
> Returning the module reference before ->destroy() is safe because
> the module unregistration is blocked on ovs_lock which we hold
> while destroying the datapath.
>
> Fixes: 62b9c8d0372d ("ovs: Turn vports with dependencies into separate
> modules")
> Reported-by: Pravin Shelar <pshe...@nicira.com>
> Signed-off-by: Thomas Graf <tg...@suug.ch>
looks good.
Acked-by: Pravin B Shelar <pshe...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
