[ovs-dev] [ovs-ofctl 00/10] Add more ovs-ofctl features

Ethan Jackson ethan at nicira.com
Fri Mar 11 13:22:41 PST 2011


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 <blp at nicira.com> 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
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list