* [PATCH 1/2] eal: add location info to interrupt error message
@ 2026-01-26 16:45 Bruce Richardson
2026-01-26 16:45 ` [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls Bruce Richardson
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Bruce Richardson @ 2026-01-26 16:45 UTC (permalink / raw)
To: dev; +Cc: Bruce Richardson
The error message about the interrupt being unallocated is called from
multiple positions, so if seen in the logs the user cannot tell which
function is actually generating the message. Add function name and line
number info to the message to clarify things.
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
lib/eal/common/eal_common_interrupts.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/eal/common/eal_common_interrupts.c b/lib/eal/common/eal_common_interrupts.c
index b42fa862f3..a2a310750a 100644
--- a/lib/eal/common/eal_common_interrupts.c
+++ b/lib/eal/common/eal_common_interrupts.c
@@ -17,7 +17,7 @@
/* Macros to check for valid interrupt handle */
#define CHECK_VALID_INTR_HANDLE(intr_handle) do { \
if (intr_handle == NULL) { \
- EAL_LOG(DEBUG, "Interrupt instance unallocated"); \
+ EAL_LOG(DEBUG, "%s, ln %d: Interrupt instance unallocated", __func__, __LINE__); \
rte_errno = EINVAL; \
goto fail; \
} \
--
2.51.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls 2026-01-26 16:45 [PATCH 1/2] eal: add location info to interrupt error message Bruce Richardson @ 2026-01-26 16:45 ` Bruce Richardson 2026-02-03 14:17 ` David Marchand 2026-01-26 18:16 ` [PATCH 1/2] eal: add location info to interrupt error message Stephen Hemminger 2026-02-13 15:22 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests Bruce Richardson 2 siblings, 1 reply; 10+ messages in thread From: Bruce Richardson @ 2026-01-26 16:45 UTC (permalink / raw) To: dev; +Cc: Bruce Richardson When we make recursive calls to dpdk-test to check flag support, either as a secondary process or as another primary process, add the log-level flag to give extra EAL output to help with debugging any issues. For calls that are spawning another primary process, pass in --no-pci flag too, to skip device probing. Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> --- app/test/test_eal_flags.c | 229 ++++++++++++++++++++------------------ 1 file changed, 122 insertions(+), 107 deletions(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 659a6745fd..1f8255cb3d 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -121,6 +121,9 @@ test_misc_flags(void) #define no_shconf "--no-shconf" #define allow "--allow" #define vdev "--vdev" +#define no_pci "--no-pci" +#define eal_debug_logs "--log-level=lib.eal:debug" +#define bus_debug_logs "--log-level=bus.*:debug" #define memtest "memtest" #define memtest1 "memtest1" #define memtest2 "memtest2" @@ -340,26 +343,26 @@ test_allow_flag(void) snprintf(prefix, sizeof(prefix), "--file-prefix=%s", tmp); #endif - const char *wlinval[][7] = { - {prgname, prefix, mp_flag, + const char *wlinval[][8] = { + {prgname, prefix, mp_flag, eal_debug_logs, allow, "error", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:0:0", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:error:0.1", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:0:0.1error", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "error0:0:0.1", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:0:0.1.2", "", ""}, }; /* Test with valid allow option */ - const char *wlval1[] = {prgname, prefix, mp_flag, + const char *wlval1[] = {prgname, prefix, mp_flag, eal_debug_logs, allow, "00FF:09:0B.3"}; - const char *wlval2[] = {prgname, prefix, mp_flag, + const char *wlval2[] = {prgname, prefix, mp_flag, eal_debug_logs, allow, "09:0B.3", allow, "0a:0b.1"}; - const char *wlval3[] = {prgname, prefix, mp_flag, + const char *wlval3[] = {prgname, prefix, mp_flag, eal_debug_logs, allow, "09:0B.3,type=test", allow, "08:00.1,type=normal", }; @@ -408,16 +411,16 @@ test_invalid_b_flag(void) snprintf(prefix, sizeof(prefix), "--file-prefix=%s", tmp); #endif - const char *blinval[][5] = { - {prgname, prefix, mp_flag, "-b", "error"}, - {prgname, prefix, mp_flag, "-b", "0:0:0"}, - {prgname, prefix, mp_flag, "-b", "0:error:0.1"}, - {prgname, prefix, mp_flag, "-b", "0:0:0.1error"}, - {prgname, prefix, mp_flag, "-b", "error0:0:0.1"}, - {prgname, prefix, mp_flag, "-b", "0:0:0.1.2"}, + const char *blinval[][6] = { + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "error"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:0:0"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:error:0.1"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:0:0.1error"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "error0:0:0.1"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:0:0.1.2"}, }; /* Test with valid blocklist option */ - const char *blval[] = {prgname, prefix, mp_flag, + const char *blval[] = {prgname, prefix, mp_flag, eal_debug_logs, "-b", "FF:09:0B.3"}; int i; @@ -454,18 +457,18 @@ test_invalid_vdev_flag(void) #endif /* Test with invalid vdev option */ - const char *vdevinval[] = {prgname, prefix, no_huge, - vdev, "eth_dummy"}; + const char *vdevinval[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "eth_dummy"}; /* Test with valid vdev option */ - const char *vdevval1[] = {prgname, prefix, no_huge, - vdev, "net_ring0"}; + const char *vdevval1[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "net_ring0"}; - const char *vdevval2[] = {prgname, prefix, no_huge, - vdev, "net_ring0,args=test"}; + const char *vdevval2[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "net_ring0,args=test"}; - const char *vdevval3[] = {prgname, prefix, no_huge, - vdev, "net_ring0,nodeaction=r1:0:CREATE"}; + const char *vdevval3[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "net_ring0,nodeaction=r1:0:CREATE"}; if (launch_proc(vdevinval) == 0) { printf("Error (line %d) - process did run ok with invalid vdev parameter\n", @@ -514,14 +517,14 @@ test_invalid_r_flag(void) snprintf(prefix, sizeof(prefix), "--file-prefix=%s", tmp); #endif - const char *rinval[][5] = { - {prgname, prefix, mp_flag, "-r", "error"}, - {prgname, prefix, mp_flag, "-r", "0"}, - {prgname, prefix, mp_flag, "-r", "-1"}, - {prgname, prefix, mp_flag, "-r", "17"}, + const char *rinval[][6] = { + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "error"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "0"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "-1"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "17"}, }; /* Test with valid blocklist option */ - const char *rval[] = {prgname, prefix, mp_flag, "-r", "16"}; + const char *rval[] = {prgname, prefix, mp_flag, eal_debug_logs, "-r", "16"}; int i; @@ -560,77 +563,77 @@ test_missing_c_flag(void) #endif /* -c flag but no coremask value */ - const char *argv1[] = { prgname, prefix, mp_flag, "-c"}; + const char *argv1[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c"}; /* No -c, -l or --lcores flag at all */ const char *argv2[] = { prgname, prefix, mp_flag}; /* bad coremask value */ - const char *argv3[] = { prgname, prefix, mp_flag, + const char *argv3[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "error" }; /* sanity check of tests - valid coremask value */ - const char *argv4[] = { prgname, prefix, mp_flag, + const char *argv4[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "1" }; /* -l flag but no corelist value */ - const char *argv5[] = { prgname, prefix, mp_flag, + const char *argv5[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l"}; - const char *argv6[] = { prgname, prefix, mp_flag, + const char *argv6[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", " " }; /* bad corelist values */ - const char *argv7[] = { prgname, prefix, mp_flag, + const char *argv7[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "error" }; - const char *argv8[] = { prgname, prefix, mp_flag, + const char *argv8[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1-" }; - const char *argv9[] = { prgname, prefix, mp_flag, + const char *argv9[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1," }; - const char *argv10[] = { prgname, prefix, mp_flag, + const char *argv10[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1#2" }; /* core number is negative value */ - const char * const argv11[] = { prgname, prefix, mp_flag, + const char * const argv11[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "-5" }; - const char * const argv12[] = { prgname, prefix, mp_flag, + const char * const argv12[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "-5-7" }; /* core number is maximum value */ - const char * const argv13[] = { prgname, prefix, mp_flag, + const char * const argv13[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", RTE_STR(RTE_MAX_LCORE) }; - const char * const argv14[] = { prgname, prefix, mp_flag, + const char * const argv14[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1-"RTE_STR(RTE_MAX_LCORE) }; /* sanity check test - valid corelist value */ - const char * const argv15[] = { prgname, prefix, mp_flag, + const char * const argv15[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1-2,3" }; /* --lcores flag but no lcores value */ - const char * const argv16[] = { prgname, prefix, mp_flag, + const char * const argv16[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores" }; - const char * const argv17[] = { prgname, prefix, mp_flag, + const char * const argv17[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", " " }; /* bad lcores value */ - const char * const argv18[] = { prgname, prefix, mp_flag, + const char * const argv18[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "1-3-5" }; - const char * const argv19[] = { prgname, prefix, mp_flag, + const char * const argv19[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "0-1,,2" }; - const char * const argv20[] = { prgname, prefix, mp_flag, + const char * const argv20[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "0-,1" }; - const char * const argv21[] = { prgname, prefix, mp_flag, + const char * const argv21[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(0-,2-4)" }; - const char * const argv22[] = { prgname, prefix, mp_flag, + const char * const argv22[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(-1,2)" }; - const char * const argv23[] = { prgname, prefix, mp_flag, + const char * const argv23[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(2-4)@(2-4-6)" }; - const char * const argv24[] = { prgname, prefix, mp_flag, + const char * const argv24[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(a,2)" }; - const char * const argv25[] = { prgname, prefix, mp_flag, + const char * const argv25[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "1-3@(1,3)" }; - const char * const argv26[] = { prgname, prefix, mp_flag, + const char * const argv26[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "3@((1,3)" }; - const char * const argv27[] = { prgname, prefix, mp_flag, + const char * const argv27[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(4-7)=(1,3)" }; - const char * const argv28[] = { prgname, prefix, mp_flag, + const char * const argv28[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "[4-7]@(1,3)" }; /* sanity check of tests - valid lcores value */ - const char * const argv29[] = { prgname, prefix, mp_flag, + const char * const argv29[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "0-1,2@(5-7),(3-5)@(0,2),(0,6),7"}; /* check an invalid cpu value >= CPU_SETSIZE */ - const char * const argv30[] = { prgname, prefix, mp_flag, + const char * const argv30[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "3@" RTE_STR(CPU_SETSIZE) }; if (launch_proc(argv2) != 0) { @@ -722,21 +725,21 @@ test_main_lcore_flag(void) return TEST_SKIPPED; /* --main-lcore flag but no value */ - const char *argv1[] = { prgname, prefix, mp_flag, + const char *argv1[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore"}; /* --main-lcore flag with invalid value */ - const char *argv2[] = { prgname, prefix, mp_flag, + const char *argv2[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "-1"}; - const char *argv3[] = { prgname, prefix, mp_flag, + const char *argv3[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "X"}; /* main lcore not in coremask */ - const char *argv4[] = { prgname, prefix, mp_flag, + const char *argv4[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "2"}; /* valid value */ - const char *argv5[] = { prgname, prefix, mp_flag, + const char *argv5[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "1"}; /* valid value set before coremask */ - const char *argv6[] = { prgname, prefix, mp_flag, + const char *argv6[] = { prgname, prefix, mp_flag, eal_debug_logs, "--main-lcore", "1", "-c", "3"}; if (launch_proc(argv1) == 0 @@ -832,9 +835,9 @@ test_no_hpet_flag(void) #endif /* With --no-hpet */ - const char *argv1[] = {prgname, prefix, mp_flag, no_hpet}; + const char *argv1[] = {prgname, prefix, mp_flag, eal_debug_logs, no_hpet}; /* Without --no-hpet */ - const char *argv2[] = {prgname, prefix, mp_flag}; + const char *argv2[] = {prgname, prefix, mp_flag, eal_debug_logs}; if (launch_proc(argv1) != 0) { printf("Error (line %d) - process did not run ok with --no-hpet flag\n", __LINE__); @@ -997,105 +1000,117 @@ test_misc_flags(void) * No further testing of output done. */ /* sanity check - failure with invalid option */ - const char *argv0[] = {prgname, prefix, mp_flag, "--invalid-opt"}; + const char *argv0[] = {prgname, prefix, mp_flag, eal_debug_logs, "--invalid-opt"}; /* With --no-pci */ - const char *argv1[] = {prgname, prefix, mp_flag, "--no-pci"}; + const char *argv1[] = {prgname, prefix, mp_flag, eal_debug_logs, "--no-pci"}; /* With -v */ - const char *argv2[] = {prgname, prefix, mp_flag, "-v"}; + const char *argv2[] = {prgname, prefix, mp_flag, eal_debug_logs, "-v"}; /* With valid --syslog */ - const char *argv3[] = {prgname, prefix, mp_flag, "--syslog=user"}; + const char *argv3[] = {prgname, prefix, mp_flag, eal_debug_logs, "--syslog=user"}; /* With empty --syslog (now defaults) */ - const char *argv4[] = {prgname, prefix, mp_flag, "--syslog"}; + const char *argv4[] = {prgname, prefix, mp_flag, eal_debug_logs, "--syslog"}; /* With invalid --syslog */ - const char *argv5[] = {prgname, prefix, mp_flag, "--syslog=invalid"}; + const char *argv5[] = {prgname, prefix, mp_flag, eal_debug_logs, "--syslog=invalid"}; /* With no-sh-conf, also use no-huge to ensure this test runs on BSD */ - const char *argv6[] = {prgname, "-m", DEFAULT_MEM_SIZE, + const char *argv6[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, no_shconf, nosh_prefix, no_huge}; /* With --huge-dir */ const char *argv7[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", hugepath}; /* With empty --huge-dir (should fail) */ const char *argv8[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir"}; /* With invalid --huge-dir */ const char *argv9[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", "invalid"}; /* With invalid --huge-dir sub-directory */ const char *argv10[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", hugepath_dir}; /* With valid --huge-dir sub-directory */ const char *argv11[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", hugepath_dir2}; /* Secondary process with invalid --huge-dir (should run as flag has no * effect on secondary processes) */ - const char *argv12[] = {prgname, prefix, mp_flag, + const char *argv12[] = {prgname, prefix, mp_flag, eal_debug_logs, "--huge-dir", "invalid"}; /* try running with base-virtaddr param */ const char *argv13[] = {prgname, "--file-prefix=virtaddr", + eal_debug_logs, no_pci, "--base-virtaddr=0x23456789"}; /* try running with --vfio-intr INTx flag */ const char *argv14[] = {prgname, "--file-prefix=intr", + eal_debug_logs, no_pci, "--vfio-intr=legacy"}; /* try running with --vfio-intr MSI flag */ const char *argv15[] = {prgname, "--file-prefix=intr", + eal_debug_logs, no_pci, "--vfio-intr=msi"}; /* try running with --vfio-intr MSI-X flag */ const char *argv16[] = {prgname, "--file-prefix=intr", + eal_debug_logs, no_pci, "--vfio-intr=msix"}; /* try running with --vfio-intr invalid flag */ const char *argv17[] = {prgname, "--file-prefix=intr", + eal_debug_logs, no_pci, "--vfio-intr=invalid"}; /* With process type as auto-detect */ const char * const argv18[] = {prgname, "--file-prefix=auto", + eal_debug_logs, no_pci, "--proc-type=auto"}; /* With process type as auto-detect with no-shconf */ - const char * const argv19[] = {prgname, "--proc-type=auto", + const char * const argv19[] = {prgname, "--proc-type=auto", eal_debug_logs, no_pci, no_shconf, nosh_prefix, no_huge}; /* With process type as --create-uio-dev flag */ const char * const argv20[] = {prgname, "--file-prefix=uiodev", + eal_debug_logs, no_pci, "--create-uio-dev"}; /* Try running with --huge-worker-stack flag */ - const char * const argv21[] = {prgname, prefix, mp_flag, + const char * const argv21[] = {prgname, prefix, mp_flag, eal_debug_logs, "--huge-worker-stack"}; /* Try running with --huge-worker-stack=512 flag */ - const char * const argv22[] = {prgname, prefix, mp_flag, + const char * const argv22[] = {prgname, prefix, mp_flag, eal_debug_logs, "--huge-worker-stack=512"}; /* Try running with --log-timestamp */ - const char * const argv23[] = {prgname, prefix, mp_flag, + const char * const argv23[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-timestamp" }; /* Try running with --log-timestamp=iso */ - const char * const argv24[] = {prgname, prefix, mp_flag, + const char * const argv24[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-timestamp=iso" }; /* Try running with invalid timestamp */ - const char * const argv25[] = {prgname, prefix, mp_flag, + const char * const argv25[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-timestamp=invalid" }; /* Try running with --log-color */ - const char * const argv26[] = {prgname, prefix, mp_flag, + const char * const argv26[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-color" }; /* Try running with --log-color=never */ - const char * const argv27[] = {prgname, prefix, mp_flag, + const char * const argv27[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-color=never" }; /* Try running with --log-color=invalid */ - const char * const argv28[] = {prgname, prefix, mp_flag, + const char * const argv28[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-color=invalid" }; /* run all tests also applicable to FreeBSD first */ @@ -1285,47 +1300,47 @@ test_file_prefix(void) #endif /* this should fail unless the test itself is run with "memtest" prefix */ - const char *argv0[] = {prgname, mp_flag, "-m", + const char *argv0[] = {prgname, mp_flag, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest }; /* primary process with memtest1 and default mem mode */ - const char *argv1[] = {prgname, "-m", + const char *argv1[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest1 }; /* primary process with memtest1 and legacy mem mode */ - const char *argv2[] = {prgname, "-m", + const char *argv2[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest1, "--legacy-mem" }; /* primary process with memtest2 and legacy mem mode */ - const char *argv3[] = {prgname, "-m", + const char *argv3[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest2, "--legacy-mem" }; /* primary process with memtest2 and default mem mode */ - const char *argv4[] = {prgname, "-m", + const char *argv4[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest2 }; /* primary process with --in-memory mode */ - const char * const argv5[] = {prgname, "-m", + const char * const argv5[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--in-memory" }; /* primary process with memtest1 and --in-memory mode */ - const char * const argv6[] = {prgname, "-m", + const char * const argv6[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--in-memory", "--file-prefix=" memtest1 }; /* primary process with parent file-prefix and --in-memory mode */ - const char * const argv7[] = {prgname, "-m", + const char * const argv7[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--in-memory", "--file-prefix", prefix }; /* primary process with memtest1 and --single-file-segments mode */ - const char * const argv8[] = {prgname, "-m", + const char * const argv8[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--single-file-segments", "--file-prefix=" memtest1 }; /* primary process with memtest1 and --huge-unlink=never mode */ - const char * const argv9[] = {prgname, "-m", + const char * const argv9[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--huge-unlink=never", "--file-prefix=" memtest1 }; @@ -1581,45 +1596,45 @@ test_memory_flags(void) #endif /* valid -m flag and mp flag */ - const char *argv0[] = {prgname, prefix, mp_flag, + const char *argv0[] = {prgname, prefix, mp_flag, eal_debug_logs, "-m", DEFAULT_MEM_SIZE}; /* valid -m flag */ - const char *argv1[] = {prgname, + const char *argv1[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE}; /* valid (zero) --socket-mem flag */ char arg2_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv2[] = {prgname, + const char *argv2[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg2_socket_mem}; /* invalid (incomplete) --socket-mem flag */ char arg3_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv3[] = {prgname, + const char *argv3[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg3_socket_mem}; /* invalid (mixed with invalid data) --socket-mem flag */ char arg4_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv4[] = {prgname, + const char *argv4[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg4_socket_mem}; /* invalid (with numeric value as last character) --socket-mem flag */ char arg5_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv5[] = {prgname, + const char *argv5[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg5_socket_mem}; /* invalid (with empty socket) --socket-mem flag */ char arg6_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv6[] = {prgname, + const char *argv6[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg6_socket_mem}; /* invalid (null) --socket-mem flag */ - const char *argv7[] = {prgname, + const char *argv7[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, "--socket-mem="}; /* valid --socket-mem specified together with -m flag */ char arg8_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv8[] = {prgname, + const char *argv8[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE, arg8_socket_mem}; @@ -1637,12 +1652,12 @@ test_memory_flags(void) /* invalid --socket-mem flag (with extra socket) */ char invalid_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv9[] = {prgname, + const char *argv9[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, invalid_socket_mem}; /* valid --socket-mem flag */ char valid_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv10[] = {prgname, + const char *argv10[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, valid_socket_mem}; if (launch_proc(argv0) != 0) { -- 2.51.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls 2026-01-26 16:45 ` [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls Bruce Richardson @ 2026-02-03 14:17 ` David Marchand 2026-02-03 15:39 ` Bruce Richardson 0 siblings, 1 reply; 10+ messages in thread From: David Marchand @ 2026-02-03 14:17 UTC (permalink / raw) To: Bruce Richardson; +Cc: dev On Mon, 26 Jan 2026 at 17:45, Bruce Richardson <bruce.richardson@intel.com> wrote: > > When we make recursive calls to dpdk-test to check flag support, either > as a secondary process or as another primary process, add the log-level > flag to give extra EAL output to help with debugging any issues. For > calls that are spawning another primary process, pass in --no-pci flag > too, to skip device probing. > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> On one side, the debug will probably help with future errors. But on the second change, the --no-pci stuff is not about debug it seems more like an actual fix (or well, a way to hide the issue in CI). I would prefer we split in two. -- David Marchand ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls 2026-02-03 14:17 ` David Marchand @ 2026-02-03 15:39 ` Bruce Richardson 0 siblings, 0 replies; 10+ messages in thread From: Bruce Richardson @ 2026-02-03 15:39 UTC (permalink / raw) To: David Marchand; +Cc: dev On Tue, Feb 03, 2026 at 03:17:43PM +0100, David Marchand wrote: > On Mon, 26 Jan 2026 at 17:45, Bruce Richardson > <bruce.richardson@intel.com> wrote: > > > > When we make recursive calls to dpdk-test to check flag support, either > > as a secondary process or as another primary process, add the log-level > > flag to give extra EAL output to help with debugging any issues. For > > calls that are spawning another primary process, pass in --no-pci flag > > too, to skip device probing. > > > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> > > On one side, the debug will probably help with future errors. > But on the second change, the --no-pci stuff is not about debug it > seems more like an actual fix (or well, a way to hide the issue in > CI). > > I would prefer we split in two. > Good point, that is correct. I felt that sometimes we may be at the mercy of the actual hardware the CI is running on, so disabling pci probing should make things more reliable. Will see about splitting this patch into two parts to evaluate separately. /Bruce ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] eal: add location info to interrupt error message 2026-01-26 16:45 [PATCH 1/2] eal: add location info to interrupt error message Bruce Richardson 2026-01-26 16:45 ` [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls Bruce Richardson @ 2026-01-26 18:16 ` Stephen Hemminger 2026-02-13 15:22 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests Bruce Richardson 2 siblings, 0 replies; 10+ messages in thread From: Stephen Hemminger @ 2026-01-26 18:16 UTC (permalink / raw) To: Bruce Richardson; +Cc: dev On Mon, 26 Jan 2026 16:45:18 +0000 Bruce Richardson <bruce.richardson@intel.com> wrote: > The error message about the interrupt being unallocated is called from > multiple positions, so if seen in the logs the user cannot tell which > function is actually generating the message. Add function name and line > number info to the message to clarify things. > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> > --- > lib/eal/common/eal_common_interrupts.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/eal/common/eal_common_interrupts.c b/lib/eal/common/eal_common_interrupts.c > index b42fa862f3..a2a310750a 100644 > --- a/lib/eal/common/eal_common_interrupts.c > +++ b/lib/eal/common/eal_common_interrupts.c > @@ -17,7 +17,7 @@ > /* Macros to check for valid interrupt handle */ > #define CHECK_VALID_INTR_HANDLE(intr_handle) do { \ > if (intr_handle == NULL) { \ > - EAL_LOG(DEBUG, "Interrupt instance unallocated"); \ > + EAL_LOG(DEBUG, "%s, ln %d: Interrupt instance unallocated", __func__, __LINE__); \ Using "ln %d" seems different from other places. Maybe EAL_LOG(DEBUG, "%s:%d Interrupt instance unallocated", __FILE__, __LINE__); \ ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 0/3] improve error reporting & resilience in unit tests 2026-01-26 16:45 [PATCH 1/2] eal: add location info to interrupt error message Bruce Richardson 2026-01-26 16:45 ` [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls Bruce Richardson 2026-01-26 18:16 ` [PATCH 1/2] eal: add location info to interrupt error message Stephen Hemminger @ 2026-02-13 15:22 ` Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 1/3] eal: add location info to interrupt error message Bruce Richardson ` (3 more replies) 2 siblings, 4 replies; 10+ messages in thread From: Bruce Richardson @ 2026-02-13 15:22 UTC (permalink / raw) To: dev; +Cc: david.marchand, Bruce Richardson In the EAL flags unit tests sporadic errors were encountered which were proving both difficult to debug and to reproduce. This series aims to improve the situation by firstly adding more debug logs and output so we can better root cause any issues experienced. The final patch then removes pci probing from many test runs where its not needed, so as to avoid issues with unexpected drivers being loaded or probed due to the HW on the system running the tests. V2: split second patch into two, to separate out logging and no-pci changes Bruce Richardson (3): eal: add location info to interrupt error message test/eal_flags: add EAL debug flags to recursive calls test/eal_flags: remove PCI probing for recursive calls app/test/test_eal_flags.c | 229 +++++++++++++------------ lib/eal/common/eal_common_interrupts.c | 2 +- 2 files changed, 123 insertions(+), 108 deletions(-) -- 2.51.0 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/3] eal: add location info to interrupt error message 2026-02-13 15:22 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests Bruce Richardson @ 2026-02-13 15:22 ` Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 2/3] test/eal_flags: add EAL debug flags to recursive calls Bruce Richardson ` (2 subsequent siblings) 3 siblings, 0 replies; 10+ messages in thread From: Bruce Richardson @ 2026-02-13 15:22 UTC (permalink / raw) To: dev; +Cc: david.marchand, Bruce Richardson, Harman Kalra The error message about the interrupt being unallocated is called from multiple positions, so if seen in the logs the user cannot tell which function is actually generating the message. Add function name and line number info to the message to clarify things. Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> --- lib/eal/common/eal_common_interrupts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/eal/common/eal_common_interrupts.c b/lib/eal/common/eal_common_interrupts.c index b42fa862f3..a2a310750a 100644 --- a/lib/eal/common/eal_common_interrupts.c +++ b/lib/eal/common/eal_common_interrupts.c @@ -17,7 +17,7 @@ /* Macros to check for valid interrupt handle */ #define CHECK_VALID_INTR_HANDLE(intr_handle) do { \ if (intr_handle == NULL) { \ - EAL_LOG(DEBUG, "Interrupt instance unallocated"); \ + EAL_LOG(DEBUG, "%s, ln %d: Interrupt instance unallocated", __func__, __LINE__); \ rte_errno = EINVAL; \ goto fail; \ } \ -- 2.51.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 2/3] test/eal_flags: add EAL debug flags to recursive calls 2026-02-13 15:22 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 1/3] eal: add location info to interrupt error message Bruce Richardson @ 2026-02-13 15:22 ` Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 3/3] test/eal_flags: remove PCI probing for " Bruce Richardson 2026-02-16 14:48 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests David Marchand 3 siblings, 0 replies; 10+ messages in thread From: Bruce Richardson @ 2026-02-13 15:22 UTC (permalink / raw) To: dev; +Cc: david.marchand, Bruce Richardson When we make recursive calls to dpdk-test to check flag support, either as a secondary process or as another primary process, add the log-level flag to give extra EAL output to help with debugging any issues. Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> --- app/test/test_eal_flags.c | 228 ++++++++++++++++++++------------------ 1 file changed, 121 insertions(+), 107 deletions(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index cfc6af94ea..83e906a069 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -121,6 +121,8 @@ test_misc_flags(void) #define no_shconf "--no-shconf" #define allow "--allow" #define vdev "--vdev" +#define eal_debug_logs "--log-level=lib.eal:debug" +#define bus_debug_logs "--log-level=bus.*:debug" #define FS_HUGETLB "hugetlbfs" @@ -312,26 +314,26 @@ test_allow_flag(void) if (prefix == NULL) return -1; - const char *wlinval[][7] = { - {prgname, prefix, mp_flag, + const char *wlinval[][8] = { + {prgname, prefix, mp_flag, eal_debug_logs, allow, "error", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:0:0", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:error:0.1", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:0:0.1error", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "error0:0:0.1", "", ""}, - {prgname, prefix, mp_flag, + {prgname, prefix, mp_flag, eal_debug_logs, allow, "0:0:0.1.2", "", ""}, }; /* Test with valid allow option */ - const char *wlval1[] = {prgname, prefix, mp_flag, + const char *wlval1[] = {prgname, prefix, mp_flag, eal_debug_logs, allow, "00FF:09:0B.3"}; - const char *wlval2[] = {prgname, prefix, mp_flag, + const char *wlval2[] = {prgname, prefix, mp_flag, eal_debug_logs, allow, "09:0B.3", allow, "0a:0b.1"}; - const char *wlval3[] = {prgname, prefix, mp_flag, + const char *wlval3[] = {prgname, prefix, mp_flag, eal_debug_logs, allow, "09:0B.3,type=test", allow, "08:00.1,type=normal", }; @@ -372,16 +374,16 @@ test_invalid_b_flag(void) if (prefix == NULL) return -1; - const char *blinval[][5] = { - {prgname, prefix, mp_flag, "-b", "error"}, - {prgname, prefix, mp_flag, "-b", "0:0:0"}, - {prgname, prefix, mp_flag, "-b", "0:error:0.1"}, - {prgname, prefix, mp_flag, "-b", "0:0:0.1error"}, - {prgname, prefix, mp_flag, "-b", "error0:0:0.1"}, - {prgname, prefix, mp_flag, "-b", "0:0:0.1.2"}, + const char *blinval[][6] = { + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "error"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:0:0"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:error:0.1"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:0:0.1error"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "error0:0:0.1"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-b", "0:0:0.1.2"}, }; /* Test with valid blocklist option */ - const char *blval[] = {prgname, prefix, mp_flag, + const char *blval[] = {prgname, prefix, mp_flag, eal_debug_logs, "-b", "FF:09:0B.3"}; int i; @@ -418,18 +420,18 @@ test_invalid_vdev_flag(void) #endif /* !RTE_EXEC_ENV_FREEBSD */ /* Test with invalid vdev option */ - const char *vdevinval[] = {prgname, prefix, no_huge, - vdev, "eth_dummy"}; + const char *vdevinval[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "eth_dummy"}; /* Test with valid vdev option */ - const char *vdevval1[] = {prgname, prefix, no_huge, - vdev, "net_ring0"}; + const char *vdevval1[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "net_ring0"}; - const char *vdevval2[] = {prgname, prefix, no_huge, - vdev, "net_ring0,args=test"}; + const char *vdevval2[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "net_ring0,args=test"}; - const char *vdevval3[] = {prgname, prefix, no_huge, - vdev, "net_ring0,nodeaction=r1:0:CREATE"}; + const char *vdevval3[] = {prgname, prefix, no_huge, eal_debug_logs, + bus_debug_logs, vdev, "net_ring0,nodeaction=r1:0:CREATE"}; if (launch_proc(vdevinval) == 0) { printf("Error (line %d) - process did run ok with invalid vdev parameter\n", @@ -470,14 +472,14 @@ test_invalid_r_flag(void) if (prefix == NULL) return -1; - const char *rinval[][5] = { - {prgname, prefix, mp_flag, "-r", "error"}, - {prgname, prefix, mp_flag, "-r", "0"}, - {prgname, prefix, mp_flag, "-r", "-1"}, - {prgname, prefix, mp_flag, "-r", "17"}, + const char *rinval[][6] = { + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "error"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "0"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "-1"}, + {prgname, prefix, mp_flag, eal_debug_logs, "-r", "17"}, }; /* Test with valid blocklist option */ - const char *rval[] = {prgname, prefix, mp_flag, "-r", "16"}; + const char *rval[] = {prgname, prefix, mp_flag, eal_debug_logs, "-r", "16"}; int i; @@ -508,77 +510,77 @@ test_missing_c_flag(void) return -1; /* -c flag but no coremask value */ - const char *argv1[] = { prgname, prefix, mp_flag, "-c"}; + const char *argv1[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c"}; /* No -c, -l or --lcores flag at all */ const char *argv2[] = { prgname, prefix, mp_flag}; /* bad coremask value */ - const char *argv3[] = { prgname, prefix, mp_flag, + const char *argv3[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "error" }; /* sanity check of tests - valid coremask value */ - const char *argv4[] = { prgname, prefix, mp_flag, + const char *argv4[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "1" }; /* -l flag but no corelist value */ - const char *argv5[] = { prgname, prefix, mp_flag, + const char *argv5[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l"}; - const char *argv6[] = { prgname, prefix, mp_flag, + const char *argv6[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", " " }; /* bad corelist values */ - const char *argv7[] = { prgname, prefix, mp_flag, + const char *argv7[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "error" }; - const char *argv8[] = { prgname, prefix, mp_flag, + const char *argv8[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1-" }; - const char *argv9[] = { prgname, prefix, mp_flag, + const char *argv9[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1," }; - const char *argv10[] = { prgname, prefix, mp_flag, + const char *argv10[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1#2" }; /* core number is negative value */ - const char * const argv11[] = { prgname, prefix, mp_flag, + const char * const argv11[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "-5" }; - const char * const argv12[] = { prgname, prefix, mp_flag, + const char * const argv12[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "-5-7" }; /* core number is maximum value */ - const char * const argv13[] = { prgname, prefix, mp_flag, + const char * const argv13[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", RTE_STR(RTE_MAX_LCORE) }; - const char * const argv14[] = { prgname, prefix, mp_flag, + const char * const argv14[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1-"RTE_STR(RTE_MAX_LCORE) }; /* sanity check test - valid corelist value */ - const char * const argv15[] = { prgname, prefix, mp_flag, + const char * const argv15[] = { prgname, prefix, mp_flag, eal_debug_logs, "-l", "1-2,3" }; /* --lcores flag but no lcores value */ - const char * const argv16[] = { prgname, prefix, mp_flag, + const char * const argv16[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores" }; - const char * const argv17[] = { prgname, prefix, mp_flag, + const char * const argv17[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", " " }; /* bad lcores value */ - const char * const argv18[] = { prgname, prefix, mp_flag, + const char * const argv18[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "1-3-5" }; - const char * const argv19[] = { prgname, prefix, mp_flag, + const char * const argv19[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "0-1,,2" }; - const char * const argv20[] = { prgname, prefix, mp_flag, + const char * const argv20[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "0-,1" }; - const char * const argv21[] = { prgname, prefix, mp_flag, + const char * const argv21[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(0-,2-4)" }; - const char * const argv22[] = { prgname, prefix, mp_flag, + const char * const argv22[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(-1,2)" }; - const char * const argv23[] = { prgname, prefix, mp_flag, + const char * const argv23[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(2-4)@(2-4-6)" }; - const char * const argv24[] = { prgname, prefix, mp_flag, + const char * const argv24[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(a,2)" }; - const char * const argv25[] = { prgname, prefix, mp_flag, + const char * const argv25[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "1-3@(1,3)" }; - const char * const argv26[] = { prgname, prefix, mp_flag, + const char * const argv26[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "3@((1,3)" }; - const char * const argv27[] = { prgname, prefix, mp_flag, + const char * const argv27[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "(4-7)=(1,3)" }; - const char * const argv28[] = { prgname, prefix, mp_flag, + const char * const argv28[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "[4-7]@(1,3)" }; /* sanity check of tests - valid lcores value */ - const char * const argv29[] = { prgname, prefix, mp_flag, + const char * const argv29[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "0-1,2@(5-7),(3-5)@(0,2),(0,6),7"}; /* check an invalid cpu value >= CPU_SETSIZE */ - const char * const argv30[] = { prgname, prefix, mp_flag, + const char * const argv30[] = { prgname, prefix, mp_flag, eal_debug_logs, "--lcores", "3@" RTE_STR(CPU_SETSIZE) }; if (launch_proc(argv2) != 0) { @@ -662,21 +664,21 @@ test_main_lcore_flag(void) return TEST_SKIPPED; /* --main-lcore flag but no value */ - const char *argv1[] = { prgname, prefix, mp_flag, + const char *argv1[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore"}; /* --main-lcore flag with invalid value */ - const char *argv2[] = { prgname, prefix, mp_flag, + const char *argv2[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "-1"}; - const char *argv3[] = { prgname, prefix, mp_flag, + const char *argv3[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "X"}; /* main lcore not in coremask */ - const char *argv4[] = { prgname, prefix, mp_flag, + const char *argv4[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "2"}; /* valid value */ - const char *argv5[] = { prgname, prefix, mp_flag, + const char *argv5[] = { prgname, prefix, mp_flag, eal_debug_logs, "-c", "3", "--main-lcore", "1"}; /* valid value set before coremask */ - const char *argv6[] = { prgname, prefix, mp_flag, + const char *argv6[] = { prgname, prefix, mp_flag, eal_debug_logs, "--main-lcore", "1", "-c", "3"}; if (launch_proc(argv1) == 0 @@ -760,9 +762,9 @@ test_no_hpet_flag(void) } /* With --no-hpet */ - const char *argv1[] = {prgname, prefix, mp_flag, no_hpet}; + const char *argv1[] = {prgname, prefix, mp_flag, eal_debug_logs, no_hpet}; /* Without --no-hpet */ - const char *argv2[] = {prgname, prefix, mp_flag}; + const char *argv2[] = {prgname, prefix, mp_flag, eal_debug_logs}; if (launch_proc(argv1) != 0) { printf("Error (line %d) - process did not run ok with --no-hpet flag\n", __LINE__); @@ -914,105 +916,117 @@ test_misc_flags(void) * No further testing of output done. */ /* sanity check - failure with invalid option */ - const char *argv0[] = {prgname, prefix, mp_flag, "--invalid-opt"}; + const char *argv0[] = {prgname, prefix, mp_flag, eal_debug_logs, "--invalid-opt"}; /* With --no-pci */ - const char *argv1[] = {prgname, prefix, mp_flag, "--no-pci"}; + const char *argv1[] = {prgname, prefix, mp_flag, eal_debug_logs, "--no-pci"}; /* With -v */ - const char *argv2[] = {prgname, prefix, mp_flag, "-v"}; + const char *argv2[] = {prgname, prefix, mp_flag, eal_debug_logs, "-v"}; /* With valid --syslog */ - const char *argv3[] = {prgname, prefix, mp_flag, "--syslog=user"}; + const char *argv3[] = {prgname, prefix, mp_flag, eal_debug_logs, "--syslog=user"}; /* With empty --syslog (now defaults) */ - const char *argv4[] = {prgname, prefix, mp_flag, "--syslog"}; + const char *argv4[] = {prgname, prefix, mp_flag, eal_debug_logs, "--syslog"}; /* With invalid --syslog */ - const char *argv5[] = {prgname, prefix, mp_flag, "--syslog=invalid"}; + const char *argv5[] = {prgname, prefix, mp_flag, eal_debug_logs, "--syslog=invalid"}; /* With no-sh-conf, also use no-huge to ensure this test runs on BSD */ - const char *argv6[] = {prgname, "-m", DEFAULT_MEM_SIZE, + const char *argv6[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, no_shconf, nosh_prefix, no_huge}; /* With --huge-dir */ const char *argv7[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, "--file-prefix=hugedir", "--huge-dir", hugepath}; /* With empty --huge-dir (should fail) */ const char *argv8[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, "--file-prefix=hugedir", "--huge-dir"}; /* With invalid --huge-dir */ const char *argv9[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, "--file-prefix=hugedir", "--huge-dir", "invalid"}; /* With invalid --huge-dir sub-directory */ const char *argv10[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, "--file-prefix=hugedir", "--huge-dir", hugepath_dir}; /* With valid --huge-dir sub-directory */ const char *argv11[] = {prgname, "-m", DEFAULT_MEM_SIZE, + eal_debug_logs, "--file-prefix=hugedir", "--huge-dir", hugepath_dir2}; /* Secondary process with invalid --huge-dir (should run as flag has no * effect on secondary processes) */ - const char *argv12[] = {prgname, prefix, mp_flag, + const char *argv12[] = {prgname, prefix, mp_flag, eal_debug_logs, "--huge-dir", "invalid"}; /* try running with base-virtaddr param */ const char *argv13[] = {prgname, "--file-prefix=virtaddr", + eal_debug_logs, "--base-virtaddr=0x23456789"}; /* try running with --vfio-intr INTx flag */ const char *argv14[] = {prgname, "--file-prefix=intr", + eal_debug_logs, "--vfio-intr=legacy"}; /* try running with --vfio-intr MSI flag */ const char *argv15[] = {prgname, "--file-prefix=intr", + eal_debug_logs, "--vfio-intr=msi"}; /* try running with --vfio-intr MSI-X flag */ const char *argv16[] = {prgname, "--file-prefix=intr", + eal_debug_logs, "--vfio-intr=msix"}; /* try running with --vfio-intr invalid flag */ const char *argv17[] = {prgname, "--file-prefix=intr", + eal_debug_logs, "--vfio-intr=invalid"}; /* With process type as auto-detect */ const char * const argv18[] = {prgname, "--file-prefix=auto", + eal_debug_logs, "--proc-type=auto"}; /* With process type as auto-detect with no-shconf */ - const char * const argv19[] = {prgname, "--proc-type=auto", + const char * const argv19[] = {prgname, "--proc-type=auto", eal_debug_logs, no_shconf, nosh_prefix, no_huge}; /* With process type as --create-uio-dev flag */ const char * const argv20[] = {prgname, "--file-prefix=uiodev", + eal_debug_logs, "--create-uio-dev"}; /* Try running with --huge-worker-stack flag */ - const char * const argv21[] = {prgname, prefix, mp_flag, + const char * const argv21[] = {prgname, prefix, mp_flag, eal_debug_logs, "--huge-worker-stack"}; /* Try running with --huge-worker-stack=512 flag */ - const char * const argv22[] = {prgname, prefix, mp_flag, + const char * const argv22[] = {prgname, prefix, mp_flag, eal_debug_logs, "--huge-worker-stack=512"}; /* Try running with --log-timestamp */ - const char * const argv23[] = {prgname, prefix, mp_flag, + const char * const argv23[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-timestamp" }; /* Try running with --log-timestamp=iso */ - const char * const argv24[] = {prgname, prefix, mp_flag, + const char * const argv24[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-timestamp=iso" }; /* Try running with invalid timestamp */ - const char * const argv25[] = {prgname, prefix, mp_flag, + const char * const argv25[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-timestamp=invalid" }; /* Try running with --log-color */ - const char * const argv26[] = {prgname, prefix, mp_flag, + const char * const argv26[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-color" }; /* Try running with --log-color=never */ - const char * const argv27[] = {prgname, prefix, mp_flag, + const char * const argv27[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-color=never" }; /* Try running with --log-color=invalid */ - const char * const argv28[] = {prgname, prefix, mp_flag, + const char * const argv28[] = {prgname, prefix, mp_flag, eal_debug_logs, "--log-color=invalid" }; /* run all tests also applicable to FreeBSD first */ @@ -1209,47 +1223,47 @@ test_file_prefix(void) } /* this should fail unless the test itself is run with "memtest" prefix */ - const char *argv0[] = {prgname, mp_flag, "-m", + const char *argv0[] = {prgname, mp_flag, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest }; /* primary process with memtest1 and default mem mode */ - const char *argv1[] = {prgname, "-m", + const char *argv1[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest1 }; /* primary process with memtest1 and legacy mem mode */ - const char *argv2[] = {prgname, "-m", + const char *argv2[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest1, "--legacy-mem" }; /* primary process with memtest2 and legacy mem mode */ - const char *argv3[] = {prgname, "-m", + const char *argv3[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest2, "--legacy-mem" }; /* primary process with memtest2 and default mem mode */ - const char *argv4[] = {prgname, "-m", + const char *argv4[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest2 }; /* primary process with --in-memory mode */ - const char * const argv5[] = {prgname, "-m", + const char * const argv5[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--in-memory" }; /* primary process with memtest1 and --in-memory mode */ - const char * const argv6[] = {prgname, "-m", + const char * const argv6[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--in-memory", "--file-prefix=" memtest1 }; /* primary process with parent file-prefix and --in-memory mode */ - const char * const argv7[] = {prgname, "-m", + const char * const argv7[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--in-memory", "--file-prefix", prefix }; /* primary process with memtest1 and --single-file-segments mode */ - const char * const argv8[] = {prgname, "-m", + const char * const argv8[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--single-file-segments", "--file-prefix=" memtest1 }; /* primary process with memtest1 and --huge-unlink=never mode */ - const char * const argv9[] = {prgname, "-m", + const char * const argv9[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, "--huge-unlink=never", "--file-prefix=" memtest1 }; @@ -1500,45 +1514,45 @@ test_memory_flags(void) } /* valid -m flag and mp flag */ - const char *argv0[] = {prgname, prefix, mp_flag, + const char *argv0[] = {prgname, prefix, mp_flag, eal_debug_logs, "-m", DEFAULT_MEM_SIZE}; /* valid -m flag */ - const char *argv1[] = {prgname, + const char *argv1[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE}; /* valid (zero) --socket-mem flag */ char arg2_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv2[] = {prgname, + const char *argv2[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, arg2_socket_mem}; /* invalid (incomplete) --socket-mem flag */ char arg3_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv3[] = {prgname, + const char *argv3[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, arg3_socket_mem}; /* invalid (mixed with invalid data) --socket-mem flag */ char arg4_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv4[] = {prgname, + const char *argv4[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, arg4_socket_mem}; /* invalid (with numeric value as last character) --socket-mem flag */ char arg5_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv5[] = {prgname, + const char *argv5[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, arg5_socket_mem}; /* invalid (with empty socket) --socket-mem flag */ char arg6_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv6[] = {prgname, + const char *argv6[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, arg6_socket_mem}; /* invalid (null) --socket-mem flag */ - const char *argv7[] = {prgname, + const char *argv7[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, "--socket-mem="}; /* valid --socket-mem specified together with -m flag */ char arg8_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv8[] = {prgname, + const char *argv8[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE, arg8_socket_mem}; @@ -1556,12 +1570,12 @@ test_memory_flags(void) /* invalid --socket-mem flag (with extra socket) */ char invalid_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv9[] = {prgname, + const char *argv9[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, invalid_socket_mem}; /* valid --socket-mem flag */ char valid_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv10[] = {prgname, + const char *argv10[] = {prgname, eal_debug_logs, "--file-prefix=" memtest, valid_socket_mem}; if (launch_proc(argv0) != 0) { -- 2.51.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 3/3] test/eal_flags: remove PCI probing for recursive calls 2026-02-13 15:22 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 1/3] eal: add location info to interrupt error message Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 2/3] test/eal_flags: add EAL debug flags to recursive calls Bruce Richardson @ 2026-02-13 15:22 ` Bruce Richardson 2026-02-16 14:48 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests David Marchand 3 siblings, 0 replies; 10+ messages in thread From: Bruce Richardson @ 2026-02-13 15:22 UTC (permalink / raw) To: dev; +Cc: david.marchand, Bruce Richardson When creating another primary process as part of EAL flag testing, avoid unexpected side-effects due to the HW being used by disabling PCI probing at process startup. Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> --- app/test/test_eal_flags.c | 67 ++++++++++++++++++++------------------- 1 file changed, 34 insertions(+), 33 deletions(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 83e906a069..bebddf1c98 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -121,6 +121,7 @@ test_misc_flags(void) #define no_shconf "--no-shconf" #define allow "--allow" #define vdev "--vdev" +#define no_pci "--no-pci" #define eal_debug_logs "--log-level=lib.eal:debug" #define bus_debug_logs "--log-level=bus.*:debug" @@ -930,28 +931,28 @@ test_misc_flags(void) const char *argv5[] = {prgname, prefix, mp_flag, eal_debug_logs, "--syslog=invalid"}; /* With no-sh-conf, also use no-huge to ensure this test runs on BSD */ - const char *argv6[] = {prgname, eal_debug_logs, "-m", DEFAULT_MEM_SIZE, + const char *argv6[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, no_shconf, nosh_prefix, no_huge}; /* With --huge-dir */ const char *argv7[] = {prgname, "-m", DEFAULT_MEM_SIZE, - eal_debug_logs, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", hugepath}; /* With empty --huge-dir (should fail) */ const char *argv8[] = {prgname, "-m", DEFAULT_MEM_SIZE, - eal_debug_logs, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir"}; /* With invalid --huge-dir */ const char *argv9[] = {prgname, "-m", DEFAULT_MEM_SIZE, - eal_debug_logs, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", "invalid"}; /* With invalid --huge-dir sub-directory */ const char *argv10[] = {prgname, "-m", DEFAULT_MEM_SIZE, - eal_debug_logs, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", hugepath_dir}; /* With valid --huge-dir sub-directory */ const char *argv11[] = {prgname, "-m", DEFAULT_MEM_SIZE, - eal_debug_logs, + eal_debug_logs, no_pci, "--file-prefix=hugedir", "--huge-dir", hugepath_dir2}; /* Secondary process with invalid --huge-dir (should run as flag has no * effect on secondary processes) */ @@ -960,41 +961,41 @@ test_misc_flags(void) /* try running with base-virtaddr param */ const char *argv13[] = {prgname, "--file-prefix=virtaddr", - eal_debug_logs, + eal_debug_logs, no_pci, "--base-virtaddr=0x23456789"}; /* try running with --vfio-intr INTx flag */ const char *argv14[] = {prgname, "--file-prefix=intr", - eal_debug_logs, + eal_debug_logs, no_pci, "--vfio-intr=legacy"}; /* try running with --vfio-intr MSI flag */ const char *argv15[] = {prgname, "--file-prefix=intr", - eal_debug_logs, + eal_debug_logs, no_pci, "--vfio-intr=msi"}; /* try running with --vfio-intr MSI-X flag */ const char *argv16[] = {prgname, "--file-prefix=intr", - eal_debug_logs, + eal_debug_logs, no_pci, "--vfio-intr=msix"}; /* try running with --vfio-intr invalid flag */ const char *argv17[] = {prgname, "--file-prefix=intr", - eal_debug_logs, + eal_debug_logs, no_pci, "--vfio-intr=invalid"}; /* With process type as auto-detect */ const char * const argv18[] = {prgname, "--file-prefix=auto", - eal_debug_logs, + eal_debug_logs, no_pci, "--proc-type=auto"}; /* With process type as auto-detect with no-shconf */ - const char * const argv19[] = {prgname, "--proc-type=auto", eal_debug_logs, + const char * const argv19[] = {prgname, "--proc-type=auto", eal_debug_logs, no_pci, no_shconf, nosh_prefix, no_huge}; /* With process type as --create-uio-dev flag */ const char * const argv20[] = {prgname, "--file-prefix=uiodev", - eal_debug_logs, + eal_debug_logs, no_pci, "--create-uio-dev"}; /* Try running with --huge-worker-stack flag */ @@ -1227,43 +1228,43 @@ test_file_prefix(void) DEFAULT_MEM_SIZE, "--file-prefix=" memtest }; /* primary process with memtest1 and default mem mode */ - const char *argv1[] = {prgname, eal_debug_logs, "-m", + const char *argv1[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest1 }; /* primary process with memtest1 and legacy mem mode */ - const char *argv2[] = {prgname, eal_debug_logs, "-m", + const char *argv2[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest1, "--legacy-mem" }; /* primary process with memtest2 and legacy mem mode */ - const char *argv3[] = {prgname, eal_debug_logs, "-m", + const char *argv3[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest2, "--legacy-mem" }; /* primary process with memtest2 and default mem mode */ - const char *argv4[] = {prgname, eal_debug_logs, "-m", + const char *argv4[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--file-prefix=" memtest2 }; /* primary process with --in-memory mode */ - const char * const argv5[] = {prgname, eal_debug_logs, "-m", + const char * const argv5[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--in-memory" }; /* primary process with memtest1 and --in-memory mode */ - const char * const argv6[] = {prgname, eal_debug_logs, "-m", + const char * const argv6[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--in-memory", "--file-prefix=" memtest1 }; /* primary process with parent file-prefix and --in-memory mode */ - const char * const argv7[] = {prgname, eal_debug_logs, "-m", + const char * const argv7[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--in-memory", "--file-prefix", prefix }; /* primary process with memtest1 and --single-file-segments mode */ - const char * const argv8[] = {prgname, eal_debug_logs, "-m", + const char * const argv8[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--single-file-segments", "--file-prefix=" memtest1 }; /* primary process with memtest1 and --huge-unlink=never mode */ - const char * const argv9[] = {prgname, eal_debug_logs, "-m", + const char * const argv9[] = {prgname, eal_debug_logs, no_pci, "-m", DEFAULT_MEM_SIZE, "--huge-unlink=never", "--file-prefix=" memtest1 }; @@ -1518,41 +1519,41 @@ test_memory_flags(void) "-m", DEFAULT_MEM_SIZE}; /* valid -m flag */ - const char *argv1[] = {prgname, eal_debug_logs, + const char *argv1[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE}; /* valid (zero) --socket-mem flag */ char arg2_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv2[] = {prgname, eal_debug_logs, + const char *argv2[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg2_socket_mem}; /* invalid (incomplete) --socket-mem flag */ char arg3_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv3[] = {prgname, eal_debug_logs, + const char *argv3[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg3_socket_mem}; /* invalid (mixed with invalid data) --socket-mem flag */ char arg4_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv4[] = {prgname, eal_debug_logs, + const char *argv4[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg4_socket_mem}; /* invalid (with numeric value as last character) --socket-mem flag */ char arg5_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv5[] = {prgname, eal_debug_logs, + const char *argv5[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg5_socket_mem}; /* invalid (with empty socket) --socket-mem flag */ char arg6_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv6[] = {prgname, eal_debug_logs, + const char *argv6[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, arg6_socket_mem}; /* invalid (null) --socket-mem flag */ - const char *argv7[] = {prgname, eal_debug_logs, + const char *argv7[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, "--socket-mem="}; /* valid --socket-mem specified together with -m flag */ char arg8_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv8[] = {prgname, eal_debug_logs, + const char *argv8[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE, arg8_socket_mem}; @@ -1570,12 +1571,12 @@ test_memory_flags(void) /* invalid --socket-mem flag (with extra socket) */ char invalid_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv9[] = {prgname, eal_debug_logs, + const char *argv9[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, invalid_socket_mem}; /* valid --socket-mem flag */ char valid_socket_mem[SOCKET_MEM_STRLEN]; - const char *argv10[] = {prgname, eal_debug_logs, + const char *argv10[] = {prgname, eal_debug_logs, no_pci, "--file-prefix=" memtest, valid_socket_mem}; if (launch_proc(argv0) != 0) { -- 2.51.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2 0/3] improve error reporting & resilience in unit tests 2026-02-13 15:22 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests Bruce Richardson ` (2 preceding siblings ...) 2026-02-13 15:22 ` [PATCH v2 3/3] test/eal_flags: remove PCI probing for " Bruce Richardson @ 2026-02-16 14:48 ` David Marchand 3 siblings, 0 replies; 10+ messages in thread From: David Marchand @ 2026-02-16 14:48 UTC (permalink / raw) To: Bruce Richardson; +Cc: dev On Fri, 13 Feb 2026 at 16:22, Bruce Richardson <bruce.richardson@intel.com> wrote: > > In the EAL flags unit tests sporadic errors were encountered which were proving > both difficult to debug and to reproduce. This series aims to improve the > situation by firstly adding more debug logs and output so we can better > root cause any issues experienced. The final patch then removes pci probing from > many test runs where its not needed, so as to avoid issues with unexpected > drivers being loaded or probed due to the HW on the system running the tests. > > V2: split second patch into two, to separate out logging and no-pci changes > > Bruce Richardson (3): > eal: add location info to interrupt error message > test/eal_flags: add EAL debug flags to recursive calls > test/eal_flags: remove PCI probing for recursive calls > > app/test/test_eal_flags.c | 229 +++++++++++++------------ > lib/eal/common/eal_common_interrupts.c | 2 +- > 2 files changed, 123 insertions(+), 108 deletions(-) Series applied, thanks. -- David Marchand ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2026-02-16 14:49 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-01-26 16:45 [PATCH 1/2] eal: add location info to interrupt error message Bruce Richardson 2026-01-26 16:45 ` [PATCH 2/2] test/eal_flags: add EAL debug logs to recursive calls Bruce Richardson 2026-02-03 14:17 ` David Marchand 2026-02-03 15:39 ` Bruce Richardson 2026-01-26 18:16 ` [PATCH 1/2] eal: add location info to interrupt error message Stephen Hemminger 2026-02-13 15:22 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 1/3] eal: add location info to interrupt error message Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 2/3] test/eal_flags: add EAL debug flags to recursive calls Bruce Richardson 2026-02-13 15:22 ` [PATCH v2 3/3] test/eal_flags: remove PCI probing for " Bruce Richardson 2026-02-16 14:48 ` [PATCH v2 0/3] improve error reporting & resilience in unit tests David Marchand
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox