git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* git-annotate - leaking hundrets of megabytes
@ 2007-10-23 17:00 Sven Herzberg
  2007-10-23 18:14 ` Linus Torvalds
  0 siblings, 1 reply; 3+ messages in thread
From: Sven Herzberg @ 2007-10-23 17:00 UTC (permalink / raw)
  To: git

Hey,

I just started to write a small tool that's supposed to provide a
graphical annotation view. It's supposed to display the information of
git-annotate and provide a way to browse through the history (so people
can click on links that refer to older revisions, etc.). According to
the man page of git-annotate, I looked into the incremental mode to make
sure the tool can load the information step by step.

However, to make sure the user interface doesn't block while
"git-annotate --incremental" is running, I decided to take a really slow
annotation while developing and then I saw that git-annotate leaks
*lots* of memory (at least in that use-case).

In my example, it used up to 450MB:
http://people.imendio.com/~sven/massif.18740.png

Reproduce:
1. git clone git://git.webkit.org/WebKit.git
2. cd WebKit/WebCore
3. git-annotate --incremental ChangeLog

Regards,
  Sven

PS: Please CC me as I'm not on the list.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: git-annotate - leaking hundrets of megabytes
  2007-10-23 17:00 git-annotate - leaking hundrets of megabytes Sven Herzberg
@ 2007-10-23 18:14 ` Linus Torvalds
  2007-10-23 20:12   ` Florian Weimer
  0 siblings, 1 reply; 3+ messages in thread
From: Linus Torvalds @ 2007-10-23 18:14 UTC (permalink / raw)
  To: Sven Herzberg; +Cc: git



On Tue, 23 Oct 2007, Sven Herzberg wrote:
> 
> In my example, it used up to 450MB:
> http://people.imendio.com/~sven/massif.18740.png
> 
> Reproduce:
> 1. git clone git://git.webkit.org/WebKit.git
> 2. cd WebKit/WebCore
> 3. git-annotate --incremental ChangeLog

Hmm. I'm not seeing this at all. I do

	/usr/bin/time git-annotate --incremental ChangeLog > /dev/null

in WebCore, and it returns

	0.16user 0.01system 0:00.18elapsed 99%CPU (0avgtext+0avgdata 0maxresident)k
	0inputs+0outputs (0major+8365minor)pagefaults 0swaps

where that minor pagefault count is a good estimate of total memory used. 
8365 minor page faults implies it only ever got 32MB of memory total.

What git version do you have, or did you perhaps mean some other file or 
other config that causes this?

(It does end up using lots more memory if you ask for "-C" to see if there 
are copies from other sources, but even then I'm not seeing anything 
close to 450MB!)

			Linus

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: git-annotate - leaking hundrets of megabytes
  2007-10-23 18:14 ` Linus Torvalds
@ 2007-10-23 20:12   ` Florian Weimer
  0 siblings, 0 replies; 3+ messages in thread
From: Florian Weimer @ 2007-10-23 20:12 UTC (permalink / raw)
  To: git

* Linus Torvalds:

> What git version do you have, or did you perhaps mean some other file or 
> other config that causes this?

I see it with Debian's 1:1.5.3.4-1 version, but also with dd8175f83c725.
In my case, it's a git-svn mirror of:

  svn://svn.debian.org/secure-testing/data/CVE/list

This is a very difficult file for git-annotate (and "svn blame") because
of its history and size.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-10-23 20:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-23 17:00 git-annotate - leaking hundrets of megabytes Sven Herzberg
2007-10-23 18:14 ` Linus Torvalds
2007-10-23 20:12   ` Florian Weimer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).