Re: [PATCH] docs: update 64-bit core.packedGitLimit default

2017-06-21 Thread Jeff King
On Wed, Jun 21, 2017 at 12:22:08PM -0700, Junio C Hamano wrote: > Jeff King writes: > > > Of course, the best alternative is retaining access to the packs, which > > is what typically happens now on 64-bit systems (it's just that the > > packedGitLimit was set pointlessly low). I'm not sure if y

Re: [PATCH] docs: update 64-bit core.packedGitLimit default

2017-06-21 Thread Junio C Hamano
Jeff King writes: > Of course, the best alternative is retaining access to the packs, which > is what typically happens now on 64-bit systems (it's just that the > packedGitLimit was set pointlessly low). I'm not sure if you're asking > in general, or as a last-ditch effort for 32-bit systems. A

Re: [PATCH] docs: update 64-bit core.packedGitLimit default

2017-06-21 Thread Jeff King
On Wed, Jun 21, 2017 at 11:38:54AM -0700, Junio C Hamano wrote: > Jeff King writes: > > > So the other direction, instead of avoiding the memory limit in (4), is > > to stop closing "small" packs in (2). But I don't think that's a good > > idea. Even with the code after David's patch, you can st

Re: [PATCH] docs: update 64-bit core.packedGitLimit default

2017-06-21 Thread Junio C Hamano
Jeff King writes: > So the other direction, instead of avoiding the memory limit in (4), is > to stop closing "small" packs in (2). But I don't think that's a good > idea. Even with the code after David's patch, you can still trigger the > problem by running out of file descriptors. And if we sto

Re: [PATCH] docs: update 64-bit core.packedGitLimit default

2017-06-21 Thread Jeff King
On Wed, Jun 21, 2017 at 09:25:09AM -0700, Stefan Beller wrote: > > -Default is 256 MiB on 32 bit platforms and 8 GiB on 64 bit platforms. > > +Default is 256 MiB on 32 bit platforms and 32 TiB (effectively > > +unlimited) on 64 bit platforms. > > nit: I would have not written "effectively unlimite

Re: [PATCH] docs: update 64-bit core.packedGitLimit default

2017-06-21 Thread Stefan Beller
ction of >> Documentation/config.txt. > > Looks like we never did that part. Here it is (Junio, this goes on top > of dt/raise-core-packed-git-limit). > > -- >8 -- > Subject: [PATCH] docs: update 64-bit core.packedGitLimit default > > We bumped the default in be4c

[PATCH] docs: update 64-bit core.packedGitLimit default

2017-06-21 Thread Jeff King
rt. Here it is (Junio, this goes on top of dt/raise-core-packed-git-limit). -- >8 -- Subject: [PATCH] docs: update 64-bit core.packedGitLimit default We bumped the default in be4ca2905 (Increase core.packedGitLimit, 2017-04-20) but never adjusted the documentation to match. Signed-off-by: Jeff