diff mbox series

Make crash02 verbose in standard runfile

Message ID 20191213133709.11845-1-mdoucha@suse.cz
State Superseded
Delegated to: Petr Vorel
Headers show
Series Make crash02 verbose in standard runfile | expand

Commit Message

Martin Doucha Dec. 13, 2019, 1:37 p.m. UTC
Verbose output from crash02 will be useful for actually reporting bugs
triggered by the test. With default verbosity level, it's difficult to figure
out which syscall arguments crashed the system even if you know the random seed
and can easily re-run `crash02 -xv 2` on a system that uses the same rand()
implementation.

Signed-off-by: Martin Doucha <mdoucha@suse.cz>
---
 runtest/crashme                |  2 +-
 testcases/misc/crash/crash02.c | 13 ++++++-------
 2 files changed, 7 insertions(+), 8 deletions(-)

Comments

Petr Vorel Dec. 13, 2019, 4:48 p.m. UTC | #1
Hi Martin,

> Verbose output from crash02 will be useful for actually reporting bugs
> triggered by the test. With default verbosity level, it's difficult to figure
> out which syscall arguments crashed the system even if you know the random seed
> and can easily re-run `crash02 -xv 2` on a system that uses the same rand()
> implementation.

Reviewed-by: Petr Vorel <pvorel@suse.cz>

BTW help for -x is (and was before this patch) misleading because "hexdump
random code instead" is now for -v2 even without -x (and before it required
-v2). But that's a detail, so I'd be for merging it.

Kind regards,
Petr
diff mbox series

Patch

diff --git a/runtest/crashme b/runtest/crashme
index a4c5b41cc..14750749d 100644
--- a/runtest/crashme
+++ b/runtest/crashme
@@ -7,7 +7,7 @@  f00f f00f
 crash01 crash01
 # Generate random code and execute it. Read f00f comment,
 # this test lockup SunOS,WindowsNT,etc. in seconds..
-crash02 crash02
+crash02 crash02 -v 2
 # Generate random syscalls and execute them, less probability
 # to hose your system, but still.
 mem01 mem01 -r
diff --git a/testcases/misc/crash/crash02.c b/testcases/misc/crash/crash02.c
index 00fee3493..29d05f82e 100644
--- a/testcases/misc/crash/crash02.c
+++ b/testcases/misc/crash/crash02.c
@@ -430,13 +430,12 @@  void try_one_crash(int try_num)
 	arg6 = rand_long();
 	arg7 = rand_long();
 
-	if (x_opt) {
-		if (verbose_level >= 1)
-			printf("%04d: syscall(%ld, %#lx, %#lx, %#lx, %#lx, "
-			       "%#lx, %#lx, %#lx)\n",
-			       try_num, sysno, arg1, arg2, arg3, arg4, arg5,
-			       arg6, arg7);
-	} else {
+	if (verbose_level >= 1)
+		printf("%04d: syscall(%ld, %#lx, %#lx, %#lx, %#lx, %#lx, "
+		       "%#lx, %#lx)\n", try_num, sysno, arg1, arg2, arg3,
+		       arg4, arg5, arg6, arg7);
+
+	if (!x_opt) {
 		syscall(sysno, arg1, arg2, arg3, arg4, arg5, arg6, arg7);
 		record_errno(errno);
 	}