[ovs-dev] [PATCH net 4/5] openvswitch: Fix ovs_flow_free() ovs-lock assert.

2014-02-04 Thread Jesse Gross
From: Pravin B Shelar 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. Sign

[ovs-dev] [PATCH net 4/5] openvswitch: Fix ovs_flow_free() ovs-lock assert.

2014-02-04 Thread Jesse Gross
From: Pravin B Shelar 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. Sign

Re: [ovs-dev] [PATCH net 4/5] openvswitch: Fix ovs_flow_free() ovs-lock assert.

2014-02-03 Thread Sergei Shtylyov
Hello. On 03-02-2014 5:08, Jesse Gross wrote: From: Pravin B Shelar 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 flo

[ovs-dev] [PATCH net 4/5] openvswitch: Fix ovs_flow_free() ovs-lock assert.

2014-02-02 Thread Jesse Gross
From: Pravin B Shelar 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. Sign