This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Tests that set aggressive switch interval hang in Cygwin on a VM
Type: Stage: resolved
Components: Tests Versions:
process
Status: closed Resolution: wont fix
Dependencies: Superseder: Cygwin is unsupported - close all open issues and list them here.
View: 45537
Assigned To: Nosy List: erik.bray
Priority: normal Keywords: patch

Created on 2017-12-07 14:38 by erik.bray, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 4964 closed erik.bray, 2017-12-21 17:09
Messages (1)
msg307808 - (view) Author: Erik Bray (erik.bray) * (Python triager) Date: 2017-12-07 14:38
This is basically the same as #26939, but rather than Android running in an emulator it is affecting Cygwin running in a slow VM (in this case it's on my university's OpenStack infrastructure--I don't know what hypervisor they're using but probably KVM--either way the point is that it's a Windows VM and relatively slow).

With Cygwin on Windows running natively this has never been a problem.

FWIW I tried my own version of Victor's patch from #23428 (adapted to the new _PyTime API).  This patch would be worth applying regardless and I can attach my version of the patch to that ticket, but alone it did not fix it.

On top of that I also added a version of Xavier's patch from #26939 that adjusts the wait interval in PyCOND_TIMEDWAIT to ensure that the deadline is late enough to account for delays.  I'm not sure that this is completely fool-proof, but it solved the problem for me.  With that patch I was unable to make the system hang again.

For some reason in #26939 that more general fix was abandoned in favor of simply setting the switch interval less aggressively for those tests in the particular case of the android emulator.  But it seems that the more general fix might still be worthwhile.
History
Date User Action Args
2022-04-11 14:58:55adminsetgithub: 76424
2021-10-21 09:57:27iritkatrielsetresolution: duplicate -> wont fix
2021-10-20 12:10:21iritkatrielsetstatus: open -> closed
resolution: duplicate
superseder: Cygwin is unsupported - close all open issues and list them here.
stage: patch review -> resolved
2017-12-21 17:09:50erik.braysetkeywords: + patch
stage: patch review
pull_requests: + pull_request4855
2017-12-07 14:38:53erik.braycreate