[Bug 51196] New: ntdll:om has a timing-related failure in Wine
https://bugs.winehq.org/show_bug.cgi?id=51196 Bug ID: 51196 Summary: ntdll:om has a timing-related failure in Wine Product: Wine Version: 6.8 Hardware: x86-64 OS: Linux Status: NEW Severity: normal Priority: P2 Component: ntdll Assignee: wine-bugs(a)winehq.org Reporter: fgouget(a)codeweavers.com Distribution: --- ntdll:om randomly fails with an error of the form: om.c:2307: Test failed: got 88 and/or om.c:2322: Test failed: got 87 The value is the time spent in RtlWaitOnAddress(..., timeout=100 ms). The test generously expects at least 90 ms and the failure occurs whenever it is below that. Because the failure is intermittent and the duration typically varies between 85 and 89 ms, this failure is prone to generating false positives in the TestBot. Furthermore this only happens in Wine which suggests there is some implementation issue there. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=51196 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |source, testcase -- 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.
https://bugs.winehq.org/show_bug.cgi?id=51196 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |e72c71d27d67462480026edc2b7 | |56a66578db7bd --- Comment #1 from François Gouget <fgouget(a)codeweavers.com> --- This is fixed: commit e72c71d27d67462480026edc2b756a66578db7bd Author: Francois Gouget <fgouget(a)codeweavers.com> AuthorDate: Mon Jul 5 18:10:39 2021 +0200 ntdll/tests: Improve the RtlWaitOnAddress() timeout checks. GetTickCount()'s granularity represents 15% of the time we want to mesure, making it hard to not get failures. So use NtQuerySystemTime() instead since we already depend on it. Narrow down the acceptable range again. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> -- 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.
https://bugs.winehq.org/show_bug.cgi?id=51196 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #2 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 6.14. -- 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.
participants (1)
-
WineHQ Bugzilla