I can review this series. Is it ok if I start looking at it monday? Ethan
On Fri, Mar 11, 2011 at 1:20 PM, Ben Pfaff <[email protected]> wrote: > Paul Ingram requested some new features in ovs-ofctl: > > * "replace-flows" to intelligently update the set of flows in > the switch's flow table. > > * Make "del-flows" able to accept a set of flows from a file. > > This implements these features. > > Ben Pfaff (10): > ofp-print: Consistently print priority only if it is not default. > classifier: Omit all-1-bits mask for VLAN TCI in cls_rule_format(). > ovs-ofctl: Check min flow format support in > negotiate_highest_flow_format(). > ofp-util: New function ofputil_decode_flow_stats_reply(). > ofp-parse: Factor out parse_ofp_add_flow_file() into new function. > ofpbuf: New function ofpbuf_steal_data(). > ovs-ofctl: Use ovs_be32 for big-endian variable. > ovs-ofctl: New commands "replace-flows" and "diff-flows". > ofp-parse: Generalize parse_ofp_add_flow_file() to > parse_ofp_flow_mod_file(). > ovs-ofctl: Make mod-flows and del-flows commands able read files. > > lib/classifier.c | 2 + > lib/dynamic-string.c | 28 +++- > lib/dynamic-string.h | 3 +- > lib/ofp-parse.c | 32 +-- > lib/ofp-parse.h | 9 +- > lib/ofp-print.c | 155 +++----------- > lib/ofp-util.c | 116 ++++++++++- > lib/ofp-util.h | 21 ++- > lib/ofpbuf.c | 21 ++- > lib/ofpbuf.h | 4 +- > tests/ofp-print.at | 46 ++-- > tests/ovs-ofctl.at | 22 ++ > utilities/ovs-controller.c | 3 +- > utilities/ovs-ofctl.8.in | 82 +++++--- > utilities/ovs-ofctl.c | 509 > +++++++++++++++++++++++++++++++++++++++----- > 15 files changed, 787 insertions(+), 266 deletions(-) > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
