[ovs-dev] [tests+nxm-ofctl 26/42] ovs-ofctl: Reimplement dumping particular tables.

Ben Pfaff blp at nicira.com
Tue Nov 23 22:43:58 UTC 2010


"dump-flows" and "dump-aggregate" are documented to accept a "table"
value to dump only a particular OpenFlow table, but somewhere along the
line this got broken.  This commit should fix it (though I haven't tested
it).
---
 utilities/ovs-ofctl.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c
index 80e3eb3..cc520cc 100644
--- a/utilities/ovs-ofctl.c
+++ b/utilities/ovs-ofctl.c
@@ -478,6 +478,7 @@ do_dump_flows(int argc, char *argv[])
     parse_ofp_str(&pf, NULL, argc > 2 ? argv[2] : "");
     ofputil_cls_rule_to_match(&pf.rule, NXFF_OPENFLOW10, &req->match);
     memset(&req->pad, 0, sizeof req->pad);
+    req->table_id = pf.table_idx;
     req->out_port = htons(pf.out_port);
 
     dump_stats_transaction(argv[1], request);
@@ -494,6 +495,7 @@ do_dump_aggregate(int argc, char *argv[])
     parse_ofp_str(&pf, NULL, argc > 2 ? argv[2] : "");
     ofputil_cls_rule_to_match(&pf.rule, NXFF_OPENFLOW10, &req->match);
     memset(&req->pad, 0, sizeof req->pad);
+    req->table_id = pf.table_idx;
     req->out_port = htons(pf.out_port);
 
     dump_stats_transaction(argv[1], request);
-- 
1.7.1





More information about the dev mailing list