Subject: Re: [PATCH] t9010: Open backflow FIFO once to work around kernel race condition

From: Jonathan Nieder <[email-masked]>
Date: Tue, 26 Jun 2012 17:40:53 -0500

Hi,

Quick first impressions:

Anders Kaseorg wrote:

> Sidestep this problem by opening the backflow FIFO once for
> read+write.

Is that portable?

>             Also, replace the stream FIFO with a shell pipe so we
> don’t have to do manual process management.

Doesn't this mean we wouldn't notice if test-svn-fe crashes?

Maybe it would be simpler to make test-svn-fe launch fast-import as a
child, avoiding the fifos altogether.

Thanks again for tracking this down, by the way.
Jonathan