[ovs-dev] [PATCH v3 0/5] Write-Actions and Groups

Simon Horman horms at verge.net.au
Thu Sep 5 05:19:11 UTC 2013


Hi,

This series is comprised of three components.

A. Enhance existing ofproto group code

   [PATCH v2 1/5] Use enum ofp11_group_type in struct ofgroup
   [PATCH v2 2/5] ovs-ofctl: Handle any number of buckets in group statistics

B. Support write actions.

   With this in place it is possible to use write actions and clear actions.
   This patch does not depend on other patches in the series.

   [PATCH v2 3/5] Add support for write-actions

C. Translation of indirect and all groups

   With these patches in place it is possible to use indirect and all groups.
   These patches depend on previous patches in the series.

   [PATCH v2 4/5] ofproto-dpif: Implement group callbacks
   [PATCH v2 5/5] Translation of indirect and all groups


Changes since v2:
* "ovs-ofctl: Handle any number of buckets in group statistics"
  + As suggested by Ben Pfaff
    - Vastly simplify the change by using an explicit pointer for
      the 'bucket_stats' member of struct ofputil_group_stats rather
      than implicit variable length array appended to the end of the
      structure.
* "ofproto-dpif: Implement group callbacks" and
  "Translation of indirect and all groups"
  + Rebase for "ofproto-dpif: Hide struct rule_dpif internally"
  + Hide group_dpif in a similar manner to rule_dpif

Overall diffstat

 lib/ofp-actions.c            | 345 +++++++++++++++++++++++++++++++++++++++++--
 lib/ofp-actions.h            |  22 ++-
 lib/ofp-parse.c              |  43 +++++-
 lib/ofp-print.c              |   4 +
 lib/ofp-util.c               |   1 +
 lib/ofp-util.h               |   2 +-
 ofproto/ofproto-dpif-xlate.c | 183 ++++++++++++++++++++---
 ofproto/ofproto-dpif.c       | 174 +++++++++++++++++++++-
 ofproto/ofproto-dpif.h       |  11 ++
 ofproto/ofproto-provider.h   |   2 +-
 ofproto/ofproto.c            |   6 +
 tests/ofp-actions.at         |  26 +++-
 tests/ofproto-dpif.at        |  51 +++++++
 utilities/ovs-ofctl.8.in     |  51 +++++++
 14 files changed, 868 insertions(+), 53 deletions(-)

-- 
1.8.4




More information about the dev mailing list