[Bug 54342] New: ws2_32:sock - test_WSARecv() sometimes fails with "got apc_count 1." on Windows 7
https://bugs.winehq.org/show_bug.cgi?id=54342 Bug ID: 54342 Summary: ws2_32:sock - test_WSARecv() sometimes fails with "got apc_count 1." on Windows 7 Product: Wine Version: 8.0-rc4 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: winsock Assignee: wine-bugs(a)winehq.org Reporter: jinoh.kang.kr(a)gmail.com CC: fgouget(a)codeweavers.com Distribution: --- test_WSARecv() sometimes fails with "got apc_count 1." on w7u_adm. -- 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=54342 Jinoh Kang <jinoh.kang.kr(a)gmail.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=54342 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|0 |1 Status|UNCONFIRMED |NEW Summary|ws2_32:sock - |ws2_32:sock - |test_WSARecv() sometimes |test_WSARecv() sometimes |fails with "got apc_count |fails with "got apc_count |1." on Windows 7 |1." on Windows --- Comment #1 from François Gouget <fgouget(a)codeweavers.com> --- This failure is pretty rare but it is not specific to Windows 7. There are 4 known instances: * 2022-10-25 w7u-es 32-bit * 2022-11-04 Windows 7 * 2022-12-01 w1064-2qxl 64-bit (22h2) * 2022-12-12 on my w10pro64-rx550 VM, 64-bit (22h2) See https://test.winehq.org/data/patterns.html#ws2_32:sock I don't see the w7u_adm case in the test.winehq.org results so I suppose it was a merge request false positive? -- 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=54342 Zeb Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |z.figura12(a)gmail.com --- Comment #2 from Zeb Figura <z.figura12(a)gmail.com> --- I wonder if Windows is doing an alertable wait inside of WSARecv()? Or if the file descriptor has FILE_SYNCHRONOUS_IO_ALERT set? Very weird if so. -- 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=54342 --- Comment #3 from Zeb Figura <z.figura12(a)gmail.com> --- (In reply to Zeb Figura from comment #2)
I wonder if Windows is doing an alertable wait inside of WSARecv()?
It does, but not if WSARecv() completes synchronously. Which it usually does, but isn't guaranteed to do. I'll send a patch. -- 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=54342 --- Comment #4 from Jinoh Kang <jinoh.kang.kr(a)gmail.com> --- (In reply to Zeb Figura from comment #3)
(In reply to Zeb Figura from comment #2)
I wonder if Windows is doing an alertable wait inside of WSARecv()?
It does, but not if WSARecv() completes synchronously. Which it usually does, but isn't guaranteed to do. I'll send a patch.
Would you link to the patch? -- 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=54342 Zeb Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED Fixed by SHA1| |9d330d81a6af73220e08a93efa1 | |8a7cbf0cc9408 --- Comment #5 from Zeb Figura <z.figura12(a)gmail.com> --- This should have been fixed, a long time ago, by 9d330d81a6af73220e08a93efa18a7cbf0cc9408. I probably never marked it fixed because I wanted to wait and see if it actually was. At any rate it doesn't appear on the patterns page 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=54342 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #6 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 10.0-rc4. -- 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