Re: [dpdk-stable] [PATCH] app/proc-info: fix compilation with no security

[Thierry Herbelot <thierry.herbelot@6wind.com>]

On 10/18/19 9:56 AM, Hemant Agrawal wrote:
> This patch fixes the compilation issues when
> RTE_LIBRTE_SECURITY is disabled.
> 
> Fixes: 8a37f37fc243 ("app/procinfo: add --show-port")
> Cc: stable@dpdk.org
> Cc: vipin.varghese@intel.com
> 
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> ---
>   app/proc-info/main.c | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/app/proc-info/main.c b/app/proc-info/main.c
> index a3f854b89..8cf501d06 100644
> --- a/app/proc-info/main.c
> +++ b/app/proc-info/main.c
> @@ -30,7 +30,9 @@
>   #include <rte_string_fns.h>
>   #include <rte_metrics.h>
>   #include <rte_cycles.h>
> +#ifdef RTE_LIBRTE_SECURITY
>   #include <rte_security.h>
> +#endif
>   #include <rte_cryptodev.h>
>   #include <rte_tm.h>
>   #include <rte_hexdump.h>
> @@ -733,6 +735,7 @@ show_port(void)
>   		}
>   
>   		printf("  - cyrpto context\n");
> +#ifdef RTE_LIBRTE_SECURITY
>   		void *p_ctx = rte_eth_dev_get_sec_ctx(i);
>   		printf("\t  -- security context - %p\n", p_ctx);
>   
> @@ -751,6 +754,7 @@ show_port(void)
>   						s_cap->crypto_capabilities->op);
>   			}
>   		}
> +#endif
>   	}
>   
>   	STATS_BDR_STR(50, "");
> 

Acked-by: Thierry Herbelot <thierry.herbelot@6wind.com>

-- 
Thierry Herbelot
6WIND
Senior Software Engineer