From d6719f7e17a613ad90039dc4adabbc82f8532d52 Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Tue, 29 May 2012 13:11:34 +0000 Subject: [PATCH] * select.cc (select_stuff::wait): Temporarily disable restarting entirely. --- winsup/cygwin/ChangeLog | 5 +++++ winsup/cygwin/select.cc | 12 +++++++++--- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index c265b202f..79588781d 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,8 @@ +2012-05-29 Corinna Vinschen + + * select.cc (select_stuff::wait): Temporarily disable restarting + entirely. + 2012-05-29 Corinna Vinschen * security.h (cygsidlist::+=): Correctly copy well_known_sid info from diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc index c8d7ddb05..19778ceb5 100644 --- a/winsup/cygwin/select.cc +++ b/winsup/cygwin/select.cc @@ -331,13 +331,17 @@ select_stuff::wait (fd_set *readfds, fd_set *writefds, fd_set *exceptfds, { case WAIT_OBJECT_0: select_printf ("signal received"); +#if 0 /* FIXME? Partial revert of change from 2012-01-22. If the signal handler is called before the threads are stopped via cleanup, - emacs 24.x crashes in thread_pipe. For now, do not call the - signal handler if we're not in the main thread, but cleanup - before calling the signal handler. */ + emacs 24.x crashes in thread_pipe. Just restarting without + calling the signal handler makes select entirely uninterruptible + when called from a thread not the main thread, see + http://cygwin.com/ml/cygwin/2012-05/msg00580.html + So, for now, just disable restarting entirely. */ if (!return_on_signal) goto looping; /* Emulate linux behavior */ +#endif cleanup (); _my_tls.call_signal_handler (); set_sig_errno (EINTR); @@ -389,7 +393,9 @@ select_stuff::wait (fd_set *readfds, fd_set *writefds, fd_set *exceptfds, cleanup (); goto out; } +#if 0 looping: +#endif if (ms == INFINITE) { select_printf ("looping");