[ovs-dev] [PATCH 05/38] ofp-msgs: Split OFPRAW_OFPST_FLOW_{REQUEST, REPLY}

Ben Pfaff blp at nicira.com
Fri Aug 10 16:35:09 UTC 2012


On Thu, Aug 09, 2012 at 05:49:26PM +0900, Simon Horman wrote:
> Split OFPRAW_OFPST_FLOW_{REQUEST,REPLY} into OpenFlow 1.0 and 1.1+
> versions.
> 
> This is in preparation for adding encoding and decoding of
> Open Flow 1.1 & 1.2 messages.
> 
> Signed-off-by: Simon Horman <horms at verge.net.au>

Because I applied your patches out of order, I had to add the
following:

diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index 34aa3ca..0cf48e6 100644
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
@@ -1487,7 +1487,7 @@ ofputil_encode_flow_stats_request(const struct ofputil_flow_stats_request *fsr,
 
         raw = (fsr->aggregate
                ? OFPRAW_OFPST_AGGREGATE_REQUEST
-               : OFPRAW_OFPST_FLOW_REQUEST);
+               : OFPRAW_OFPST11_FLOW_REQUEST);
         msg = ofpraw_alloc(raw, OFP12_VERSION, NXM_TYPICAL_LEN);
         ofsr = ofpbuf_put_zeros(msg, sizeof *ofsr);
         ofsr->table_id = fsr->table_id;



More information about the dev mailing list