Internally UINT64_MAX is used to denote that the cookie should not be changed, so it seems appropriate to omit this value when pretty printing flow mods.
Signed-off-by: Simon Horman <ho...@verge.net.au> --- v14 * No change v13 * No change v12 * No change v11 * Initial post --- lib/ofp-print.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/ofp-print.c b/lib/ofp-print.c index 8166ab1..ce07b15 100644 --- a/lib/ofp-print.c +++ b/lib/ofp-print.c @@ -743,7 +743,7 @@ ofp_print_flow_mod(struct ds *s, const struct ofp_header *oh, int verbosity) if (ds_last(s) != ' ') { ds_put_char(s, ' '); } - if (fm.new_cookie != htonll(0)) { + if (fm.new_cookie != htonll(0) && fm.new_cookie != htonll(UINT64_MAX)) { ds_put_format(s, "cookie:0x%"PRIx64" ", ntohll(fm.new_cookie)); } if (fm.cookie_mask != htonll(0)) { -- 1.7.10.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev