[ovs-dev] [PATCH] netdev-linux: correct unit of burst parameter

Yi-Hung Wei yihung.wei at gmail.com
Wed Feb 24 21:57:19 UTC 2021


On Wed, Feb 24, 2021 at 6:38 AM 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.

A minor nit.

s/Its/It's
>
> Fixes: e7f6ba220 ("lib/tc: add ingress ratelimiting support for tc-offload")
> Signed-off-by: Yong Xu <yong.xu at corigine.com>
> [simon: reworked changelog]
> Signed-off-by: Simon Horman <simon.horman at netronome.com>
> Signed-off-by: Louis Peens <louis.peens at netronome.com>
> ---

This patch LGTM.

Acked-by: Yi-Hung Wei <yihung.wei at gmail.com>


More information about the dev mailing list