Re: [PATCH v2] Cygwin: signal: Avoid frequent TLS lock/unlock for SIGCONT processing

2025-01-22 Thread Corinna Vinschen
On Jan 22 07:58, Takashi Yano wrote: > On Tue, 21 Jan 2025 13:20:46 -0800 (PST) > Jeremy Drake wrote: > > dscho hooked me up with a workflow to run Git for Windows' test suite on > > an msys2-runtime pull request's CI artifacts. With this patch and the > > three you pushed (reverting the v2 patch w

Re: [PATCH v2] Cygwin: signal: Avoid frequent TLS lock/unlock for SIGCONT processing

2025-01-21 Thread Takashi Yano
On Tue, 21 Jan 2025 13:20:46 -0800 (PST) Jeremy Drake wrote: > dscho hooked me up with a workflow to run Git for Windows' test suite on > an msys2-runtime pull request's CI artifacts. With this patch and the > three you pushed (reverting the v2 patch we had applied already, and > applying the two f

Re: [PATCH v2] Cygwin: signal: Avoid frequent TLS lock/unlock for SIGCONT processing

2025-01-21 Thread Jeremy Drake via Cygwin-patches
dscho hooked me up with a workflow to run Git for Windows' test suite on an msys2-runtime pull request's CI artifacts. With this patch and the three you pushed (reverting the v2 patch we had applied already, and applying the two from the cygwin-3_5-branch), the test suite no longer hangs. On Mon,

[PATCH v2] Cygwin: signal: Avoid frequent TLS lock/unlock for SIGCONT processing

2025-01-20 Thread Takashi Yano
It seems that current _cygtls::handle_SIGCONT() code sometimes falls into a deadlock due to frequent TLS lock/unlock operation in the yield() loop. With this patch, the yield() in the wait loop is placed outside the TLS lock to avoid frequent TLS lock/unlock. Fixes: 9ae51bcc51a7 ("Cygwin: signal: