Re: [PATCH] Bump core.deltaBaseCacheLimit to 96m

2014-05-05 Thread Jeff King
On Mon, May 05, 2014 at 01:20:09PM +0200, David Kastrup wrote: > > Would it make more sense to advise git devs to set this per repo > > instead? The majority of (open source) repositories out there are > > small if I'm not mistaken. Of those few big repos, we could have a > > section listing all t

Re: [PATCH] Bump core.deltaBaseCacheLimit to 96m

2014-05-05 Thread Duy Nguyen
On Mon, May 5, 2014 at 6:03 PM, Matthieu Moy wrote: >>> -Default is 16 MiB on all platforms. This should be reasonable >>> +Default is 96 MiB on all platforms. This should be reasonable >>> for all users/operating systems, except on the largest projects. >>> You probably do not need to adjust

Re: [PATCH] Bump core.deltaBaseCacheLimit to 96m

2014-05-05 Thread David Kastrup
Duy Nguyen writes: > On Mon, May 5, 2014 at 12:13 AM, David Kastrup wrote: >> The default of 16m causes serious thrashing for large delta chains >> combined with large files. >> >> Here are some benchmarks (pu variant of git blame): >> >> time git blame -C src/xdisp.c >/dev/null > > ... > >> dif

Re: [PATCH] Bump core.deltaBaseCacheLimit to 96m

2014-05-05 Thread Matthieu Moy
Duy Nguyen writes: > On Mon, May 5, 2014 at 12:13 AM, David Kastrup wrote: >> The default of 16m causes serious thrashing for large delta chains >> combined with large files. >> >> Here are some benchmarks (pu variant of git blame): >> >> time git blame -C src/xdisp.c >/dev/null > > ... > >> dif

Re: [PATCH] Bump core.deltaBaseCacheLimit to 96m

2014-05-05 Thread Duy Nguyen
On Mon, May 5, 2014 at 5:26 PM, Duy Nguyen wrote: > part. Would it make more sense to advise git devs to set this per repo s/advise git devs/advise emacs devs/ -- Duy -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majo

Re: [PATCH] Bump core.deltaBaseCacheLimit to 96m

2014-05-05 Thread Duy Nguyen
On Mon, May 5, 2014 at 12:13 AM, David Kastrup wrote: > The default of 16m causes serious thrashing for large delta chains > combined with large files. > > Here are some benchmarks (pu variant of git blame): > > time git blame -C src/xdisp.c >/dev/null ... > diff --git a/Documentation/config.txt