From: Shawn O. Pearce <spearce@spearce.org>
Date: Tue, 03 Apr 2007 21:11:49 GMT
Subject: Re: git-index-pack really does suck..
Message-ID: <20070403211149.GI27706@spearce.org>
In-Reply-To: <alpine.LFD.0.98.0704031657130.28181@xanadu.home>

Nicolas Pitre <nico@cam.org> wrote:
> On Tue, 3 Apr 2007, Linus Torvalds wrote:
> 
> > All that extra memory is just for SHA1 commit ID information. 
> 
> I don't see where that might be.  The only thing that the paranoia check 
> triggers is:
> 
> 	foo = read_sha1_file(blah);
> 	memcmp(foo with bar);
> 	free(foo);
> 
> So where is that commit ID information actually stored when using 
> read_sha1_file()?

That might just be the mmap code in sha1_file.c.  We are mmaping
the existing packfile, to fetch the object.  That counts as virtual
memory.  ;-)

-- 
Shawn.