[ovs-dev] [PATCH OF12+ 1/4] OpenFlow 1.2 fixes and initial OpenFlow 1.3 support
Ben Pfaff
blp at nicira.com
Fri Nov 23 00:14:28 UTC 2012
On Thu, Nov 22, 2012 at 05:24:06PM +0200, Jarno Rajahalme wrote:
> OpenFlow 1.2 protocol fixes: Add OFPP_ANY to include/openflow/openflow-1.1.h,
> and allow it as a port in queue stats request. Make ovs_ofctl use OFPP_ANY
> instead of OFPP_ALL for queue stats requests on OF 1.1+.
> Do not check out_group on flow_mod unless the command is DELETE*.
Thank you for this patch (and the rest)!
The change to ofputil_decode_flow_mod() looks good, except that you
should break the line so that it does not exceed 79 columns wide.
The purpose of ofputil_queue_stats_request and the functions that encode
and decode it is to insulate other code from having to know the
differences between the various OpenFlow versions. Therefore, instead
of modifying handle_queue_stats_request() and ofctl_queue_stats() to
support the various versions, I would modify
ofputil_encode_queue_stats_request() and
ofputil_decode_queue_stats_request() so that they do not have to know
the difference. For example, ofputil_encode_queue_stats_request() could
translate OFPP_ANY to the appropriate value in whatever version it is
encoding, and ofputil_decode_queue_stats_request() could translate the
particular version's value to OFPP_ANY. The clients would then use
OFPP_ANY regardless of version.
Thanks,
Ben.
More information about the dev
mailing list