On Wed, Jan 29, 2014 at 9:20 AM, <[email protected]> wrote: > From: Pravin <[email protected]> > > ovs_flow_free() is not called under ovs-lock during packet > execute path. Since packet execute does not touch flow->mask, > there is no need to take that lock either. So move assert in > case where flow->mask is checked. > > Found by code inspection. > > Signed-off-by: Pravin B Shelar <[email protected]>
The idea for putting it here is that callers should always hold OVS mutex and by putting the check earlier it increases the chances that problems will be caught sooner. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
