ofproto-dpif: Avoid extra flow copy in xlate_actions() for unneeded warnings.
authorBen Pfaff <blp@nicira.com>
Mon, 16 Apr 2012 22:54:37 +0000 (15:54 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 19 Apr 2012 03:38:01 +0000 (20:38 -0700)
commit43d50bc845055bee6243456d934106fc537d793d
tree95bb18bfdb8022ecdf5f85aba8f28253b839d266
parentccb7c863c137635bc02031590a80f3999f46cb32
ofproto-dpif: Avoid extra flow copy in xlate_actions() for unneeded warnings.

The copy of the extra flow copy here was showing up in profiles.  We only
need this copy if we end up doing a "trace" to warn the user.  Most runs
won't ever do that, so don't start making copies until we actually hit
such a case.

This has a small behavioral change in that we'll only get a warning on the
*second* time we hit the resubmit recursion limit, not on the first.  I
doubt that's really a problem.

Signed-off-by: Ben Pfaff <blp@nicira.com>
ofproto/ofproto-dpif.c