On Tue, Feb 11, 2014 at 04:30:45PM -0800, Jarno Rajahalme wrote:
> When the target flow exists already as intended, only the 'modified'
> time needs to be updated. Allow modification without taking the
> 'ofproto_mutex' by always using the 'rule->mutex' when accessing the
> 'modified' time.
>
> Signed-off-by: Jarno Rajahalme <[email protected]>
This is a good idea. Thank you.
In ofproto_flow_mod(), the check
&& !(fm->flags & OFPUTIL_FF_RESET_COUNTS)
can be moved to the top of the function, since it doesn't depend on what
we find in the rule (or whether we find a rule).
This adds a spurious blank line to ofopgroup_complete().
Acked-by: Ben Pfaff <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
