git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: b73d3fe) | patch
datapath: Fix ovs_flow_free() ovs-lock assert.
authorPravin B Shelar <pshelar@nicira.com>
Tue, 28 Jan 2014 02:18:33 +0000 (18:18 -0800)
committerPravin B Shelar <pshelar@nicira.com>
Tue, 28 Jan 2014 02:25:23 +0000 (18:25 -0800)
commitb1705c56f366d7dd47c60fed13229db022ce3198
tree0819b6ed0773d069d06e1e50cc6d46629c9362adtree | snapshot
parentb73d3fe2c11d2d670092bce2da48c1ee0f6eaf5fcommit | diff
datapath: Fix ovs_flow_free() ovs-lock assert.

ovs_flow_free() is not called under ovs-lock during packet
execute path (ovs_packet_cmd_execute()). 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 <pshelar@nicira.com>
Acked-by: Andy Zhou <azhou@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>
datapath/flow_table.c diff | blob | history
sliver-openvswitch