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

2014-10-20 Thread 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 alr

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

2014-10-20 Thread Anup Khadka
This is something I have been doing as a temporary workaround (the code wrapped inside ifdef ANUP_EXTENSION). But, I am not sure if the aim was to handle this somewhere else, and I am missing something. features.capabilities = (OFPUTIL_C_FLOW_STATS | OFPUTIL_C_TABLE_STATS |

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

2014-10-20 Thread Anup Khadka
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 (funct