bug#16737: [PATCH] Re-enable SIGIO when waiting for events

bug#16737: [PATCH] Re-enable SIGIO when waiting for events

[Tassilo Horn]

Mike Crowe <mac@mcrowe.com> writes:

>> > Thanks for checking.  I installed a similar patch as Emacs master
>> > commit 0592cefd03f1de2f04b721d07a16e6e0a9e48f73; could you please
>> > give it a try?  It is like Mike Crowe's patch, except it avoids a
>> > race that could lose SIGIOs.  I'll also send a heads-up email to
>> > the relevant bug reports.
>> 
>> I prefer your version and will switch to testing it instead.
>
> I've been testing with 0592cefd03f1de2f04b721d07a16e6e0a9e48f73
> applied to Debian Jessie's emacs24 (24.4+1-5) for over a week now
> without seeing any problems so I think we can call the bug fixed.

And I run the git head updating daily, and the issue didn't occur to me
since that commit whereas it bit me several times a day before.  So I
can confirm that it's fixed (as multiple other already did in the
debbugs issue), so I close it with this mail.

Thanks!
Tassilo

bug#16737: [PATCH] Re-enable SIGIO when waiting for events

[Alex Bennée]

Tassilo Horn <tsdh@gnu.org> writes:

> Mike Crowe <mac@mcrowe.com> writes:
>
>>> > Thanks for checking.  I installed a similar patch as Emacs master
>>> > commit 0592cefd03f1de2f04b721d07a16e6e0a9e48f73; could you please
>>> > give it a try?  It is like Mike Crowe's patch, except it avoids a
>>> > race that could lose SIGIOs.  I'll also send a heads-up email to
>>> > the relevant bug reports.
>>> 
>>> I prefer your version and will switch to testing it instead.
>>
>> I've been testing with 0592cefd03f1de2f04b721d07a16e6e0a9e48f73
>> applied to Debian Jessie's emacs24 (24.4+1-5) for over a week now
>> without seeing any problems so I think we can call the bug fixed.
>
> And I run the git head updating daily, and the issue didn't occur to me
> since that commit whereas it bit me several times a day before.  So I
> can confirm that it's fixed (as multiple other already did in the
> debbugs issue), so I close it with this mail.

Will it get applied to the emacs-24 branch or is that considered dead now?

>
> Thanks!
> Tassilo

-- 
Alex Bennée

bug#16737: [PATCH] Re-enable SIGIO when waiting for events (bug# 16737)

[Alan D. Salewski]

[Sent this to emacs-devel, too, but needed to "unarchive" the bug in the
bug tracker and then re-send separately to add this comment]

On 2015-09-13 14:57:23, Alan D. Salewski spake thus:
> On 2015-08-05 10:46:21, Paul Eggert spake thus:
> > On 08/05/2015 07:28 AM, Mike Crowe wrote:
> > >Is there any chance of committing the fix to emacs-24? It doesn't
> > >cherry-pick cleanly but only requires minor fixing (attached.)
> > 
> > Don't see why not.  I pushed it.  I doubt whether emacs-24 will be
> > used for any new release, so this is mostly just a bookkeeping
> > entry.
> 
> I just want to say thanks to all who have contributed to addressing this
> bug, and especially to Mike Crowe for homing in on the problem and Paul
> Eggert for the final fix (and its application to both branches[0]).  I
> very much appreciate all the time and effort folks have expended on this
> issue.
> 
> Thank you!
> 
> -Al
> 
> 
> [0] master   (commit: 0592cefd03f1de2f04b721d07a16e6e0a9e48f73 )
>     and
>     emacs-24 (commit: a27ae9d7650a1230d4359eaf0a949f827315a6d2 )
>