On Wed, Feb 24, 2021 at 6:38 AM Simon Horman <simon.hor...@netronome.com> wrote: > > From: Yong Xu <yong...@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.
A minor nit. s/Its/It's > > Fixes: e7f6ba220 ("lib/tc: add ingress ratelimiting support for tc-offload") > Signed-off-by: Yong Xu <yong...@corigine.com> > [simon: reworked changelog] > Signed-off-by: Simon Horman <simon.hor...@netronome.com> > Signed-off-by: Louis Peens <louis.pe...@netronome.com> > --- This patch LGTM. Acked-by: Yi-Hung Wei <yihung....@gmail.com> _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev