The "ovs-appctl dpif/dump-flows" command wasn't updated to print megaflows, so the output would not include wildcards even though the datapath may, so the output was inconsistent and confusing.
Signed-off-by: Justin Pettit <jpet...@nicira.com> --- ofproto/ofproto-dpif.c | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 5c9527b..79e23a4 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -6355,12 +6355,21 @@ ofproto_unixctl_dpif_dump_flows(struct unixctl_conn *conn, HMAP_FOR_EACH (subfacet, hmap_node, &ofproto->backer->subfacets) { struct facet *facet = subfacet->facet; + struct odputil_keybuf maskbuf; + struct ofpbuf mask; if (facet->ofproto != ofproto) { continue; } - odp_flow_key_format(subfacet->key, subfacet->key_len, &ds); + ofpbuf_use_stack(&mask, &maskbuf, sizeof maskbuf); + if (enable_megaflows) { + odp_flow_key_from_mask(&mask, &facet->xout.wc.masks, + &facet->flow, UINT32_MAX); + } + + odp_flow_format(subfacet->key, subfacet->key_len, + mask.data, mask.size, &ds); ds_put_format(&ds, ", packets:%"PRIu64", bytes:%"PRIu64", used:", subfacet->dp_packet_count, subfacet->dp_byte_count); -- 1.7.5.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev