[PATCH 1/1] common/sfc_efx/base: remove unreachable code
Andrew Rybchenko
andrew.rybchenko at oktetlabs.ru
Sat Dec 21 18:42:06 CET 2024
NACK
it is the base driver and corresponding code is unreachable with current
build options only.
On 12/21/24 16:27, Ariel Otilibili wrote:
> The default switch case ends with a goto; meaning these instructions are
> never reached.
>
> Coverity issue: 121742
> Fixes: 19b64c6ac3 ("net/sfc/base: import libefx base")
> Signed-off-by: Ariel Otilibili <otilibil at eurecom.fr>
> --
> Cc: stable at dpdk.org
> Cc: Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>
> ---
> drivers/common/sfc_efx/base/efx_mon.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/common/sfc_efx/base/efx_mon.c b/drivers/common/sfc_efx/base/efx_mon.c
> index ee449ef5eb..5706171abd 100644
> --- a/drivers/common/sfc_efx/base/efx_mon.c
> +++ b/drivers/common/sfc_efx/base/efx_mon.c
> @@ -80,9 +80,6 @@ efx_mon_init(
> goto fail2;
> }
>
> - emp->em_emop = emop;
> - return (0);
> -
> fail2:
> EFSYS_PROBE(fail2);
>
More information about the stable
mailing list