[ovs-dev] [PATCH v2 ovn 0/4] respin CoPP series

[Mark Gray]

On 24/05/2021 19:44, Lorenzo Bianconi wrote:
> This series respin CoPP support introduced here [0] by Dumitru rebasing on top
> of ovn master branch and adding some missing meters (e.g. bfd or acl reject).
> The main goal of this series is to continue the discussion about the proposed
> approach and to align on CMS APIs.
> For the moment no ddlog support has been added.
> Related bz:
> https://bugzilla.redhat.com/show_bug.cgi?id=1947913
> https://bugzilla.redhat.com/show_bug.cgi?id=1946610

I mentioned the possibility of adding a tutorial. Is that something you
will do in a separate patch?
> 
> Changes since v1:
> - merge patch 3/5 and 4/5
> - cosmetics
> - improve naming conventions
> - add more unit-tests/system-tests
> - remove duplicated flow
> - remove some leftover entries in ovn-nbctl.8.xml
> - add metering for sctp abort packets
> 
> Changes since RFC:
> - drop per-port metering
> - add unit/system tests
> - add reject action metering
> 
> [0] https://patchwork.ozlabs.org/project/openvswitch/list/?series=140778&state=*
> 
> Dumitru Ceara (4):
>   ovn-controller: Add support for Logical_Flow control meters
>   ovn-northd: Add support for CoPP.
>   ovn-northd: Add CoPP policies for flows that punt packets to
>     ovn-controller.
>   NEWS: Add CoPP support.
> 
>  NEWS                      |   1 +
>  controller/lflow.c        |  40 ++-
>  controller/ofctrl.c       |  55 ++--
>  controller/ofctrl.h       |  21 +-
>  controller/physical.c     |   7 +-
>  include/ovn/actions.h     |   3 +-
>  lib/actions.c             | 116 ++++-----
>  lib/automake.mk           |   2 +
>  lib/copp.c                | 143 ++++++++++
>  lib/copp.h                |  59 +++++
>  northd/ovn-northd.c       | 535 ++++++++++++++++++++++++--------------
>  ovn-nb.ovsschema          |  16 +-
>  ovn-nb.xml                |  81 ++++++
>  ovn-sb.ovsschema          |   6 +-
>  ovn-sb.xml                |   6 +
>  tests/atlocal.in          |   3 +
>  tests/ovn-controller.at   |  48 ++++
>  tests/ovn-northd.at       |  81 ++++++
>  tests/ovn.at              |   4 +-
>  tests/system-ovn.at       | 135 ++++++++++
>  utilities/ovn-nbctl.8.xml | 119 +++++++++
>  utilities/ovn-nbctl.c     | 178 +++++++++++++
>  22 files changed, 1360 insertions(+), 299 deletions(-)
>  create mode 100644 lib/copp.c
>  create mode 100644 lib/copp.h
>