[ovs-dev] [PATCH v3 1/2] netdev-linux: correct unit of burst parameter

Simon Horman simon.horman at netronome.com
Tue Apr 13 17:16:57 UTC 2021


On Tue, Apr 13, 2021 at 03:54:16PM +0200, Simon Horman wrote:
> On Tue, Apr 13, 2021 at 09:04:02PM +0800, Tonghao Zhang wrote:
> > On Fri, Apr 9, 2021 at 7:52 PM Simon Horman <simon.horman at netronome.com> wrote:
> > >
> > > From: Yong Xu <yong.xu at corigine.com>
> > >
> > > Correct calculation of burst parameter used when configuring TC policer
> > > action for ingress port-based policing in the case where TC offload is in
> > > use. This now matches the value calculated for the case where TC offload is
> > > not in use.
> > >
> > > The division by 8 is to convert from bits to bytes.
> > > Its unclear why 64 was previously used.
> > >
> > > Fixes: e7f6ba220 ("lib/tc: add ingress ratelimiting support for tc-offload")
> > > Signed-off-by: Yong Xu <yong.xu at corigine.com>
> > > Signed-off-by: Simon Horman <simon.horman at netronome.com>
> > > Signed-off-by: Louis Peens <louis.peens at netronome.com>
> > Reviewed-by: Tonghao Zhang <xiangxia.m.yue at gmail.com>
> 
> Thanks, pushed to master.
> 
> I will also work on backporting this to branch 2.14, 2.13 and 2.12.

Backports to branches 2.12 - 2.15 also pushed.


More information about the dev mailing list