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

Re: reducing prune sync()s

From
LTLinus Torvalds <torvalds@linuxfoundation.org>
Date
May 30, 2008, 00:32 UTC
Message-ID
<alpine.LFD.1.10.0805291727490.3141@woody.linux-foundation.org>
In-Reply-To
<alpine.LFD.1.10.0805291656260.3141@woody.linux-foundation.org>
On Thu, 29 May 2008, Linus Torvalds wrote:
Show 6 quoted lines
> On Thu, 29 May 2008, Frank Ch. Eigler wrote:
> > 
> >	  Or perhaps having the blanket sync be replaced a
> > list of fsync()s for only the relevant git repository files?
> 
> That would be much better.

Side note: a lot of systems make "fsync()" pretty expensive too. It's one of my main disagreements with most log-based filesystems - fsync() can in theory be fast, but almost always implies flushing the whole log, even if 99.9% of that log is totally unrelated to the actual file you want to fsync().

So fsync() isn't always all that much better than sync(). It *should* be, but reality sometimes bites. So testing should include at least some level of "yes, it actually improves things at least on xyz"...

			Linus
Previous: Linus TorvaldsNext: Frank Ch. Eigler
Message 3 of 15 in “reducing prune sync()s”
  1. Frank Ch. EiglerMay 29, 2008
  2. Linus TorvaldsMay 30, 2008
  3. Linus TorvaldsMay 30, 2008
  4. Frank Ch. EiglerMay 30, 2008
  5. Florian WeimerMay 30, 2008
  6. David DillowMay 30, 2008
  7. Linus TorvaldsMay 30, 2008
  8. Linus TorvaldsMay 30, 2008
  9. Frank Ch. EiglerMay 30, 2008
  10. Linus TorvaldsMay 30, 2008
  11. 1/2 Make pack creation always fsync() the resultLinus Torvalds, May 30, 2008
  12. 2/2 Remove now unnecessary 'sync()' callsLinus Torvalds, May 30, 2008
  13. Nicolas PitreMay 30, 2008
  14. Frank Ch. EiglerMay 31, 2008
  15. Linus TorvaldsJun 2, 2008

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.