[Bug 51696] New: ntoskrnl.exe:ntoskrnl outputs too much traces (and broke the Wine tests entirely)

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=51696

            Bug ID: 51696
           Summary: ntoskrnl.exe:ntoskrnl outputs too much traces (and
                    broke the Wine tests entirely)
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: ntoskrnl
          Assignee: wine-bugs at winehq.org
          Reporter: fgouget at codeweavers.com
      Distribution: ---

ntoskrnl.exe:ntoskrnl outputs more than 200kB of traces:

$ ./wine dlls/ntoskrnl.exe/tests/ntoskrnl.exe_test.exe ntoskrnl
>ntoskrnl.report

$ tail -n3 ntoskrnl.report
0270:ntoskrnl: 96 tests executed (1 marked as todo, 0 failures), 0 skipped.
0020:ntoskrnl: Silenced 5475 todos, 0 skips and 0 traces.
0020:ntoskrnl: 21296 tests executed (1967 marked as todo, 0 failures), 0
skipped.

$ ls -l ntoskrnl.report
-rw-r--r-- 1 fgouget fgouget 208428 août  28 18:37 ntoskrnl.report

$ grep "Test marked todo:" ntoskrnl.report | wc -cl
1994  204531

This is not a problem of a runaway test or even that many trace() statements.
It's just that there are too many todos despite the vast majority being
silenced already.

The big issue is that adding so much data to a full Winetest run pushes it over
the 1.5MB limit (before and after debiant2 WineTest reports):

-rw-r--r-- 1 fgouget fgouget 1477298 août  28 20:27
/tmp/job96432_1_1_win32.report
-rw-r--r-- 1 fgouget fgouget 1689913 août  28 20:24
/tmp/job96543_1_1_win32.report

The commit that introduced the extra todos is:

commit 8d37962b43e5074bc0421611267dc41593114f85
Author:     Zebediah Figura <zfigura at codeweavers.com>
AuthorDate: Thu Aug 26 20:54:10 2021 -0500

    ntoskrnl/tests: Add more tests for IRP status handling.

    Signed-off-by: Zebediah Figura <zfigura at codeweavers.com>
    Signed-off-by: Alexandre Julliard <julliard at winehq.org>


Also the TestBot warned about this issue when the patch was submitted, though
that did not carry over to sending an email to wine-devel or marking the patch
as failed on the patches website (bug 51695).

https://testbot.winehq.org/JobDetails.pl?Key=96439#k401

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.