Re: [LTP] [PATCH v2 2/2] pipeio: prevent race between SIGCHLD and open()

Jan Stancek
2011-12-15T10:36:22Z

Garrett,

I'm not sure what happened, but I see only first patch from series
committed with description belonging to second one.

Regards,
Jan

----- Original Message -----
> From: "Garrett Cooper" <yanegomi@gmail.com>
> To: "Jan Stancek" <jstancek@redhat.com>
> Cc: ltp-list@lists.sourceforge.net
> Sent: Thursday, December 15, 2011 11:15:34 AM
> Subject: Re: [LTP] [PATCH v2 2/2] pipeio: prevent race between SIGCHLD and open()
>
> On Thu, Dec 15, 2011 at 1:35 AM, Jan Stancek <jstancek@redhat.com>
> wrote:
> >
> > This test occasionally hangs on some machines. The hang has been
> > observed mostly on single CPU ones.
> >
> > pipeio code is using signal(2), setting by default SA_RESTART
> > flag, which is also the case for SIGCHLD.
> >
> > If last child manages to exit while parent is still at open(),
> > parent gets SIGCHLD and open() is restarted. At this point test
> > hangs.
> >
> > Here's strace output from parent point of view:
>
> Committed -- thanks!
> -Garrett