> -----Original Message-----
> From: Li Zhang <[email protected]>
> Sent: Thursday, April 1, 2021 7:16 AM
> To: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; Dumitrescu, Cristian
> <[email protected]>; [email protected]; Singh, Jasvinder
> <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]
> Subject: [PATCH 1/2] net/softnic: check meter packet mode
>
> Currently meter algorithms only supports bytes per second(BPS).
> Check packet_mode set to TRUE are rejected.
>
> Signed-off-by: Li Zhang <[email protected]>
> ---
> drivers/net/softnic/rte_eth_softnic_meter.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/net/softnic/rte_eth_softnic_meter.c
> b/drivers/net/softnic/rte_eth_softnic_meter.c
> index 0cbf94e8b0..50db74f52f 100644
> --- a/drivers/net/softnic/rte_eth_softnic_meter.c
> +++ b/drivers/net/softnic/rte_eth_softnic_meter.c
> @@ -128,6 +128,14 @@ meter_profile_check(struct rte_eth_dev *dev,
> NULL,
> "Metering alg not supported");
>
> + /* Not support packet mode, just support byte mode. */
> + if (profile->packet_mode)
> + return -rte_mtr_error_set(error,
> + EINVAL,
> + RTE_MTR_ERROR_TYPE_METER_PROFILE,
Can you please add a new error type for this specific error case, i.e.
RTE_MTR_ERROR_TYPE_METER_PROFILE_PACKET_MODE.
> + NULL,
> + "Meter packet mode not supported");
> +
> return 0;
> }
>
> --
> 2.27.0
