systemc: Merge stderr and stdout when running tests.
authorGabe Black <gabeblack@google.com>
Thu, 6 Sep 2018 23:49:24 +0000 (16:49 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 9 Oct 2018 21:38:49 +0000 (21:38 +0000)
The golden output for tests doesn't distinguish between stderr and
stdout, and by only comparing against stdout we have to throw away
errors which would be good to verify we get right. Also the tests
sometimes send output to stderr for no apparent reason, requiring
manually patching the tests.

This change adds filters for two messages which used to go to stderr
in gem5 but now show up in the diffs, one that just says the simulation
is starting, and the other for warns of unimplemented functionality.
The second warning should be turned on at some point so we make sure
everything the tests touch works and they don't just work by
coincidence, but for now it introduces a lot of noise among otherwise
passing tests.

Change-Id: I3b14f7807af561a79d6e0ca87aff1ab6051be596
Reviewed-on: https://gem5-review.googlesource.com/c/12600
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/systemc/tests/verify.py

index 19cca05609fea04b4f850d74df5c6945911eff96..128332223fea1e17536712a4a9e0e7d7de030ada 100755 (executable)
@@ -152,7 +152,7 @@ class RunPhase(TestPhaseBase):
                 cmd.extend(timeout_cmd)
             cmd.extend([
                 test.full_path(),
-                '-red', os.path.abspath(test.m5out_dir()),
+                '-rd', os.path.abspath(test.m5out_dir()),
                 '--listener-mode=off',
                 '--quiet',
                 config_path,
@@ -225,17 +225,13 @@ class LogChecker(Checker):
         warning_filt(540),
         warning_filt(569),
         warning_filt(571),
-        error_filt(514),
-        error_filt(515),
-        error_filt(525),
-        error_filt(541),
-        error_filt(542),
-        error_filt(543),
         info_filt(804),
         in_file_filt,
     )
     test_filt = merge_filts(
         r'^Global frequency set at \d* ticks per second\n',
+        r'^info: Entering event queue @ \d*\.  Starting simulation\.\.\.\n',
+        r'warn: [^(]+\([^)]*\)( \[with [^]]*\])? not implemented\.\n',
         info_filt(804),
         in_file_filt,
     )