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

Jan Stancek
2011-12-15T12:05:13Z

----- 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 12:10:17 PM
> Subject: Re: [LTP] [PATCH v2 2/2] pipeio: prevent race between SIGCHLD and open()
>
> On Thu, Dec 15, 2011 at 2:36 AM, Jan Stancek <jstancek@redhat.com>
> wrote:
> > Garrett,
> >
> > I'm not sure what happened, but I see only first patch from series
> > committed with description belonging to second one.
> >
> > Regards,
> > Jan
>
> I got confused by the multiple patches. Is this thread the one I
> need to look at to rectify the other issues identified?

Yes, this is the one.

I posted single patch yesterday:
[LTP] [PATCH] pipeio: prevent race between SIGCHLD and open()

You requested to use TERRNO instead of strerror(errno), so
while I was at it, I made this change for all existing code in pipeio.c:
[LTP] [PATCH v2 1/2] pipeio: convert strerror(errno) to TERRNO

and then put my original patch (with minor changes) on top of that:
[LTP] [PATCH v2 2/2] pipeio: prevent race between SIGCHLD and open()

Regards,
Jan

> Thanks,
> -Garrett