[ovs-dev] [PATCH 00/11] ofproto-dpif: Cleanup, reduce copying struct flow.

Jarno Rajahalme jarno.rajahalme at nsn.com
Fri May 31 11:35:10 UTC 2013


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




More information about the dev mailing list