[ovs-dev] [PATCH 08/47] ofp-print: Use the prevailing protocol to call ofputil_decode_flow_mod()

Simon Horman horms at verge.net.au
Wed Aug 1 07:01:51 UTC 2012


Pass a protocol based on the prevailing protocol to ofputil_decode_flow_mod()

Note that ofputil_protocol_from_ofp_version() needs to be made
Open Flow 1.1 aware.

Signed-off-by: Simon Horman <horms at verge.net.au>

---

v9
* No change

v8
* Make use of ofputil_protocol_set_tid()
  As suggested by Ben Pfaff

v7
* Manual Rebase

v6
* No change

v5
* No change

v4
* No change

v3
* No change

v2
* Actually pass calculated protocol to ofputil_decode_flow_mod()

Conflicts:
	lib/ofp-print.c
---
 lib/ofp-print.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lib/ofp-print.c b/lib/ofp-print.c
index e8a928e..b4b8bb6 100644
--- a/lib/ofp-print.c
+++ b/lib/ofp-print.c
@@ -685,9 +685,13 @@ ofp_print_flow_mod(struct ds *s, const struct ofp_header *oh, int verbosity)
     bool need_priority;
     enum ofperr error;
     enum ofpraw raw;
+    enum ofputil_protocol protocol;
+
+    protocol = ofputil_protocol_from_ofp_version(oh->version);
+    protocol = ofputil_protocol_set_tid(protocol, true);
 
     ofpbuf_init(&ofpacts, 64);
-    error = ofputil_decode_flow_mod(&fm, oh, OFPUTIL_P_OF10_TID, &ofpacts);
+    error = ofputil_decode_flow_mod(&fm, oh, protocol, &ofpacts);
     if (error) {
         ofpbuf_uninit(&ofpacts);
         ofp_print_error(s, error);
-- 
1.7.10.2.484.gcd07cc5




More information about the dev mailing list