Re: [PATCH] test/debug: fix test failure on ppc64

[Bruce Richardson <bruce.richardson@intel.com>]

On Tue, Nov 25, 2025 at 10:39:13AM +0000, luca.boccassi@gmail.com wrote:
> From: Luca Boccassi <bluca@debian.org>
> 
> 1001s Running binary with argv[]:'/tmp/autopkgtest.LvdXhc/build.6vT/src/build/app/dpdk-test' '--file-prefix=debug' '-l' '0' '--no-huge' '-m' '2048'
> <...>
> 1001s EAL: Cannot use IOVA as 'PA' since physical addresses are not available
> 
> This is similar to 6b07762e949036389030cf5a1a9bb8a1e4d6ee2d,
> on ppc64 iova-mode=pa is the default, but it doesn't work with --no-huge,
> so add --iova-mode=va to the test arguments.
> 
> Fixes: 2b403dd8fb37 ("test/debug: fix crash with mlx5 devices")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Luca Boccassi <bluca@debian.org>
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>

One comment inline below.
/Bruce

>  app/test/test_debug.c | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/app/test/test_debug.c b/app/test/test_debug.c
> index fe5dd5b02d..5ff9068e2b 100644
> --- a/app/test/test_debug.c
> +++ b/app/test/test_debug.c
> @@ -46,7 +46,7 @@ test_debug(void)
>   * ==========
>   */
>  
> -static const char *test_args[7];
> +static const char *test_args[8];
>  
>  int
>  test_panic(void)
> @@ -158,10 +158,17 @@ test_debug(void)
>  		test_args[4] = "";
>  		test_args[5] = "";
>  		test_args[6] = "";
> +		test_args[7] = "";
>  	} else {
>  		test_args[4] = "--no-huge";
>  		test_args[5] = "-m";
>  		test_args[6] = "2048";
> +#ifdef RTE_ARCH_PPC_64
> +		/* iova=pa is the default, but fails on ppc64 with --no-huge */
> +		test_args[7] = "--iova-mode=va";
> +#else
> +		test_args[7] = "";
> +#endif

Next release we should maybe see if we can remove the #ifdefs and just
iova=va for all architectures.