[dpdk-dev] [dpdk-stable] [PATCH] app/testpmd: fix runtime dead code

Ferruh Yigit ferruh.yigit at intel.com
Thu Oct 12 21:28:09 CEST 2017


On 9/29/2017 1:56 PM, Michal Jastrzebski wrote:
> From: Tomasz Kulasek <tomaszx.kulasek at intel.com>
> 
> Coverity reports DEADCODE, as assumes that RTE_LIBRTE_I40E_PMD
> is defined and function compiles entirely. 
> The fix is about to place printf function into #else branch
> of conditional compilation, since it is known at compile
> time if i40e is supported. No need to check it at runtime. 

This is to highlight this is not supported other than defined PMD, and
used like this in similar functions, I am for keeping this and marking
coverity item as false positive.

Hi Jingjing,

What do you think, if you have strong opinion to get this, I will.

Thanks,
ferruh

> 
> Coverity issue: 158646
> Fixes: 37a56cce7b8e ("app/testpmd: enable DDP get info feature")
> Cc: andrey.chilikin at intel.com
> Cc: stable at dpdk.org
> 
> Signed-off-by: Tomasz Kulasek <tomaszx.kulasek at intel.com>
> ---
>  app/test-pmd/cmdline.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> index ccdf239..c604dac 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -13414,8 +13414,8 @@ struct cmd_ddp_info_result {
>  	struct cmd_ddp_info_result *res = parsed_result;
>  	uint8_t *pkg;
>  	uint32_t pkg_size;
> -	int ret = -ENOTSUP;
>  #ifdef RTE_LIBRTE_I40E_PMD
> +	int ret;
>  	uint32_t i;
>  	uint8_t *buff;
>  	uint32_t buff_size;
> @@ -13495,10 +13495,9 @@ struct cmd_ddp_info_result {
>  			free(devs);
>  		}
>  	}
> -	ret = 0;
> +#else
> +	printf("Function not supported in PMD driver\n");
>  #endif
> -	if (ret == -ENOTSUP)
> -		printf("Function not supported in PMD driver\n");
>  	close_ddp_package_file(pkg);
>  }
>  
> 



More information about the dev mailing list