[ovs-dev] [PATCH v2 0/2] ofproto: Honour Table Mod settings for table-miss handling

Simon Horman horms at verge.net.au
Wed Nov 13 04:25:21 UTC 2013


This series of patches adds support for using Table Mod settings for
table-miss handling.

This series does not alter the default behaviour of Open vSwitch. And in
particular the OpenFlow1.1 behaviour is the default regardless of which
OpenFlow version is negotiated between the switch and the controller.

Changes between v1 and v2:

* Dropped patches which have been merged. Thanks Ben!
* Use ofproto->up.n_tables instead of N_TABLES as the number
  of tables present.
* Do not use the presence of rules in a table to indicate that it is
  present. Instead treat all tables, other than TBL_INTERNAL, that
  exist in ofproto.up as present. This simplifies things slightly
  and seems to be a more logical approach.
* As pointed out by Yamamoto-san
  - Read config for each table that is missed rather
    than just the first one. A logic bug that somehow I was blind to.


To aid review I have made this available in git:

git://github.com/horms/openvswitch.git devel/table_mod-v2

Simon Horman (2):
  ofproto: Add table config to struct ofproto
  ofproto: Honour Table Mod settings for table-miss handling

 OPENFLOW-1.1+                |    5 
 ofproto/ofproto-dpif-xlate.c |   41 ++++--
 ofproto/ofproto-dpif.c       |  121 ++++++++++++++++++--
 ofproto/ofproto-dpif.h       |   19 ++-
 ofproto/ofproto-provider.h   |   13 ++
 ofproto/ofproto.c            |   47 +++++++
 ofproto/ofproto.h            |    2 
 tests/ofproto-dpif.at        |  248 ++++++++++++++++++++++++++++++++++++++++++
 8 files changed, 459 insertions(+), 37 deletions(-)



More information about the dev mailing list