[ovs-git] [openvswitch/ovs] 3c1bb3: ofp-util: Abstract table miss configuration and fi...

GitHub noreply at github.com
Mon Aug 11 18:34:57 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 3c1bb3962236c6471fea3eb8dd2e5a83b1b4ea81
      https://github.com/openvswitch/ovs/commit/3c1bb3962236c6471fea3eb8dd2e5a83b1b4ea81
  Author: Ben Pfaff <blp at nicira.com>
  Date:   2014-08-11 (Mon, 11 Aug 2014)

  Changed paths:
    M lib/ofp-parse.c
    M lib/ofp-print.c
    M lib/ofp-util.c
    M lib/ofp-util.h
    M ofproto/connmgr.c
    M ofproto/ofproto-dpif.c
    M ofproto/ofproto-provider.h
    M ofproto/ofproto.c
    M ofproto/ofproto.h
    M tests/ofp-print.at
    M tests/ofproto-dpif.at
    M tests/ofproto.at

  Log Message:
  -----------
  ofp-util: Abstract table miss configuration and fix related bugs.

The ofproto implementation has had an abstraction layer on top of
OFPTC11_TABLE_MISS for a while.  This commit pushes that abstraction layer
farther down, into ofp-util.  This will be more useful in an upcoming
commit.

During the conversion I realized that the previous implementation was
not entirely correct.  In particular, the OpenFlow 1.3+ "table mod" was
still being treated as if it had table miss configuration bits, even
though it doesn't.  This commit fixes that issue and updates the tests.

OpenFlow 1.4 adds some more OFPTC_* flags that this new abstraction doesn't
yet support, but OVS didn't support those flags any better before this
commit, so abstracting those is left as future work.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Jarno Rajahalme <jrajahalme at nicira.com>




More information about the git mailing list