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:22:43 +0000 (10:22 -0700)
commit94aa0d197d3898264ed9761dd5a2089fed40fca3
treeb49c0f1c0c4f809a2856bddcf7e98540f50f68ec
parentccc096898c4618e8869a7696b40ca7f17e8b9560
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