[ovs-dev] [PATCH 6/9] ofp-util: Improve function to emit a bitmap property.

Ben Pfaff blp at ovn.org
Tue Dec 29 21:09:34 UTC 2015


The callers had some common code that could be reasonably encapsulated, so
this commit does so.

Signed-off-by: Ben Pfaff <blp at ovn.org>
---
 lib/ofp-util.c | 25 +++++++++++--------------
 1 file changed, 11 insertions(+), 14 deletions(-)

diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index f2ae9c7..96d0d63 100644
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
@@ -209,12 +209,16 @@ end_property(struct ofpbuf *msg, size_t start_ofs)
     ofpbuf_padto(msg, ROUND_UP(msg->size, 8));
 }
 
+/* Appends a property to 'msg' whose type is 'type' and whose contents is a
+ * series of property headers, one for each 1-bit in 'bitmap'. */
 static void
-put_bitmap_properties(struct ofpbuf *msg, uint64_t bitmap)
+put_bitmap_property(struct ofpbuf *msg, uint16_t type, uint64_t bitmap)
 {
+    size_t start_ofs = start_property(msg, type);
     for (; bitmap; bitmap = zero_rightmost_1bit(bitmap)) {
         start_property(msg, rightmost_1bit_idx(bitmap));
     }
+    end_property(msg, start_ofs);
 }
 
 /* Given the wildcard bit count in the least-significant 6 of 'wcbits', returns
@@ -4847,14 +4851,9 @@ put_table_action_features(struct ofpbuf *reply,
                           enum ofp13_table_feature_prop_type set_fields_type,
                           int miss_offset, enum ofp_version version)
 {
-    size_t start_ofs;
-
-    start_ofs = start_property(reply, actions_type + miss_offset);
-    put_bitmap_properties(reply,
-                          ntohl(ofpact_bitmap_to_openflow(taf->ofpacts,
-                                                          version)));
-    end_property(reply, start_ofs);
-
+    put_bitmap_property(reply, actions_type + miss_offset,
+                        ntohl(ofpact_bitmap_to_openflow(taf->ofpacts,
+                                                        version)));
     put_fields_property(reply, &taf->set_fields, NULL,
                         set_fields_type + miss_offset, version);
 }
@@ -4867,11 +4866,9 @@ put_table_instruction_features(
     size_t start_ofs;
     uint8_t table_id;
 
-    start_ofs = start_property(reply, OFPTFPT13_INSTRUCTIONS + miss_offset);
-    put_bitmap_properties(reply,
-                          ntohl(ovsinst_bitmap_to_openflow(tif->instructions,
-                                                           version)));
-    end_property(reply, start_ofs);
+    put_bitmap_property(reply, OFPTFPT13_INSTRUCTIONS + miss_offset,
+                        ntohl(ovsinst_bitmap_to_openflow(tif->instructions,
+                                                         version)));
 
     start_ofs = start_property(reply, OFPTFPT13_NEXT_TABLES + miss_offset);
     BITMAP_FOR_EACH_1 (table_id, 255, tif->next) {
-- 
2.1.3




More information about the dev mailing list