Re: SA_RESETHAND race (or the mpv(1) hanging story)

2018-06-23 Thread Hiltjo Posthuma
On Fri, Mar 16, 2018 at 02:40:00PM +0100, Hiltjo Posthuma wrote: > On Fri, Mar 16, 2018 at 02:29:24PM +0100, Hiltjo Posthuma wrote: > > On Thu, Mar 15, 2018 at 03:27:47PM +0100, Martin Pieuchot wrote: > > > On 05/03/18(Mon) 21:26, Hiltjo Posthuma wrote: > > > > > [...] > > > > > I tried your patch

Re: SA_RESETHAND race (or the mpv(1) hanging story)

2018-03-16 Thread Hiltjo Posthuma
On Fri, Mar 16, 2018 at 02:29:24PM +0100, Hiltjo Posthuma wrote: > On Thu, Mar 15, 2018 at 03:27:47PM +0100, Martin Pieuchot wrote: > > On 05/03/18(Mon) 21:26, Hiltjo Posthuma wrote: > > > > [...] > > > > I tried your patch on a source build of about 25 februari on amd64. I > > > > still > > > >

Re: SA_RESETHAND race (or the mpv(1) hanging story)

2018-03-16 Thread Hiltjo Posthuma
On Thu, Mar 15, 2018 at 03:27:47PM +0100, Martin Pieuchot wrote: > On 05/03/18(Mon) 21:26, Hiltjo Posthuma wrote: > > > [...] > > > I tried your patch on a source build of about 25 februari on amd64. I > > > still > > > notice mpv hanging when I try to close it or when the video ends. > > Thanks

Re: SA_RESETHAND race (or the mpv(1) hanging story)

2018-03-15 Thread Martin Pieuchot
On 05/03/18(Mon) 21:26, Hiltjo Posthuma wrote: > > [...] > > I tried your patch on a source build of about 25 februari on amd64. I still > > notice mpv hanging when I try to close it or when the video ends. Thanks for reporting. This is a different issue. > > Let me know how/if I can help you f

Re: SA_RESETHAND race (or the mpv(1) hanging story)

2018-03-05 Thread Hiltjo Posthuma
On Fri, Mar 02, 2018 at 02:18:46PM +0100, Hiltjo Posthuma wrote: > On Wed, Feb 21, 2018 at 02:15:07PM +0100, Martin Pieuchot wrote: > > Diff below fixes a race in signal handling that can be triggered when > > a multi-threaded process installs a signal handler with SA_RESETHAND. > > > > I describe

Re: SA_RESETHAND race (or the mpv(1) hanging story)

2018-03-02 Thread Hiltjo Posthuma
On Wed, Feb 21, 2018 at 02:15:07PM +0100, Martin Pieuchot wrote: > Diff below fixes a race in signal handling that can be triggered when > a multi-threaded process installs a signal handler with SA_RESETHAND. > > I described a case of this problem on bugs@ a month ago: > https://marc.info/?l=ope

SA_RESETHAND race (or the mpv(1) hanging story)

2018-02-21 Thread Martin Pieuchot
Diff below fixes a race in signal handling that can be triggered when a multi-threaded process installs a signal handler with SA_RESETHAND. I described a case of this problem on bugs@ a month ago: https://marc.info/?l=openbsd-bugs&m=151672599508049&w=2 The race occurs when one of the threads is