diff options
author | Ben Pfaff <blp@nicira.com> | 2014-11-04 11:17:11 -0800 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2014-11-04 11:17:32 -0800 |
commit | a8c3134833dcfbd3a72ffa5aa5d79e752cea871b (patch) | |
tree | 71554c8e81eed34573af545a1450152a1afa6e62 /tests | |
parent | 5859b0aa8e35b174aa5a50b31fa2f059c62647de (diff) |
ofproto-dpif: Make ofproto/trace a bit more like real packet translation.
Until now, ofproto/trace has looked up the flow itself. xlate_actions()
can do the flow lookup internally and, since that is what happens when a
packet arrives, having it do its own packet lookup makes a lot of sense.
I noticed this in connection with the actset_output field, which
xlate_actions() should set to OFPP_UNSET at the beginning of translation
before looking up the flow. ofproto/trace didn't do that, so it looked
up a rule with actset_output=0 instead. By having xlate_actions() do the
lookup, the behavior can be consistent and correct.
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jarno Rajahalme <jrajahalme@nicira.com>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions