mbox series

[ovs-dev,v2,0/2] netdev-linux: correct unit of burst parameter

Message ID 20210312115917.6838-1-simon.horman@netronome.com
Headers show
Series netdev-linux: correct unit of burst parameter | expand

Message

Simon Horman March 12, 2021, 11:59 a.m. UTC 
Hi,

this short series corrects the unit of the burst parameter used to
configure TC policer actions in the case case where ingress rate limiting
is configured when TC flow offload is enabled.

Patch: 1/2
Change since v1: minor update to changelog

Adds tests for the configuration of rate limiting.

Patches 2/2
New in v2


Tianyu Yuan (1):
  testsuite: add test cases for ingress_policing parameters

Yong.Xu (1):
  netdev-linux: correct unit of burst parameter

 lib/netdev-linux.c               |  2 +-
 tests/ovs-vsctl.at               | 58 ++++++++++++++++++++++++++++++++
 tests/system-offloads-traffic.at | 47 ++++++++++++++++++++++++++
 3 files changed, 106 insertions(+), 1 deletion(-)

Comments

Simon Horman March 18, 2021, 1:47 p.m. UTC | #1 
On Fri, Mar 12, 2021 at 12:59:15PM +0100, Simon Horman wrote:
> Hi,
> 
> this short series corrects the unit of the burst parameter used to
> configure TC policer actions in the case case where ingress rate limiting
> is configured when TC flow offload is enabled.
> 
> Patch: 1/2
> Change since v1: minor update to changelog
> 
> Adds tests for the configuration of rate limiting.
> 
> Patches 2/2
> New in v2

Hi Marcello,

could I trouble you to review this?
Marcelo Leitner March 25, 2021, 6:45 p.m. UTC | #2 
On Thu, Mar 18, 2021 at 02:47:32PM +0100, Simon Horman wrote:
> On Fri, Mar 12, 2021 at 12:59:15PM +0100, Simon Horman wrote:
> > Hi,
> > 
> > this short series corrects the unit of the burst parameter used to
> > configure TC policer actions in the case case where ingress rate limiting
> > is configured when TC flow offload is enabled.
> > 
> > Patch: 1/2
> > Change since v1: minor update to changelog
> > 
> > Adds tests for the configuration of rate limiting.
> > 
> > Patches 2/2
> > New in v2
> 
> Hi Marcello,
> 
> could I trouble you to review this?

Hi Simon,

Sorry the delay..
Disclaimer: while I probably should be, I'm not that involved with
this part of the code.  But yes :-)

  Marcelo