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

Re: [PATCH] repack: respect gc.pid lock

From
Jeff King <peff@peff.net>
Date
Apr 18, 2017, 17:16 UTC
Message-ID
<20170418171646.5a5mjhd4qjr6ot7d@sigill.intra.peff.net>
In-Reply-To
<d5c43adf0b074c6ebe43439bc3fc7539@exmbdft7.ad.twosigma.com>
On Tue, Apr 18, 2017 at 05:08:14PM +0000, David Turner wrote:
> On 64-bit systems, I think core.packedGitLimit doesn't make a 
> lot of sense. There is plenty of address space.  Why not use it?

That's my gut feeling, too. I'd have a slight worry that the OS's paging behavior may respond differently if we have more memory mapped. But that's not based on numbers, just a fear of the unknown. :)

If we have infinite windows anyway, I suspect we could just mmap entire packfiles and forget about all the window complexity in the first place. Although IIRC some operating systems take a long time to set up large mmaps, and we may only need a small part of a large pack.

> I'll ask our git server administrator to adjust core.packedGitLimit
> and turn repacks back on to see if that fixes the issue.
Thanks. Let us know if you get any results, either way.
-Peff
Previous: David TurnerNext: David Turner
Message 7 of 13 in “repack: respect gc.pid lock”
  1. repack: respect gc.pid lockDavid Turner, Apr 13, 2017
  2. Jacob KellerApr 14, 2017
  3. Jeff KingApr 14, 2017
  4. David TurnerApr 17, 2017
  5. Jeff KingApr 18, 2017
  6. David TurnerApr 18, 2017
  7. Jeff KingApr 18, 2017
  8. David TurnerApr 18, 2017
  9. Jeff KingApr 18, 2017
  10. David TurnerApr 18, 2017
  11. Jeff KingApr 18, 2017
  12. David TurnerApr 20, 2017
  13. Jeff KingApr 20, 2017

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.