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

2014-05-05 Thread Duy Nguyen
On Mon, May 5, 2014 at 12:13 AM, David Kastrup d...@gnu.org 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

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

2014-05-05 Thread Duy Nguyen
On Mon, May 5, 2014 at 5:26 PM, Duy Nguyen pclo...@gmail.com 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

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

2014-05-05 Thread Matthieu Moy
Duy Nguyen pclo...@gmail.com writes: On Mon, May 5, 2014 at 12:13 AM, David Kastrup d...@gnu.org 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

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

2014-05-05 Thread David Kastrup
Duy Nguyen pclo...@gmail.com writes: On Mon, May 5, 2014 at 12:13 AM, David Kastrup d...@gnu.org 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

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

2014-05-05 Thread Duy Nguyen
On Mon, May 5, 2014 at 6:03 PM, Matthieu Moy matthieu@grenoble-inp.fr 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

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 the tips

[PATCH] Bump core.deltaBaseCacheLimit to 96m

2014-05-04 Thread David Kastrup
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 for a repository of Emacs repacked with git gc --aggressive (v1.9, resulting in a window size of 250) located