[Bug 51153] New: A winegstreamer change caused a new quartz:filtergraph failure in Wine
https://bugs.winehq.org/show_bug.cgi?id=51153 Bug ID: 51153 Summary: A winegstreamer change caused a new quartz:filtergraph failure in Wine Product: Wine Version: 6.8 Hardware: x86-64 OS: Linux Status: NEW Severity: normal Priority: P2 Component: winegstreamer Assignee: wine-bugs(a)winehq.org Reporter: fgouget(a)codeweavers.com Distribution: --- quartz:filtergraph started failing in Wine on 2021-04-26: http://winetest.dolphin/data/patterns.html#quartz:filtergraph filtergraph.c:440: Test failed: GetState() failed: 40237 Strangely this failure only happens on the TestBot's debiant2 VM. A bisect showed this failure to be caused by the patch below: commit d7fecebe93938bf1ef2349ac74413e28d6b8e153 Author: Anton Baskanov <baskanov(a)gmail.com> Date: Mon Apr 26 09:51:24 2021 +0700 winegstreamer: Allow setting the stop position to the stream duration. Since we no longer stop the GStreamer pipeline, it does not reset the stop position for us. Fixes a hang during intro video playback in RC Cars. Signed-off-by: Anton Baskanov <baskanov(a)gmail.com> Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> See also: https://www.winehq.org/pipermail/wine-devel/2021-April/185904.html -- 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=51153 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |d7fecebe93938bf1ef2349ac744 | |13e28d6b8e153 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=51153 Zebediah Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression -- 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=51153 Zebediah Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |z.figura12(a)gmail.com --- Comment #1 from Zebediah Figura <z.figura12(a)gmail.com> --- This needs some restructuring of the way locking is done in strmbase. -- 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=51153 Zebediah Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|winegstreamer |quartz -- 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=51153 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |baskanov(a)gmail.com -- 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=51153 mrdeathjr28(a)yahoo.es changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mrdeathjr28(a)yahoo.es -- 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=51153 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WORKSFORME Status|NEW |RESOLVED --- Comment #2 from François Gouget <fgouget(a)codeweavers.com> --- There is no record of this happening going back to at least 2022-06-02. So this can be considered to be fixed somehow. -- 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=51153 --- Comment #3 from Zeb Figura <z.figura12(a)gmail.com> --- The issue is still there, but it was marked flaky. Although I guess it was marked flaky after June, so all I can really say is that fundamentally the problem still exists, and just happens not to occur on the testbots anymore. -- 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=51153 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|WORKSFORME |--- Status|RESOLVED |REOPENED --- Comment #4 from François Gouget <fgouget(a)codeweavers.com> --- Oh, I see. Also the message changed: hr = IMediaControl_GetState(control, 1000, &state); flaky_wine ok(hr == S_OK, "Got hr %#lx.\n", hr); And I find matching lines on debian11, debiant and fg-deb64, starting on 2022-06-03 up to now: filtergraph.c:434: Test marked flaky: Got hr 0x40237. So I'm reopening this bug. -- 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