The code in ofproto-dpif.c became harder on the eye after the reorg of struct xlate. This series tries cleaning that up a bit, and while at it, identifying and reducing unnecessary copying of struct flow. The main enabler for this is to 1) pass struct flow around via a pointer, and 2) initializing a new one only if necessary.
The last patch may require a bit more thorough review, others are mostly syntactic transformations. Jarno Rajahalme (11): ofproto-dpif: Check for MPLS depth at the flow. ofproto-dpif: Use flow pointer in action execution. ofproto-dpif: Harmonize naming of internal functions. ofproto-dpif: Use flow pointers inside functions. ofproto-dpif: Make xlate_in.flow a pointer. ofproto-dpif: Avoid unnecessary copying of struct flow with mirrors. ofproto-dpif: Avoid unnecessary copying of struct flow with patch ports. ofproto-dpif: pass flow pointer as an argument. ofproto-dpif: Remove flow_storage from struct xlate_in. ofproto-dpif: Avoid extra copy of struct flow in facet_create(). ofproto-dpif: Avoid unnecessary struct flow copies. ofproto/ofproto-dpif.c | 727 ++++++++++++++++++++++++++---------------------- 1 file changed, 397 insertions(+), 330 deletions(-) -- 1.7.10.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev