diff options
author | Ben Pfaff <blp@ovn.org> | 2016-10-23 10:49:27 -0700 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2016-11-28 17:08:15 -0800 |
commit | a9ba22b7890650cca7c7364f626ca488d6c910e9 (patch) | |
tree | 6f46fa40813cfbe97b5fbf4bcbaa2109de0ba566 /ovn | |
parent | 88a4e856b01fafbe7f65dd2d922f14ecb3c0cf39 (diff) | |
download | openvswitch-a9ba22b7890650cca7c7364f626ca488d6c910e9.tar.gz |
ovn-trace: Fix implementation of get_arp and get_nd logical actions.
These actions looked up the MAC binding but failed to update eth.dst with
the result.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Reviewed-by: Aaron Conole <aconole@redhat.com>
Diffstat (limited to 'ovn')
-rw-r--r-- | ovn/utilities/ovn-trace.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/ovn/utilities/ovn-trace.c b/ovn/utilities/ovn-trace.c index 1f41fb4fe..2711f44a5 100644 --- a/ovn/utilities/ovn-trace.c +++ b/ovn/utilities/ovn-trace.c @@ -1188,17 +1188,18 @@ execute_get_mac_bind(const struct ovnact_get_mac_bind *bind, const struct ovntrace_mac_binding *binding = ovntrace_mac_binding_find(dp, port_key, &ip); - const struct eth_addr mac = binding ? binding->mac : eth_addr_zero; + uflow->dl_dst = binding ? binding->mac : eth_addr_zero; if (binding) { ovntrace_node_append(super, OVNTRACE_NODE_ACTION, "/* MAC binding to "ETH_ADDR_FMT". */", - ETH_ADDR_ARGS(mac)); + ETH_ADDR_ARGS(uflow->dl_dst)); } else { ovntrace_node_append(super, OVNTRACE_NODE_ACTION, "/* No MAC binding. */"); } ovntrace_node_append(super, OVNTRACE_NODE_MODIFY, - "eth.dst = "ETH_ADDR_FMT, ETH_ADDR_ARGS(mac)); + "eth.dst = "ETH_ADDR_FMT, + ETH_ADDR_ARGS(uflow->dl_dst)); } static void |