[ovs-git] [openvswitch/ovs] efefbc: ofp-actions: Make formatting and parsing functions...

GitHub noreply at github.com
Thu Feb 1 18:12:57 UTC 2018


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: efefbcae0119b1a9d224e80ca39040c6aed58246
      https://github.com/openvswitch/ovs/commit/efefbcae0119b1a9d224e80ca39040c6aed58246
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-01-31 (Wed, 31 Jan 2018)

  Changed paths:
    M include/openvswitch/ofp-actions.h
    M lib/ofp-actions.c
    M lib/ofp-parse.c
    M lib/ofp-print.c
    M ofproto/ofproto-dpif-trace.c
    M ofproto/ofproto-dpif-xlate.c
    M ofproto/ofproto.c
    M ovn/controller/ofctrl.c
    M ovn/utilities/ovn-sbctl.c
    M tests/test-ovn.c
    M utilities/ovs-ofctl.c

  Log Message:
  -----------
  ofp-actions: Make formatting and parsing functions take a struct argument.

An upcoming commit will add another parameter for parsing and formatting
actions.  It is much easier to add these parameters if they are
encapsulated in a struct, so this commit first makes that change.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Reviewed-by: Yifeng Sun <pkusunyifeng at gmail.com>
Acked-by: Mark Michelson <mmichels at redhat.com>


  Commit: a203f653b1a6a2544603be00966b8d4731d91b69
      https://github.com/openvswitch/ovs/commit/a203f653b1a6a2544603be00966b8d4731d91b69
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-01-31 (Wed, 31 Jan 2018)

  Changed paths:
    M include/openvswitch/ofp-util.h
    M lib/ofp-util.c

  Log Message:
  -----------
  ofp-util: New data structure for mapping between table names and numbers.

This shares the infrastructure for mapping port names and numbers.  It will
be used in an upcoming commit.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Reviewed-by: Yifeng Sun <pkusunyifeng at gmail.com>
Acked-by: Mark Michelson <mmichels at redhat.com>


  Commit: 4bc938ccb3124282c873084a7d11cd8bc37e27a7
      https://github.com/openvswitch/ovs/commit/4bc938ccb3124282c873084a7d11cd8bc37e27a7
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-02-01 (Thu, 01 Feb 2018)

  Changed paths:
    M NEWS
    M include/openvswitch/ofp-actions.h
    M include/openvswitch/ofp-parse.h
    M include/openvswitch/ofp-print.h
    M include/openvswitch/ofp-util.h
    M lib/learn.c
    M lib/learn.h
    M lib/learning-switch.c
    M lib/ofp-actions.c
    M lib/ofp-parse.c
    M lib/ofp-print.c
    M lib/ofp-util.c
    M lib/vconn.c
    M ofproto/ofproto-dpif.c
    M ofproto/ofproto.c
    M ovn/controller/ofctrl.c
    M ovn/controller/pinctrl.c
    M ovn/utilities/ovn-sbctl.c
    M ovn/utilities/ovn-trace.c
    M tests/ofproto.at
    M utilities/ovs-ofctl.8.in
    M utilities/ovs-ofctl.c
    M utilities/ovs-testcontroller.c

  Log Message:
  -----------
  Support accepting and displaying table names in OVS tools.

OpenFlow has little-known support for naming tables.  Open vSwitch has
supported table names for ages, but it has never used or displayed them
outside of commands dedicated to table manipulation.  This commit adds
support for table names in ovs-ofctl.  When a table has a name, it displays
that name in flows and actions, so that, for example, the following:
    table=1, arp, actions=resubmit(,2)
might become:
    table=ingress_acl, arp, actions=resubmit(,mac_learning)
given appropriately named tables.

For backward compatibility, only interactive ovs-ofctl commands by default
display table names; to display them in scripts, use the new --names
option.

This feature was inspired by a talk that Kei Nohguchi presented at Open
vSwitch 2017 Fall Conference.

CC: Kei Nohguchi <kei at nohguchi.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Mark Michelson <mmichels at redhat.com>
Reviewed-by: Yifeng Sun <pkusunyifeng at gmail.com>


Compare: https://github.com/openvswitch/ovs/compare/46ab60bfe562...4bc938ccb312


More information about the git mailing list