Re: [ovs-dev] [PATCH 6/7] datapath: Factor out allocation and verification of actions.

2014-06-19 Thread Jesse Gross
On Thu, Jun 19, 2014 at 1:31 PM, Pravin Shelar wrote: > Git am warning. > > Applying: datapath: Factor out allocation and verification of actions. > > /home/pravin/ovs/w7/.git/rebase-apply/patch:64: trailing whitespace. > > warning: 1 line adds whitespace errors. > > On Tue, Jun 10, 2014 at 4:47 P

Re: [ovs-dev] [PATCH 6/7] datapath: Factor out allocation and verification of actions.

2014-06-19 Thread Pravin Shelar
Git am warning. Applying: datapath: Factor out allocation and verification of actions. /home/pravin/ovs/w7/.git/rebase-apply/patch:64: trailing whitespace. warning: 1 line adds whitespace errors. On Tue, Jun 10, 2014 at 4:47 PM, Jesse Gross wrote: > As the size of the flow key grows, it can pu

Re: [ovs-dev] [PATCH 6/7] datapath: Factor out allocation and verification of actions.

2014-06-11 Thread Thomas Graf
On 06/10/14 at 04:47pm, Jesse Gross wrote: > As the size of the flow key grows, it can put some pressure on the > stack. This is particularly true in ovs_flow_cmd_set(), which needs several > copies of the key on the stack. One of those uses is logically separate, > so this factors it out to reduce

[ovs-dev] [PATCH 6/7] datapath: Factor out allocation and verification of actions.

2014-06-10 Thread Jesse Gross
As the size of the flow key grows, it can put some pressure on the stack. This is particularly true in ovs_flow_cmd_set(), which needs several copies of the key on the stack. One of those uses is logically separate, so this factors it out to reduce stack pressure and improve readibility. Signed-of