This feature bit was overlooked when we added support for group stats. Reported-by: Anup Khadka <khadka...@gmail.com> Signed-off-by: Ben Pfaff <b...@nicira.com> --- ofproto/ofproto.c | 3 ++- tests/ofproto.at | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c index 2cb93b0..8bd5f03 100644 --- a/ofproto/ofproto.c +++ b/ofproto/ofproto.c @@ -2997,7 +2997,8 @@ handle_features_request(struct ofconn *ofconn, const struct ofp_header *oh) features.n_buffers = pktbuf_capacity(); features.n_tables = ofproto_get_n_visible_tables(ofproto); features.capabilities = (OFPUTIL_C_FLOW_STATS | OFPUTIL_C_TABLE_STATS | - OFPUTIL_C_PORT_STATS | OFPUTIL_C_QUEUE_STATS); + OFPUTIL_C_PORT_STATS | OFPUTIL_C_QUEUE_STATS | + OFPUTIL_C_GROUP_STATS); if (arp_match_ip) { features.capabilities |= OFPUTIL_C_ARP_MATCH_IP; } diff --git a/tests/ofproto.at b/tests/ofproto.at index 51efd37..3278287 100644 --- a/tests/ofproto.at +++ b/tests/ofproto.at @@ -482,7 +482,7 @@ do AT_CHECK_UNQUOTED([STRIP_XIDS stdout], [0], [dnl OFPT_FEATURES_REPLY (OF1.2): dpid:fedcba9876543210 n_tables:254, n_buffers:256 -capabilities: FLOW_STATS TABLE_STATS PORT_STATS QUEUE_STATS +capabilities: FLOW_STATS TABLE_STATS PORT_STATS GROUP_STATS QUEUE_STATS LOCAL(br0): addr:aa:55:aa:55:00:00 config: $config state: $state @@ -513,7 +513,7 @@ do AT_CHECK_UNQUOTED([STRIP_XIDS stdout], [0], [dnl OFPT_FEATURES_REPLY (OF1.4): dpid:fedcba9876543210 n_tables:254, n_buffers:256 -capabilities: FLOW_STATS TABLE_STATS PORT_STATS QUEUE_STATS +capabilities: FLOW_STATS TABLE_STATS PORT_STATS GROUP_STATS QUEUE_STATS OFPST_PORT_DESC reply (OF1.4): LOCAL(br0): addr:aa:55:aa:55:00:00 config: $config -- 1.7.10.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev