[ovs-dev] [PATCH v3 0/2] add port-based ingress policing based packet-per-second rate-limiting

[Simon Horman]

On Wed, Jun 09, 2021 at 11:52:07AM +0200, Simon Horman wrote:
> Hi,
> 
> this short test adds support for add port-based ingress policing based
> packet-per-second rate-limiting. This builds on existing support for
> byte-per-second rate limiting.
> 
> Changes since v2
> 
> * Remove the for loop in function nl_msg_put_act_police()
> * Remove unused enum definition for qos type
> * Define 1 kpkts as 1000 packets rather than 1024 packets
> * Update the description for the new item in ovsdb
> * Fix some format warnings according robot's comments
> 
> Changes between v1 and v2
> * Correct typo: s/comsume/consume/

Hi Marcelo,

could I trouble you for a review of this series.
I believe it addresses the issues that you raised in v2.

> Tianyu Yuan (1):
>   add test cases for ingress_policing_kpkts parameters
> 
> Yong Xu (1):
>   add port-based ingress policing based packet-per-second rate-limiting
> 
>  acinclude.m4                     |   6 +-
>  include/linux/pkt_cls.h          |   6 +-
>  lib/netdev-dpdk.c                |   4 +-
>  lib/netdev-linux-private.h       |   4 +-
>  lib/netdev-linux.c               | 109 ++++++++++++++++++++++---------
>  lib/netdev-provider.h            |  11 ++--
>  lib/netdev.c                     |  15 +++--
>  lib/netdev.h                     |   3 +-
>  tests/atlocal.in                 |  17 ++++-
>  tests/ovs-vsctl.at               |  23 +++++++
>  tests/system-offloads-traffic.at |  50 ++++++++++++++
>  vswitchd/bridge.c                |   6 +-
>  vswitchd/vswitch.ovsschema       |  10 ++-
>  vswitchd/vswitch.xml             |  59 ++++++++++++++++-
>  14 files changed, 266 insertions(+), 57 deletions(-)
> 
> -- 
> 2.20.1
>