[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: iolint race condition
From: |
Andrew J. Schorr |
Subject: |
Re: iolint race condition |
Date: |
Tue, 7 Dec 2021 22:20:42 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Dec 07, 2021 at 12:10:00AM -0500, Miguel Pineiro Jr. wrote:
> On Mon, Dec 6, 2021, at 10:29 PM, Andrew J. Schorr wrote:
> > I don't love that it's implementation-dependent, but it may be the most
> > expedient way of solving the problem.
>
> Yes. It's brittle. If the implementation changes, the race resurfaces.
I committed your patch. Thanks.
> Is this test even necessary? In its present form, it covers the same
> ground as the final test, output file and output pipe.
>
> The cksum test has the same implementation dependency which
> you noted. No race there at the moment, though, since the cksum
> child is waited on by the adjacent close.
I don't know, but I guess I'd leave well enough alone for now, until
it breaks again.
Regards,
Andy