Re: [dpdk-dev] [PATCH] net/ena: remove useless address check

["Michał Krawczyk" <mk@semihalf.com>]

pon., 4 paź 2021 o 19:27 Ferruh Yigit <ferruh.yigit@intel.com> napisał(a):
>
> Reported by "gcc (GCC) 12.0.0 20211003 (experimental)":
>
> ./drivers/net/ena/ena_rss.c: In function ‘ena_rss_reta_query’:
> ./drivers/net/ena/ena_rss.c:140:66:
>         error: the comparison will always evaluate as ‘false’ for the
>         pointer operand in ‘reta_conf + 136’ must not be NULL
>         [-Werror=address]
>   140 |  (reta_size > RTE_RETA_GROUP_SIZE && ((reta_conf + 1) == NULL)))
>       |                                                       ^~
>
> Fixing it by removing useless check.
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Michal Krawczyk <mk@semihalf.com>

> ---
> Not clear about the intention of the check, removing it in this patch to
> trigger the discussion.

It looks like this condition was unneeded from the beginning and is in
general invalid and comes from some misunderstanding. I'm ok with
removing this as it was dead code anyway.

> ---
>  drivers/net/ena/ena_rss.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/net/ena/ena_rss.c b/drivers/net/ena/ena_rss.c
> index 88afe13da04d..152098410fa2 100644
> --- a/drivers/net/ena/ena_rss.c
> +++ b/drivers/net/ena/ena_rss.c
> @@ -136,8 +136,7 @@ int ena_rss_reta_query(struct rte_eth_dev *dev,
>         int reta_conf_idx;
>         int reta_idx;
>
> -       if (reta_size == 0 || reta_conf == NULL ||
> -           (reta_size > RTE_RETA_GROUP_SIZE && ((reta_conf + 1) == NULL)))
> +       if (reta_size == 0 || reta_conf == NULL)
>                 return -EINVAL;
>
>         if (!(dev->data->dev_conf.rxmode.offloads & DEV_RX_OFFLOAD_RSS_HASH)) {
> --
> 2.31.1
>