On Fri, 7 Dec 2007, Jon Smirl wrote: > On 12/7/07, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > > > > > > On Thu, 6 Dec 2007, Jon Smirl wrote: > > > > > > > > time git blame -C gcc/regclass.c > /dev/null > > > > > > jonsmirl@terra:/video/gcc$ time git blame -C gcc/regclass.c > /dev/null > > > > > > real 1m21.967s > > > user 1m21.329s > > > > Well, I was also hoping for a "compared to not-so-aggressive packing" > > number on the same machine.. IOW, what I was wondering is whether there is > > a visible performance downside to the deeper delta chains in the 300MB > > pack vs the (less aggressive) 500MB pack. > > Same machine with a default pack > > jonsmirl@terra:/video/gcc/.git/objects/pack$ ls -l > total 2145716 > -r--r--r-- 1 jonsmirl jonsmirl 23667932 2007-12-07 02:03 > pack-bd163555ea9240a7fdd07d2708a293872665f48b.idx > -r--r--r-- 1 jonsmirl jonsmirl 2171385413 2007-12-07 02:03 > pack-bd163555ea9240a7fdd07d2708a293872665f48b.pack > jonsmirl@terra:/video/gcc/.git/objects/pack$ > > Delta lengths have virtually no impact. I can confirm this. I just did a repack keeping the default depth of 50 but with window=100 instead of the default of 10, and the pack shrunk from 2171385413 bytes down to 410607140 bytes. So our default window size is definitely not adequate for the gcc repo. OTOH, I recall tytso mentioning something about not having much return on a bigger window size in his tests when he proposed to increase the default delta depth to 50. So there is definitely some kind of threshold at which point the increased window size stops being advantageous wrt the number of cycles involved, and we should find a way to correlate it to the data set to have a better default window size than the current fixed default. Nicolas - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
