[ovs-dev] [PATCH 00/30] ovn-trace: New utility for testing and debugging OVN

Ben Pfaff blp at ovn.org
Mon Aug 8 07:52:34 UTC 2016


You can also pull this from:
        https://github.com/blp/ovs-reviews/tree/ovnacts


On Mon, Aug 08, 2016 at 12:44:39AM -0700, Ben Pfaff wrote:
> The exciting part is the final commit.  I recommend reading the
> documentation for the new program.
> 
> Ben Pfaff (30):
>   ovn-nbctl: Add "sync" command to wait for previous changes to take
>     effect.
>   ovsdb-idl: Wake up ovsdb_idl_loop when a transaction commits.
>   flow: Comment that nw_src holds ARP SPA and nw_dst holds ARP TPA.
>   packets: Improve comment on eth_addr_from_string().
>   ovn-sbctl: Remove unused enum.
>   ofp-actions: Correct member name for write_actions.
>   ofp-actions: Factor OFPACT_PADDED_MEMBERS out into a more general
>     form.
>   ovn: Fix spelling errors in manpages.
>   ovn-nbctl: Add details for common options to manpage.
>   smap: New function smap_get_def().
>   smap: New function smap_get_ullong().
>   expr: Fine-tune parser error message for common typo.
>   expr: Tolerate having no macros.
>   expr: Refine handling of error parameter to expr_annotate().
>   expr: Initialize 'relop' of allocated exprs in crush_and_string().
>   expr: Track writability as part of expr_symbol.
>   expr: Give a subfield a direct pointer to its parent in struct
>     expr_symbol.
>   logical-fields: Get rid of MFF_LOG_REGS in favor of something
>     sensible.
>   lflow: Correct register definitions to use subfields for overlaps.
>   ovn: Use a common symbol table for ovn-controller and test-ovn.
>   logical-fields: Beautify conntrack definitions.
>   meta-flow: Add ovs_be128 member to union mf_subvalue.
>   ovn: Rewrite logical action parsing and encoding library.
>   ovn-util: Add common functions for the default NB and SB databases.
>   lex: Integrate error handling into struct lexer.
>   meta-flow: New functions mf_subfield_copy() and mf_subfield_swap().
>   ovn-northd: Copy name in logical datapath southbound representations.
>   expr: New function expr_evaluate().
>   expr: New function expr_parse_microflow().
>   ovn-trace: New utility.
> 
>  include/openvswitch/flow.h                |    4 +-
>  include/openvswitch/meta-flow.h           |   15 +
>  include/openvswitch/ofp-actions.h         |   16 +-
>  include/openvswitch/util.h                |   22 +-
>  include/ovn/actions.h                     |  304 +++-
>  include/ovn/expr.h                        |   96 +-
>  include/ovn/lex.h                         |   12 +
>  lib/automake.mk                           |    6 +-
>  lib/bfd.c                                 |   60 +-
>  lib/common.xml                            |   14 +
>  lib/daemon.xml                            |  121 ++
>  lib/meta-flow.c                           |   74 +
>  lib/netdev-dpdk.c                         |   16 +-
>  lib/netdev-linux.c                        |   89 +-
>  lib/nx-match.c                            |   31 -
>  lib/nx-match.h                            |    2 -
>  lib/ovsdb-idl.c                           |   27 +-
>  lib/ovsdb-idl.h                           |    3 +-
>  lib/packets.c                             |    3 +
>  lib/smap.c                                |   36 +-
>  lib/smap.h                                |    6 +-
>  lib/ssl.xml                               |   36 +
>  lib/vlog.xml                              |  144 ++
>  ofproto/ofproto-dpif-xlate.c              |    3 +-
>  ovn/controller-vtep/ovn-controller-vtep.c |    5 +-
>  ovn/controller/chassis.c                  |    7 +-
>  ovn/controller/encaps.c                   |    5 +-
>  ovn/controller/lflow.c                    |  174 +--
>  ovn/controller/ovn-controller.c           |    6 +-
>  ovn/controller/patch.c                    |    6 +-
>  ovn/controller/physical.c                 |    6 +-
>  ovn/lib/actions.c                         | 2202 ++++++++++++++++++-----------
>  ovn/lib/automake.mk                       |    1 +
>  ovn/lib/expr.c                            |  977 +++++++------
>  ovn/lib/lex.c                             |  109 ++
>  ovn/lib/logical-fields.c                  |  198 +++
>  ovn/lib/logical-fields.h                  |   19 +-
>  ovn/lib/ovn-util.c                        |   27 +
>  ovn/lib/ovn-util.h                        |    4 +
>  ovn/northd/ovn-northd.c                   |   50 +-
>  ovn/ovn-sb.xml                            |   13 +-
>  ovn/utilities/automake.mk                 |   11 +-
>  ovn/utilities/ovn-ctl.8.xml               |    2 +-
>  ovn/utilities/ovn-nbctl.8.xml             |   53 +-
>  ovn/utilities/ovn-nbctl.c                 |   47 +-
>  ovn/utilities/ovn-sbctl.c                 |   48 +-
>  ovn/utilities/ovn-trace.8.xml             |  287 ++++
>  ovn/utilities/ovn-trace.c                 | 1429 +++++++++++++++++++
>  tests/ovn.at                              |  743 ++++++++--
>  tests/test-ovn.c                          |  348 ++---
>  tests/test-ovsdb.c                        |    5 +-
>  utilities/ovs-vsctl.c                     |    2 +-
>  vswitchd/bridge.c                         |  162 +--
>  53 files changed, 5807 insertions(+), 2279 deletions(-)
>  create mode 100644 lib/common.xml
>  create mode 100644 lib/daemon.xml
>  create mode 100644 lib/ssl.xml
>  create mode 100644 lib/vlog.xml
>  create mode 100644 ovn/lib/logical-fields.c
>  create mode 100644 ovn/utilities/ovn-trace.8.xml
>  create mode 100644 ovn/utilities/ovn-trace.c
> 
> -- 
> 2.1.3
> 



More information about the dev mailing list