[ovs-discuss] group stats bit in capabilities inside OFPT_FEATURES_REPLY

[Ben Pfaff]

On Mon, Oct 20, 2014 at 03:25:40PM -0400, Anup Khadka wrote:
> Seeing this in OVS 2.1 encounter (but looks like the code is same now too)
> 
> The group stats bit is set to false in capabilities inside
> OFPT_FEATURES_REPLY.
> 
> Is there a reason why that's done, despite the fact that there is already
> support for group stats?
> 
> Looking at the git source right now (function: handle_features_request,
> file: ofproto.c), and I still see:
> features.capabilities = (OFPUTIL_C_FLOW_STATS | OFPUTIL_C_TABLE_STATS |
>                              OFPUTIL_C_PORT_STATS | OFPUTIL_C_QUEUE_STATS);
> 
> Why isn't OFPUTIL_C_GROUP_STATS included there?

It's an oversight.  Thank you for reporting this bug.

I sent out a patch:
        http://openvswitch.org/pipermail/dev/2014-October/047556.html