[ovs-git] [ovn-org/ovn] 54513a: ovn-sbctl: Sort "dump-flows" output by actions if ...
Ben Pfaff
noreply at github.com
Wed Mar 10 18:29:53 UTC 2021
Branch: refs/heads/master
Home: https://github.com/ovn-org/ovn
Commit: 54513ad0461f940d977b81453551793c09b6c913
https://github.com/ovn-org/ovn/commit/54513ad0461f940d977b81453551793c09b6c913
Author: Ben Pfaff <blp at ovn.org>
Date: 2021-03-10 (Wed, 10 Mar 2021)
Changed paths:
M utilities/ovn-sbctl.c
Log Message:
-----------
ovn-sbctl: Sort "dump-flows" output by actions if all else fails.
The Logical_Flow table can have rows that differ only in actions. Such
rows would have identical matches and priorities and be in the same
datapath. At first glance, any these rows would be a bug, since they'd
match the same packets and there'd be no way to choose between them.
In practice, though, actions can have prerequisites. If two different
flows have mutually exclusive prerequisites, the otherwise identical
flows are actually valid.
This comes up in practice in DNS response flows. There are two of
them, in the same table and pipeline and with the same match expression
"udp.dst == 53 && reg0[4]", but one of them includes the action
"ip4.src <-> ip4.dst;" and the other "ip6.src <-> ip6.dst;". Because
the first has an IPv4 prerequisite and the latter an IPv6 prerequisite,
their matches don't really overlap.
Anyway, regardless of whether they're valid, it's still good to sort
consistently, so that's all a digression.
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Han Zhou <hzhou at ovn.org>
More information about the git
mailing list