ofproto-dpif: Avoid figuring out sFlow and IPFIX actions twice.
authorBen Pfaff <blp@nicira.com>
Sat, 4 May 2013 00:14:19 +0000 (17:14 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 8 May 2013 17:56:27 +0000 (10:56 -0700)
commita75faeef3cb4084105c214db596f119526494645
tree24550c63203148d4c493db75bcccdf32e953a19b
parent7e3a6167d7d268d5681489d66fc39c8af679e039
ofproto-dpif: Avoid figuring out sFlow and IPFIX actions twice.

Not only is it easier to re-use the actions we already have, this avoids
potential problems due to the state that add_sflow_action() and
add_ipfix_action() look at having possibly been changed by
do_xlate_actions().  Currently those functions appear to look only at
the flow's 'in_port', which currently can't change.  However, an upcoming
commit will make it possible for actions to change the flow's 'in_port',
and in addition, with this change, one doesn't have to wonder whether these
functions might look at other state that translation might change.

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