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

[Stephen Hemminger <stephen@networkplumber.org>]

On Thu, 19 Mar 2026 17:16:27 +0000
Bruce Richardson <bruce.richardson@intel.com> wrote:

> 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
> >   

Couldn't there be a common dependency in meson.build
would rather not build tests for platforms that are missing the
secondary_process support.