Hi,
Maybe I’ve found a problem on "ofputil_encode_group_desc_request" of ofp-util.c.
According to the OFspec v1.5, the last element of ofp_group_multipart_request
has to be filled with zeros.
/* Body of OFPMP_GROUP_STATS and OFPMP_GROUP_DESC requests. */
struct ofp_group_multipart_request {
uint32_t group_id; /* All groups if OFPG_ALL. */
uint8_t pad[4]; /* Align to 64 bits. */
};
OFP_ASSERT(sizeof(struct ofp_group_multipart_request) == 8);
I’ve checked the source code of "ofputil_encode_group_desc_request", such a
process it could not find.
* ofputil_encode_group_desc_request
https://github.com/openvswitch/ovs/blob/master/lib/ofp-util.c#L7274-L7279
I guess it should be write in the same way as follows.
* ofputil_encode_group_stats_request
https://github.com/openvswitch/ovs/blob/master/lib/ofp-util.c#L7207-L7213
And, I think "ofputil_encode_port_desc_stats_request" also has a similar
problem.
* ofputil_encode_port_desc_stats_request
https://github.com/openvswitch/ovs/blob/master/lib/ofp-util.c#L4009-L4014
Please confirm if it’s correct.
_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss
