From b8c5c0dc73c3ff58c55b63b6afa56749706217da Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Fri, 3 Dec 2010 10:32:38 -0800 Subject: [PATCH] ovs-ofctl: Reimplement dumping particular tables. "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 | 2 ++ 1 file changed, 2 insertions(+) diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c index 80e3eb39..cc520cc1 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); -- 2.30.2