[ovs-dev] [PATCH 05/14] ofp-util: Allow use of OpenFlow 12 flow format

Ben Pfaff blp at nicira.com
Mon Nov 12 17:57:26 UTC 2012


On Wed, Nov 07, 2012 at 05:03:03PM +0900, Simon Horman wrote:
> This enables the use of the OpenFlow 12 flow format.
> 
> Signed-off-by: Simon Horman <horms at verge.net.au>

Based on "grep", I'm nervous that we're missing important handling for
OFPUTIL_P_OF12 in a number of places.  For example, it looks like
ofputil_usable_protocols(), ofputil_encode_set_protocol(), and
ofputil_flow_mod_usable_protocols() don't have any provisions for
OF1.2.  Do any of the later patches in this series fix those up?  If so,
then I guess we should try to fold them into this patch.  If not, then
we should probably fix them.



More information about the dev mailing list