Re: [PATCH v3 0/2] read-cache: call verify_hdr() in a background thread
- From
- Jeff Hostetler <git@jeffhostetler.com>
- Date
- Mar 31, 2017, 13:23 UTC
- Message-ID
- <094f16d2-e1d9-1478-df6a-4f24f17e184e@jeffhostetler.com>
- In-Reply-To
- <20170330203916.ckp62syupioo3hxc@sigill.intra.peff.net>
On 3/30/2017 4:39 PM, Jeff King wrote:
Show 17 quoted lines
> On Thu, Mar 30, 2017 at 09:06:48PM +0100, Thomas Gummerer wrote: > >>> Yeah, I think that would be fine. You _could_ write a t/perf test and >>> then use your 400MB monstrosity as GIT_PERF_LARGE_REPO. But given that >>> most people don't have such a thing, there's not much value over you >>> just showing off the perf improvement in the commit message. >> >> Sorry if this was already discussed, but we already do have a perf >> test for the index (p0002), and a corresponding helper program which >> just does read_cache() and discard_cache(). Maybe we could re-use >> that and add a second test running the same using the new config? > > Oh, indeed. Yes, I would think the results of p0002 would probably show > off Jeff's improvements. > > -Peff >
Let me re-roll it with Junio's cleanup, update fsck to force it on, and look at using p0002.
Thanks, Jeff