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

Lorenzo Bianconi lorenzo.bianconi at redhat.com
Thu Jun 3 16:18:25 UTC 2021


> 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?

Hi Mark,

yes, my intention is to add it in a dedicated patch (sorry to not be clear on
it).

Regards,
Lorenzo

> > 
> > 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
> > 
> 


More information about the dev mailing list