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 <[email protected]> --- 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 2276e89..268ffd9 100644 --- a/lib/ofp-print.c +++ b/lib/ofp-print.c @@ -742,7 +742,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.2.484.gcd07cc5 _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
