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

Re: [ANNOUNCE] Git v2.32.0-rc3 - t5300 Still Broken on NonStop ia64/x86

From
Jeff King <peff@peff.net>
Date
Jun 3, 2021, 19:31 UTC
Message-ID
<YLkt+w9Lxyy8iLS5@coredump.intra.peff.net>
In-Reply-To
<003c01d757ee$c0664600$4132d200$@nexbridge.com>
On Wed, Jun 02, 2021 at 04:34:51PM -0400, Randall S. Becker wrote:
Show 20 quoted lines
> >> The wrapper should apply to all platforms.  NFS (and presumably other
> >> network FSes) can be mounted with interrupts enabled.
> >
> >I don't mind that, as the wrapper is pretty low-cost (and one less Makefile knob is nice). If it's widespread, though, I find it curious that
> >nobody has run into it before now.
> 
> I suspect this is because of the way the file system on NonStop behaves. It is a multi-processor platform, with multi-cores, so anything can happen. If the file system is delayed for any reason, like a signal coming from a different core (EINTR has high priority), then fsync() will be interrupted. EINTR is allowed on NonStop for fsync(). So it would be really great if the patch included a modification to config.mak.uname to include that. This would be a timing-only issue on most other systems, probably something that would hit NFS.
> 
> The patch for the config is:
> diff --git a/config.mak.uname b/config.mak.uname
> index cb443b4e02..ac3e3ca2c5 100644
> --- a/config.mak.uname
> +++ b/config.mak.uname
> @@ -566,6 +566,7 @@ ifeq ($(uname_S),NONSTOP_KERNEL)
>         NO_REGEX = NeedsStartEnd
>         NO_PTHREADS = UnfortunatelyYes
>         FREAD_READS_DIRECTORIES = UnfortunatelyYes
> +       FSYNC_NEEDS_RESTART = YesPlease
> 
>         # Not detected (nor checked for) by './configure'.

Yeah, if we don't make it unconditional, then this is the obvious next step. But the more important question is: did you test this out and did it fix the test breakage you saw on NonStop?

-Peff
Previous: Randall S. BeckerNext: Randall S. Becker
Message 19 of 22 in “RE: [ANNOUNCE] Git v2.32.0-rc3 - t5300 Still Broken on NonStop ia64/x86”
  1. Randall S. BeckerJun 2, 2021
  2. Taylor BlauJun 2, 2021
  3. Jeff KingJun 2, 2021
  4. Taylor BlauJun 2, 2021
  5. Jeff KingJun 2, 2021
  6. Randall S. BeckerJun 2, 2021
  7. Junio C HamanoJun 4, 2021
  8. Taylor BlauJun 4, 2021
  9. Jeff KingJun 4, 2021
  10. Junio C HamanoJun 4, 2021
  11. Randall S. BeckerJun 6, 2021
  12. Jeff KingJun 8, 2021
  13. René ScharfeJun 5, 2021
  14. Junio C HamanoJun 5, 2021
  15. Eric WongJun 2, 2021
  16. Jeff KingJun 2, 2021
  17. Taylor BlauJun 2, 2021
  18. Randall S. BeckerJun 2, 2021
  19. Jeff KingJun 3, 2021
  20. Randall S. BeckerJun 3, 2021
  21. Bryan TurnerJun 3, 2021
  22. Randall S. BeckerJun 3, 2021

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.