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

Reply via email to