ovs-ofctl: Reimplement dumping particular tables.
authorBen Pfaff <blp@nicira.com>
Fri, 3 Dec 2010 18:32:38 +0000 (10:32 -0800)
committerBen Pfaff <blp@nicira.com>
Fri, 3 Dec 2010 18:32:38 +0000 (10:32 -0800)
"dump-flows" and "dump-aggregate" are documented to accept a "table"
value to dump only a particular OpenFlow table, but commit 8050b31d6
"ofp-parse: Refactor flow parsing" broke this, by always dumping table
0.  This commit should fix it (though I haven't tested it).

utilities/ovs-ofctl.c

index 80e3eb392bd4ab076c6dbe265cfd75416f0f6642..cc520cc1e82cbb58797c4e85f45cd7ae2956f85b 100644 (file)
@@ -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);