On Tue, Nov 25, 2025 at 10:39:13AM +0000, [email protected] wrote:
> From: Luca Boccassi <[email protected]>
>
> 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: [email protected]
>
> Signed-off-by: Luca Boccassi <[email protected]>
> ---
Acked-by: Bruce Richardson <[email protected]>
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.
