patch 'test: fix failing tests on FreeBSD' has been queued to stable release 24.11.7
luca.boccassi at gmail.com
luca.boccassi at gmail.com
Thu Jun 11 15:19:12 CEST 2026
Hi,
FYI, your patch has been queued to stable release 24.11.7
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 06/13/26. So please
shout if anyone has objections.
Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.
Queued patches are on a temporary branch at:
https://github.com/bluca/dpdk-stable
This queued commit can be viewed at:
https://github.com/bluca/dpdk-stable/commit/ab3d84b865309fb3f2107c8c9c029c3cd1ce1dd7
Thanks.
Luca Boccassi
---
>From ab3d84b865309fb3f2107c8c9c029c3cd1ce1dd7 Mon Sep 17 00:00:00 2001
From: Bruce Richardson <bruce.richardson at intel.com>
Date: Tue, 21 Apr 2026 20:52:26 +0100
Subject: [PATCH] test: fix failing tests on FreeBSD
[ upstream commit f9276057924256430cfe72f60505551d79c9f9ba ]
Many autotests were failing on FreeBSD, mostly because of the lack of
secondary process support. Skip these tests, and also fix the debug
autotest which crashes on FreeBSD if telemetry is not disabled for the
extra spawned process.
Bugzilla ID: 761
Fixes: 764bf26873b9 ("add FreeBSD support")
Fixes: 50247fe03fe0 ("test/timer: exercise new APIs in secondary process")
Fixes: 086eb64db39e ("test/pdump: add unit test for pdump library")
Fixes: b270daa43b3d ("test: support external memory")
Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
Acked-by: Anatoly Burakov <anatoly.burakov at intel.com>
Tested-by: Adam Hassick <ahassick at iol.unh.edu>
Acked-by: John McNamara <john.mcnamara at intel.com>
---
.mailmap | 1 +
app/test/test_debug.c | 6 +++---
app/test/test_eal_flags.c | 26 +++++++++++++-------------
app/test/test_external_mem.c | 4 ++--
app/test/test_interrupts.c | 2 +-
app/test/test_mp_secondary.c | 4 ++--
app/test/test_pdump.c | 4 ++++
app/test/test_timer_secondary.c | 4 ++--
8 files changed, 28 insertions(+), 23 deletions(-)
diff --git a/.mailmap b/.mailmap
index eadb404f29..bf790d3c6d 100644
--- a/.mailmap
+++ b/.mailmap
@@ -16,6 +16,7 @@ Abhishek Sachan <abhishek.sachan at altran.com>
Abraham Tovar <abrahamx.tovar at intel.com>
Adam Bynes <adambynes at outlook.com>
Adam Dybkowski <adamx.dybkowski at intel.com>
+Adam Hassick <ahassick at iol.unh.edu>
Adam Ludkiewicz <adam.ludkiewicz at intel.com>
Adel Belkhiri <adel.belkhiri at polymtl.ca>
Adham Masarwah <adham at nvidia.com> <adham at mellanox.com>
diff --git a/app/test/test_debug.c b/app/test/test_debug.c
index 5ff9068e2b..59687644bd 100644
--- a/app/test/test_debug.c
+++ b/app/test/test_debug.c
@@ -154,15 +154,15 @@ test_debug(void)
test_args[2] = "-l";
test_args[3] = core;
- if (rte_eal_has_hugepages()) {
+ if (rte_eal_has_hugepages() && RTE_EXEC_ENV_IS_LINUX) {
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";
+ test_args[5] = "-m2048";
+ test_args[6] = "--no-telemetry";
#ifdef RTE_ARCH_PPC_64
/* iova=pa is the default, but fails on ppc64 with --no-huge */
test_args[7] = "--iova-mode=va";
diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c
index e32f83d3c8..d22d74ee61 100644
--- a/app/test/test_eal_flags.c
+++ b/app/test/test_eal_flags.c
@@ -10,88 +10,88 @@
#include <string.h>
#include <stdarg.h>
-#ifdef RTE_EXEC_ENV_WINDOWS
+#ifndef RTE_EXEC_ENV_LINUX
static int
test_missing_c_flag(void)
{
- printf("missing_c_flag not supported on Windows, skipping test\n");
+ printf("missing_c_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_main_lcore_flag(void)
{
- printf("main_lcore_flag not supported on Windows, skipping test\n");
+ printf("main_lcore_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_invalid_n_flag(void)
{
- printf("invalid_n_flag not supported on Windows, skipping test\n");
+ printf("invalid_n_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_no_hpet_flag(void)
{
- printf("no_hpet_flag not supported on Windows, skipping test\n");
+ printf("no_hpet_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_no_huge_flag(void)
{
- printf("no_huge_flag not supported on Windows, skipping test\n");
+ printf("no_huge_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_allow_flag(void)
{
- printf("allow_flag not supported on Windows, skipping test\n");
+ printf("allow_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_invalid_b_flag(void)
{
- printf("invalid_b_flag not supported on Windows, skipping test\n");
+ printf("invalid_b_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_invalid_vdev_flag(void)
{
- printf("invalid_vdev_flag not supported on Windows, skipping test\n");
+ printf("invalid_vdev_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_invalid_r_flag(void)
{
- printf("invalid_r_flag not supported on Windows, skipping test\n");
+ printf("invalid_r_flag only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_memory_flags(void)
{
- printf("memory_flags not supported on Windows, skipping test\n");
+ printf("memory_flags only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_file_prefix(void)
{
- printf("file_prefix not supported on Windows, skipping test\n");
+ printf("file_prefix only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
static int
test_misc_flags(void)
{
- printf("misc_flags not supported on Windows, skipping test\n");
+ printf("misc_flags only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
diff --git a/app/test/test_external_mem.c b/app/test/test_external_mem.c
index 0c64b610a9..d925736c1f 100644
--- a/app/test/test_external_mem.c
+++ b/app/test/test_external_mem.c
@@ -10,11 +10,11 @@
#include <string.h>
#include <fcntl.h>
-#ifdef RTE_EXEC_ENV_WINDOWS
+#ifndef RTE_EXEC_ENV_LINUX
static int
test_external_mem(void)
{
- printf("external_mem not supported on Windows, skipping test\n");
+ printf("external_mem only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
diff --git a/app/test/test_interrupts.c b/app/test/test_interrupts.c
index 3952f9685f..d57fe2fb63 100644
--- a/app/test/test_interrupts.c
+++ b/app/test/test_interrupts.c
@@ -429,7 +429,7 @@ test_interrupt(void)
int ret = -1;
struct rte_intr_handle *test_intr_handle;
- if (RTE_EXEC_ENV_IS_WINDOWS)
+ if (RTE_EXEC_ENV_IS_WINDOWS || RTE_EXEC_ENV_IS_FREEBSD)
return TEST_SKIPPED;
if (test_interrupt_init() < 0) {
diff --git a/app/test/test_mp_secondary.c b/app/test/test_mp_secondary.c
index f3694530a8..d7c9795c57 100644
--- a/app/test/test_mp_secondary.c
+++ b/app/test/test_mp_secondary.c
@@ -15,11 +15,11 @@
#include <string.h>
#include <unistd.h>
-#ifdef RTE_EXEC_ENV_WINDOWS
+#ifndef RTE_EXEC_ENV_LINUX
int
test_mp_secondary(void)
{
- printf("mp_secondary not supported on Windows, skipping test\n");
+ printf("Multiprocess support only present on Linux, skipping test\n");
return TEST_SKIPPED;
}
#else
diff --git a/app/test/test_pdump.c b/app/test/test_pdump.c
index 9f7769707e..39abcf2f1e 100644
--- a/app/test/test_pdump.c
+++ b/app/test/test_pdump.c
@@ -204,6 +204,10 @@ int
test_pdump(void)
{
int ret = 0;
+
+ if (RTE_EXEC_ENV_IS_FREEBSD)
+ return TEST_SKIPPED;
+
if (rte_eal_process_type() == RTE_PROC_PRIMARY) {
printf("IN PRIMARY PROCESS\n");
ret = run_pdump_server_tests();
diff --git a/app/test/test_timer_secondary.c b/app/test/test_timer_secondary.c
index 7d146edefb..8b8e305c19 100644
--- a/app/test/test_timer_secondary.c
+++ b/app/test/test_timer_secondary.c
@@ -18,11 +18,11 @@
#include "test.h"
-#ifdef RTE_EXEC_ENV_WINDOWS
+#ifndef RTE_EXEC_ENV_LINUX
int
test_timer_secondary(void)
{
- printf("timer_secondary not supported on Windows, skipping test\n");
+ printf("timer_secondary only supported on Linux, skipping test\n");
return TEST_SKIPPED;
}
#else
--
2.47.3
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2026-06-11 14:20:01.512968386 +0100
+++ 0003-test-fix-failing-tests-on-FreeBSD.patch 2026-06-11 14:20:01.166744816 +0100
@@ -1 +1 @@
-From f9276057924256430cfe72f60505551d79c9f9ba Mon Sep 17 00:00:00 2001
+From ab3d84b865309fb3f2107c8c9c029c3cd1ce1dd7 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit f9276057924256430cfe72f60505551d79c9f9ba ]
+
@@ -16 +17,0 @@
-Cc: stable at dpdk.org
@@ -34 +35 @@
-index 0e0d83e1c6..895412e568 100644
+index eadb404f29..bf790d3c6d 100644
@@ -46 +47 @@
-index bfda4d1aa3..eda9c3e0e1 100644
+index 5ff9068e2b..59687644bd 100644
@@ -69 +70 @@
-index 5594cc992d..b945553fcd 100644
+index e32f83d3c8..d22d74ee61 100644
@@ -175 +176 @@
-index 1c977b749b..53300983ed 100644
+index 0c64b610a9..d925736c1f 100644
@@ -193 +194 @@
-index 2eb876e65a..3a5be92cd7 100644
+index 3952f9685f..d57fe2fb63 100644
@@ -206 +207 @@
-index 218eacb3bf..d89419c253 100644
+index f3694530a8..d7c9795c57 100644
@@ -224 +225 @@
-index 824caef2d4..64f1bbf21c 100644
+index 9f7769707e..39abcf2f1e 100644
@@ -227 +228 @@
-@@ -198,6 +198,10 @@ int
+@@ -204,6 +204,10 @@ int
@@ -239 +240 @@
-index 57ab39130d..529774f4d0 100644
+index 7d146edefb..8b8e305c19 100644
More information about the stable
mailing list