On Thu, Aug 7, 2014 at 7:21 PM, Jesse Gross <[email protected]> wrote: > On Thu, Aug 7, 2014 at 12:51 PM, Pravin B Shelar <[email protected]> wrote: >> There are two separate API to allocate and copy actions list. Anytime >> OVS needs to copy action list, it needs to call both functions. >> Following patch moves action allocation to copy function to avoid >> code duplication. >> >> Signed-off-by: Pravin B Shelar <[email protected]> > > This introduces a warning: > > CC [M] /home/jesse/openvswitch/datapath/linux/datapath.o > /home/jesse/openvswitch/datapath/linux/datapath.c: In function > ‘ovs_flow_cmd_set’: > /home/jesse/openvswitch/datapath/linux/datapath.c:1094:1: warning: the > frame size of 1256 bytes is larger than 1024 bytes > [-Wframe-larger-than=] > } > > This is essentially undoing commit > 1d2a1b5f5252e4c6ce8bbf8d91ca27aba52496e6 (datapath: Factor out > allocation and verification of actions.), which was supposed to avoid > this problem. It's actually not a real problem but the warning is > annoying.
I reintroduced that commit to fix the warning. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
