From c3da0ae652137457c7abe1ebcd2b049178d69a66 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Thu, 21 Jan 2021 14:49:22 -0800 Subject: ofp-table: Improve log message in ofputil_decode_table_features(). Signed-off-by: Ben Pfaff Acked-by: Ilya Maximets --- lib/ofp-table.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'lib/ofp-table.c') diff --git a/lib/ofp-table.c b/lib/ofp-table.c index 6ed6c5e3b..a956754f2 100644 --- a/lib/ofp-table.c +++ b/lib/ofp-table.c @@ -523,9 +523,12 @@ ofputil_decode_table_features(struct ofpbuf *msg, /* OpenFlow 1.3 and 1.4 always require all of the required properties. * OpenFlow 1.5 requires all of them if any property is present. */ - if ((seen & OFPTFPT13_REQUIRED) != OFPTFPT13_REQUIRED - && (tf->any_properties || oh->version < OFP15_VERSION)) { - VLOG_WARN_RL(&rl, "table features message missing required property"); + unsigned int missing = (seen & OFPTFPT13_REQUIRED) ^ OFPTFPT13_REQUIRED; + if (missing && (tf->any_properties || oh->version < OFP15_VERSION)) { + VLOG_WARN_RL(&rl, + "table features message missing %u required " + "properties, including property %d", + count_1bits(missing), rightmost_1bit_idx(missing)); return OFPERR_OFPTFFC_BAD_FEATURES; } -- cgit v1.2.1