public inbox for dev@dpdk.org
 help / color / mirror / Atom feed
From: Bruce Richardson <bruce.richardson@intel.com>
To: dev@dpdk.org
Cc: david.marchand@redhat.com, Bruce Richardson <bruce.richardson@intel.com>
Subject: [PATCH v2 2/3] test/eal_flags: add EAL debug flags to recursive calls
Date: Fri, 13 Feb 2026 15:22:25 +0000	[thread overview]
Message-ID: <20260213152227.879170-3-bruce.richardson@intel.com> (raw)
In-Reply-To: <20260213152227.879170-1-bruce.richardson@intel.com>

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


  parent reply	other threads:[~2026-02-13 15:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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   ` Bruce Richardson [this message]
2026-02-13 15:22   ` [PATCH v2 3/3] test/eal_flags: remove PCI probing for recursive calls Bruce Richardson
2026-02-16 14:48   ` [PATCH v2 0/3] improve error reporting & resilience in unit tests David Marchand

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260213152227.879170-3-bruce.richardson@intel.com \
    --to=bruce.richardson@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox