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

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 9/29/2017 1:56 PM, Michal Jastrzebski wrote:
> From: Tomasz Kulasek <tomaszx.kulasek@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@intel.com
> Cc: stable@dpdk.org
> 
> Signed-off-by: Tomasz Kulasek <tomaszx.kulasek@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);
>  }
>  
>