Re: [PATCH 1/2] virnetdevopenvswitch: unify calculation of ovs and tc. For other interface type, values in tc rules are calculated by multiply by 8*1000 instead of 8*1024.

[Michal Prívozník <mprivozn@xxxxxxxxxx>]

On 11/1/21 8:39 AM, jx8zjs wrote:
> From: zhangjl02 <zhangjl02@xxxxxxxxxx>
> 
> Signed-off-by: zhangjl02 <zhangjl02@xxxxxxxxxx>
> ---
>  src/util/virnetdevopenvswitch.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/util/virnetdevopenvswitch.c b/src/util/virnetdevopenvswitch.c
> index 8955f0fa66..250726a633 100644
> --- a/src/util/virnetdevopenvswitch.c
> +++ b/src/util/virnetdevopenvswitch.c
> @@ -642,7 +642,7 @@ virNetDevOpenvswitchFindUUID(const char *table,
>   * However other_config in ovs qos is in bit.
>   * ingress_policing_rate in ovs interface is in kbit.
>   */
> -#define VIR_NETDEV_TX_TO_OVS 8192
> +#define VIR_NETDEV_TX_TO_OVS 8000
>  #define VIR_NETDEV_RX_TO_OVS 8
>  
>  /**
> 

The other patch should have been merged into this one, because we don't
allow broken tests between commits. I'm squashing it in, breaking the
commit message into multiple lines and pushing.

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal