Re: [ovs-dev] [PATCH v5 6/9] datapath: Split ovs_flow_cmd_new_or_set().

2014-03-28 Thread Pravin Shelar
On Tue, Mar 25, 2014 at 2:35 PM, Jarno Rajahalme wrote: > Following patch will be easier to reason about with separate > ovs_flow_cmd_new() and ovs_flow_cmd_set() functions. > > Signed-off-by: Jarno Rajahalme > --- > datapath/datapath.c | 160 > +

Re: [ovs-dev] [PATCH v5 6/9] datapath: Split ovs_flow_cmd_new_or_set().

2014-03-28 Thread Jarno Rajahalme
On Mar 28, 2014, at 1:26 PM, Pravin Shelar wrote: > On Tue, Mar 25, 2014 at 2:35 PM, Jarno Rajahalme > wrote: >> Following patch will be easier to reason about with separate >> ovs_flow_cmd_new() and ovs_flow_cmd_set() functions. >> >> Signed-off-by: Jarno Rajahalme >> — ... >> -

[ovs-dev] [PATCH v5 6/9] datapath: Split ovs_flow_cmd_new_or_set().

2014-03-25 Thread Jarno Rajahalme
Following patch will be easier to reason about with separate ovs_flow_cmd_new() and ovs_flow_cmd_set() functions. Signed-off-by: Jarno Rajahalme --- datapath/datapath.c | 160 +-- 1 file changed, 116 insertions(+), 44 deletions(-) diff --git a/da