Re: [PATCH] Limit the size of the new delta_base_cache

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Mon, 19 Mar 2007, Shawn O. Pearce wrote:
>
> Round two, based on comments on IRC from Junio:

One more change: please don't even *add* objects to the cache that are 
bigger than 10-20% of the cache limit!

If you start adding big objects, and you have some cache limit, that will 
make the cache seriously less useful by causing eviction of potentially 
much more interesting objects, and also obviously causing the cache code 
itself to spend much more time picking objects (since it's enough to have 
just a few big objects to make the cache eviction decide it needs to 
evict).

Limiting by size is also effective since anything that is more than a 
megabyte is likely to be a blob anyway, and thus much less useful for 
caching in the first place. So there are just tons of reasons to say 
"don't even add it in the first place" if you decide to go with any limit 
at all.

		Linus
-
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]