Re: [PATCH 7/7] test/debug: fix unit test on FreeBSD

Burakov, Anatoly
2026-03-20T14:00:27Z

On 3/19/2026 6:11 PM, Bruce Richardson wrote:
> Fix the test_debug unit test on FreeBSD by:
> * passing no-telemetry to the additional process calls
> * ensuring that we always use the no-huge and no-shconf params
> * removing the dependency on procfs to get the exe and using argv[0]
> instead
>
> Bugzilla ID: 761 (partial fix)
> Fixes: 764bf26873b9 ("add FreeBSD support")
> Cc: stable@dpdk.org
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
> app/test/process.h | 11 ++++++++---
> app/test/test_debug.c | 6 +++---
> 2 files changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/app/test/process.h b/app/test/process.h
> index df43966a2a..46b69ce330 100644
> --- a/app/test/process.h
> +++ b/app/test/process.h
> @@ -96,7 +96,7 @@ process_dup(const char *const argv[], int numargs, const char *env_value)
> int driver_path_num;
> int argv_num;
> int i, status;
> - char path[32];
> + char *path;
> #ifdef RTE_LIB_PDUMP
> #ifdef RTE_NET_RING
> rte_thread_t thread;
> @@ -188,8 +188,13 @@ process_dup(const char *const argv[], int numargs, const char *env_value)
> /* set the environment variable */
> if (setenv(RECURSIVE_ENV_VAR, env_value, 1) != 0)
> rte_panic("Cannot export environment variable\n");
> -
> - strlcpy(path, "/proc/" self "/" exe, sizeof(path));
> +#ifdef RTE_EXEC_ENV_LINUX
> + char path_buf[32];
> + strlcpy(path_buf, "/proc/" self "/" exe, sizeof(path_buf));
> + path = path_buf;
> +#else
> + path = argv_cpy[0];

Why can't we just use argv_cpy everywhere?

Otherwise,

Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
--
Thanks,
Anatoly