Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-19 Thread Eli Zaretskii
> From: Mark H Weaver > Cc: l...@gnu.org (Ludovic Courtès), guile-devel@gnu.org, > gdb-patc...@sourceware.org > Date: Wed, 19 Feb 2014 02:50:36 -0500 > > Eli Zaretskii writes: > > Also, since the only way I could get a functional MinGW Guile was to > > configure it without threads, I would su

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Mark H Weaver
Eli Zaretskii writes: > Also, since the only way I could get a functional MinGW Guile was to > configure it without threads, I would suggest that this be the default > for MinGW, but that isn't a big deal. FWIW, the situation seems to have improved since you last looked. In the last couple of we

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Ludovic Courtès
Eli Zaretskii skribis: >> From: l...@gnu.org (Ludovic Courtès) >> Cc: xdj...@gmail.com, gdb-patc...@sourceware.org, guile-devel@gnu.org >> Date: Tue, 18 Feb 2014 18:45:53 +0100 >> >> > Sorry, I meant why that code couldn't be used when Guile was built >> > without pthreads. >> >> Because with

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Eli Zaretskii
> From: l...@gnu.org (Ludovic Courtès) > Cc: xdj...@gmail.com, gdb-patc...@sourceware.org, guile-devel@gnu.org > Date: Tue, 18 Feb 2014 18:45:53 +0100 > > > Sorry, I meant why that code couldn't be used when Guile was built > > without pthreads. > > Because with the patch Doug posted, both the

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Ludovic Courtès
Eli Zaretskii skribis: >> From: l...@gnu.org (Ludovic Courtès) >> Cc: xdj...@gmail.com, gdb-patc...@sourceware.org, guile-devel@gnu.org >> Date: Tue, 18 Feb 2014 17:45:27 +0100 [...] >> >> The non-pthread code is used when Guile is built without pthread >> >> support. In that case, the async

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Doug Evans
On Tue, Feb 18, 2014 at 3:20 AM, Ludovic Courtès wrote: > Right, when Guile is built with pthread support, it has a signal > delivery thread. The actual SIGINT handler ('take_signal' in scmsigs.c) > just write one byte to a pipe; the signal delivery thread reads from > that pipe, and queues an as

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Eli Zaretskii
> From: l...@gnu.org (Ludovic Courtès) > Cc: xdj...@gmail.com, gdb-patc...@sourceware.org, guile-devel@gnu.org > Date: Tue, 18 Feb 2014 17:45:27 +0100 > > >> I don’t remember, Eli: do you have patches pending review for these > >> issues and other MinGW issues in Guile? > > > > I don't know, you

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Ludovic Courtès
Eli Zaretskii skribis: >> From: l...@gnu.org (Ludovic Courtès) >> Cc: Eli Zaretskii , gdb-patc...@sourceware.org, >> guile-devel@gnu.org >> Date: Tue, 18 Feb 2014 12:20:39 +0100 >> >> Doug Evans skribis: >> >> I don’t remember, Eli: do you have patches pending review for these >> issues and o

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Eli Zaretskii
> From: l...@gnu.org (Ludovic Courtès) > Cc: Eli Zaretskii , gdb-patc...@sourceware.org, > guile-devel@gnu.org > Date: Tue, 18 Feb 2014 12:20:39 +0100 > > Doug Evans skribis: > > I don’t remember, Eli: do you have patches pending review for these > issues and other MinGW issues in Guile? I don

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-18 Thread Ludovic Courtès
Doug Evans skribis: > On Mon, Feb 17, 2014 at 1:13 PM, Eli Zaretskii wrote: >>> Date: Mon, 17 Feb 2014 12:59:22 -0800 >>> From: Doug Evans >>> Cc: "gdb-patc...@sourceware.org" , >>> guile-devel@gnu.org >>> >>> >> +void >>> >> +gdbscm_initialize_sigint (void) >>> >> +{ >>> >> + siscm_sigint_pi

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-17 Thread Doug Evans
On Mon, Feb 17, 2014 at 1:13 PM, Eli Zaretskii wrote: >> Date: Mon, 17 Feb 2014 12:59:22 -0800 >> From: Doug Evans >> Cc: "gdb-patc...@sourceware.org" , >> guile-devel@gnu.org >> >> >> +void >> >> +gdbscm_initialize_sigint (void) >> >> +{ >> >> + siscm_sigint_pipe[0] = siscm_sigint_pipe[1] = -1

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-17 Thread Eli Zaretskii
> Date: Mon, 17 Feb 2014 12:59:22 -0800 > From: Doug Evans > Cc: "gdb-patc...@sourceware.org" , > guile-devel@gnu.org > > >> +void > >> +gdbscm_initialize_sigint (void) > >> +{ > >> + siscm_sigint_pipe[0] = siscm_sigint_pipe[1] = -1; > >> + > >> + if (!SCM_USE_PTHREAD_THREADS) > >> +{ > >>

Re: [PATCH v2] Improved ^c support for gdb/guile

2014-02-17 Thread Doug Evans
[+ guile-devel] On Mon, Feb 17, 2014 at 12:43 PM, Eli Zaretskii wrote: >> From: Doug Evans >> Date: Mon, 17 Feb 2014 15:26:27 -0500 >> >> Unworkable-as-is optimization trying to avoid queueing asyncs. Blech. >> >> I'm still seeing intermittent testsuite failures because Guile is >> getting an u