ofp-util: New function ofputil_encode_barrier_request().

This new function will acquire a new user in an upcoming commit.

Signed-off-by: Ben Pfaff <blp@nicira.com>

diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index 6f8b033eceda8b9936b15af3add5f40911b7c5f9..c377e3e4cc1f1ec0795039b0ea1506d553980b40 100644 (file)
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
@@ -2248,6 +2248,15 @@ make_echo_reply(const struct ofp_header *rq)
     return out;
 }
 
+struct ofpbuf *
+ofputil_encode_barrier_request(void)
+{
+    struct ofpbuf *msg;
+
+    make_openflow(sizeof(struct ofp_header), OFPT_BARRIER_REQUEST, &msg);
+    return msg;
+}
+
 const char *
 ofputil_frag_handling_to_string(enum ofp_config_flags flags)
 {

diff --git a/lib/ofp-util.h b/lib/ofp-util.h
index e2b32ddc1a0ce2e194fdfd2c9af3cefe59261ae9..0b6970a86def18ddb6c17266e4583f9c86643ead 100644 (file)
--- a/lib/ofp-util.h
+++ b/lib/ofp-util.h
@@ -313,6 +313,8 @@ struct ofpbuf *make_packet_in(uint32_t buffer_id, uint16_t in_port,
 struct ofpbuf *make_echo_request(void);
 struct ofpbuf *make_echo_reply(const struct ofp_header *rq);
 
+struct ofpbuf *ofputil_encode_barrier_request(void);
+
 const char *ofputil_frag_handling_to_string(enum ofp_config_flags);
 bool ofputil_frag_handling_from_string(const char *, enum ofp_config_flags *);
 \f

diff --git a/lib/vconn.c b/lib/vconn.c
index 8e6374e7031def445426f852873d8d6ef3e15846..5da5026272fe787c87f8e31c428dc9bb4eaeea36 100644 (file)
--- a/lib/vconn.c
+++ b/lib/vconn.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2008, 2009, 2010, 2011 Nicira Networks.
+ * Copyright (c) 2008, 2009, 2010, 2011, 2012 Nicira Networks.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -730,7 +730,7 @@ vconn_transact_noreply(struct vconn *vconn, struct ofpbuf *request,
     }
 
     /* Send barrier. */
-    make_openflow(sizeof(struct ofp_header), OFPT_BARRIER_REQUEST, &barrier);
+    barrier = ofputil_encode_barrier_request();
     barrier_xid = ((struct ofp_header *) barrier->data)->xid;
     error = vconn_send_block(vconn, barrier);
     if (error) {