diff options
Diffstat (limited to 'winsup/cygwin')
| -rw-r--r-- | winsup/cygwin/ChangeLog | 7 | ||||
| -rw-r--r-- | winsup/cygwin/exceptions.cc | 8 |
2 files changed, 13 insertions, 2 deletions
diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index a32a28a..ffa1960 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,5 +1,12 @@ 2013-06-08 Christopher Faylor <me.cygwin2013@cgf.cx> + * exceptions.cc (try_to_debug): Don't use yield() when waiting for + another process. + (sigpacket::setup_handler): Fix long-standing problem where loop could + exit with lock held. + +2013-06-08 Christopher Faylor <me.cygwin2013@cgf.cx> + * miscfuncs.cc (yield): Revert (after researching) to calling SleepEx with 0. We don't want to actually sleep when calling this function. diff --git a/winsup/cygwin/exceptions.cc b/winsup/cygwin/exceptions.cc index ac13478..46af2ba 100644 --- a/winsup/cygwin/exceptions.cc +++ b/winsup/cygwin/exceptions.cc @@ -508,7 +508,7 @@ try_to_debug (bool waitloop) return dbg; SetThreadPriority (GetCurrentThread (), THREAD_PRIORITY_IDLE); while (!being_debugged ()) - yield (); + Sleep (1); Sleep (2000); } @@ -923,7 +923,10 @@ sigpacket::setup_handler (void *handler, struct sigaction& siga, _cygtls *tls) DWORD res; HANDLE hth = (HANDLE) *tls; if (!hth) - sigproc_printf ("thread handle NULL, not set up yet?"); + { + tls->unlock (); + sigproc_printf ("thread handle NULL, not set up yet?"); + } else { /* Suspend the thread which will receive the signal. @@ -936,6 +939,7 @@ sigpacket::setup_handler (void *handler, struct sigaction& siga, _cygtls *tls) /* Just set pending if thread is already suspended */ if (res) { + tls->unlock (); ResumeThread (hth); goto out; } |