Re: [PATCH 3/7] test/pdump: skip test on FreeBSD

Bruce Richardson
2026-03-19T17:16:46Z

On Thu, Mar 19, 2026 at 05:11:37PM +0000, Bruce Richardson wrote:
> FreeBSD doesn't support multiprocess so skip the pdump unit test which
> requires multi-process support.
>
> Bugzilla ID: 761 (partial fix)
> Fixes: 086eb64db39e ("test/pdump: add unit test for pdump library")
> Cc: stable@dpdk.org
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
> app/test/test_pdump.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/app/test/test_pdump.c b/app/test/test_pdump.c
> index 824caef2d4..9d1175b4b0 100644
> --- a/app/test/test_pdump.c
> +++ b/app/test/test_pdump.c
> @@ -5,6 +5,7 @@
> #include <unistd.h>
> #include <stdint.h>
> #include <limits.h>
> +#include <sys/utsname.h>
>
> #include <ethdev_driver.h>
> #include <rte_pdump.h>
> @@ -197,7 +198,12 @@ run_pdump_server_tests(void)
> int
> test_pdump(void)
> {
> + struct utsname name;
> int ret = 0;
> +
> + if (uname(&name) < 0 || strcmp(name.sysname, "FreeBSD") == 0)
> + return TEST_SKIPPED;
> +

Self-NAK, I forgot to update this patch to just use
"RTE_EXEC_ENV_IS_FREEBSD" instead of calling "uname". Will fix in V2.

> if (rte_eal_process_type() == RTE_PROC_PRIMARY) {
> printf("IN PRIMARY PROCESS\n");
> ret = run_pdump_server_tests();
> --
> 2.51.0
>