ofp-util: Fix inconsistencies in table features type.
authorJarno Rajahalme <jrajahalme@nicira.com>
Mon, 24 Mar 2014 17:16:03 +0000 (10:16 -0700)
committerJarno Rajahalme <jrajahalme@nicira.com>
Mon, 24 Mar 2014 17:16:19 +0000 (10:16 -0700)
'metadata_match' and 'metadata_write' fields are defined as ovs_be64,
but sometimes used and referred to as uint64_t.

Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
lib/ofp-print.c
ofproto/ofproto-provider.h

index f9fd4c5..da89405 100644 (file)
@@ -2625,7 +2625,7 @@ ofp_print_table_features(struct ds *s, const struct ofp_header *oh)
         ds_put_format(s, "\n  table %"PRIu8":\n", tf.table_id);
         ds_put_format(s, "    name=\"%s\"\n", tf.name);
         ds_put_format(s, "    metadata: match=%#"PRIx64" write=%#"PRIx64"\n",
-                      tf.metadata_match, tf.metadata_write);
+                      ntohll(tf.metadata_match), ntohll(tf.metadata_write));
 
         ds_put_cstr(s, "    config=");
         ofp_print_table_miss_config(s, tf.config);
index dd10f3a..8c11aff 100644 (file)
@@ -835,7 +835,7 @@ struct ofproto_class {
      *
      *   - 'write_setfields' and 'apply_setfields' to OFPXMT12_MASK.
      *
-     *   - 'metadata_match' and 'metadata_write' to UINT64_MAX.
+     *   - 'metadata_match' and 'metadata_write' to OVS_BE64_MAX.
      *
      *   - 'instructions' to OFPIT11_ALL.
      *