Some tunnel flags are purely internal implementation details (primarily
FLOW_TNL_F_UDPIF). These shouldn't be output when we format tunnel
flows, so this masks them out.

Signed-off-by: Jesse Gross <je...@kernel.org>
---
 lib/match.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/match.c b/lib/match.c
index d5deb7d..d78e6a1 100644
--- a/lib/match.c
+++ b/lib/match.c
@@ -1035,9 +1035,9 @@ format_flow_tunnel(struct ds *s, const struct match 
*match)
     if (wc->masks.tunnel.ip_ttl) {
         ds_put_format(s, "tun_ttl=%"PRIu8",", tnl->ip_ttl);
     }
-    if (wc->masks.tunnel.flags) {
+    if (wc->masks.tunnel.flags & FLOW_TNL_F_MASK) {
         format_flags_masked(s, "tun_flags", flow_tun_flag_to_string,
-                            tnl->flags,
+                            tnl->flags & FLOW_TNL_F_MASK,
                             wc->masks.tunnel.flags & FLOW_TNL_F_MASK,
                             FLOW_TNL_F_MASK);
         ds_put_char(s, ',');
-- 
2.7.4

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to