git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 26, 2012, 22:58 UTC
Message-ID
<7vsjdh8yu8.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20120626224053.GA7155@burratino>
Jonathan Nieder <jrnieder@gmail.com> writes:
Show 10 quoted lines
> Hi,
>
> Quick first impressions:
>
> Anders Kaseorg wrote:
>
>> Sidestep this problem by opening the backflow FIFO once for
>> read+write.
>
> Is that portable?

If you mean [n]<>word, it probably is, even though it is somewhat unusual and I wouldn't be surprised if implementations get wrong.

>>             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?
Very good point.
Previous: Anders KaseorgNext: Stefano Lattarini
Message 4 of 5 in “t9010: Open backflow FIFO once to work around kernel race condition”
  1. t9010: Open backflow FIFO once to work around kernel race conditionAnders Kaseorg, Jun 26, 2012
  2. Jonathan NiederJun 26, 2012
  3. Anders KaseorgJun 26, 2012
  4. Junio C HamanoJun 26, 2012
  5. Stefano LattariniJun 26, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.