login
Header Space

 
 

Re: Something is broken in repack

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Nicolas Pitre <nico@...>, Junio C Hamano <gitster@...>, <gcc@...>
Cc: Git Mailing List <git@...>
Date: Tuesday, December 11, 2007 - 3:01 am

Switching to the Google perftools malloc
http://goog-perftools.sourceforge.net/

10%   30  828M
20%   15  831M
30%   10  834M
40%   50  1014M
50%   80  1086M
60%   80  1500M
70% 200  1.53G
80% 200  1.85G
90% 260  1.87G
95% 520  1.97G
100% 1335 2.24G

Google allocator knocked 600MB off from memory use.
Memory consumption did not fall during the write out phase like it did with gcc.

Since all of this is with the same code except for changing the
threading split, those runs where memory consumption went to 4.5GB
with the gcc allocator must have triggered an extreme problem with
fragmentation.

Total CPU time 196 CPU minutes vs 190 for gcc. Google's claims of
being faster are not true.

So why does our threaded code take 20 CPU minutes longer (12%) to run
than the same code with a single thread? Clock time is obviously
faster. Are the threads working too close to each other in memory and
bouncing cache lines between the cores? Q6600 is just two E6600s in
the same package, the caches are not shared.

Why does the threaded code need 2.24GB (google allocator, 2.85GB gcc)
with 4 threads? But only need 950MB with one thread? Where's the extra
gigabyte going?

Is there another allocator to try? One that combines Google's
efficiency with gcc's speed?


On 12/11/07, Jon Smirl <jonsmirl@gmail.com> wrote:


-- 
Jon Smirl
jonsmirl@gmail.com
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Something is broken in repack, Jon Smirl, (Fri Dec 7, 7:05 pm)
Re: Something is broken in repack, Nicolas Pitre, (Mon Dec 10, 3:56 pm)
Re: Something is broken in repack, Jon Smirl, (Mon Dec 10, 4:05 pm)
Re: Something is broken in repack, Morten Welinder, (Mon Dec 10, 4:16 pm)
Re: Something is broken in repack, Nicolas Pitre, (Fri Dec 7, 9:46 pm)
Re: Something is broken in repack, Jon Smirl, (Fri Dec 7, 10:22 pm)
Re: Something is broken in repack, Jon Smirl, (Fri Dec 7, 10:04 pm)
Re: Something is broken in repack, Nicolas Pitre, (Fri Dec 7, 10:28 pm)
Re: Something is broken in repack, Jon Smirl, (Fri Dec 7, 11:29 pm)
Re: Something is broken in repack, David Brown, (Fri Dec 7, 11:37 pm)
Re: Something is broken in repack, Jon Smirl, (Sat Dec 8, 12:22 am)
Re: Something is broken in repack, Nicolas Pitre, (Sat Dec 8, 12:30 am)
Re: Something is broken in repack, Jon Smirl, (Sat Dec 8, 1:01 am)
Re: Something is broken in repack, Nicolas Pitre, (Sat Dec 8, 1:12 am)
Re: Something is broken in repack, Harvey Harrison, (Fri Dec 7, 11:48 pm)
Re: Something is broken in repack, Junio C Hamano, (Sat Dec 8, 6:18 pm)
Re: Something is broken in repack, Nicolas Pitre, (Sun Dec 9, 10:49 pm)
Re: Something is broken in repack, Junio C Hamano, (Sun Dec 9, 4:05 am)
Re: Something is broken in repack, Jon Smirl, (Sun Dec 9, 2:25 pm)
Re: Something is broken in repack, Nicolas Pitre, (Sun Dec 9, 9:07 pm)
Re: Something is broken in repack, Jon Smirl, (Sun Dec 9, 11:19 am)
Re: Something is broken in repack, Harvey Harrison, (Fri Dec 7, 11:44 pm)
Re: Something is broken in repack, Linus Torvalds, (Fri Dec 7, 8:37 pm)
[PATCH] pack-objects: fix delta cache size accounting, Nicolas Pitre, (Fri Dec 7, 9:27 pm)
Re: Something is broken in repack, Jon Smirl, (Mon Dec 10, 10:25 pm)
Re: Something is broken in repack, Nicolas Pitre, (Mon Dec 10, 11:49 pm)
Re: Something is broken in repack, Jon Smirl, (Tue Dec 11, 1:25 am)
Re: Something is broken in repack, Sean, (Tue Dec 11, 2:01 am)
Re: Something is broken in repack, Jon Smirl, (Tue Dec 11, 2:20 am)
Re: Something is broken in repack, Jon Smirl, (Tue Dec 11, 1:29 am)
Re: Something is broken in repack, Nicolas Pitre, (Tue Dec 11, 9:31 am)
Re: Something is broken in repack, Jon Smirl, (Tue Dec 11, 3:01 am)
Re: Something is broken in repack, Nicolas Pitre, (Tue Dec 11, 9:49 am)
Re: Something is broken in repack, Nicolas Pitre, (Tue Dec 11, 11:00 am)
Re: Something is broken in repack, Nicolas Pitre, (Tue Dec 11, 12:20 pm)
Re: Something is broken in repack, Jon Smirl, (Tue Dec 11, 12:21 pm)
Re: Something is broken in repack, Nicolas Pitre, (Wed Dec 12, 1:12 am)
Re: Something is broken in repack, Nicolas Pitre, (Wed Dec 12, 12:13 pm)
Re: Something is broken in repack, Andreas Ericsson, (Thu Dec 13, 3:32 am)
Re: Something is broken in repack, Wolfram Gloger, (Fri Dec 14, 12:03 pm)
Re: Something is broken in repack, Nicolas Pitre, (Wed Dec 12, 11:48 am)
Re: Something is broken in repack, Nguyen Thai Ngoc Duy, (Thu Dec 13, 9:32 am)
Re: Something is broken in repack, Paolo Bonzini, (Thu Dec 13, 11:32 am)
Re: Something is broken in repack, Johannes Sixt, (Thu Dec 13, 12:39 pm)
Re: Something is broken in repack, Paolo Bonzini, (Thu Dec 13, 12:29 pm)
Re: Something is broken in repack, Linus Torvalds, (Wed Dec 12, 12:37 pm)
Re: Something is broken in repack, Wolfram Gloger, (Fri Dec 14, 12:12 pm)
Re: Something is broken in repack, David Kastrup, (Fri Dec 14, 12:45 pm)
Re: Something is broken in repack, Wolfram Gloger, (Fri Dec 14, 12:59 pm)
Re: Something is broken in repack, Jon Smirl, (Wed Dec 12, 1:12 pm)
Re: Something is broken in repack, David Miller, (Wed Dec 12, 12:42 pm)
Re: Something is broken in repack, Linus Torvalds, (Wed Dec 12, 12:54 pm)
Re: Something is broken in repack, David Kastrup, (Wed Dec 12, 4:05 am)
Re: Something is broken in repack, Wolfram Gloger, (Fri Dec 14, 12:18 pm)
Re: Something is broken in repack, Jon Smirl, (Tue Dec 11, 11:36 am)
Re: Something is broken in repack, Linus Torvalds, (Tue Dec 11, 12:33 pm)
Re: Something is broken in repack, Nicolas Pitre, (Tue Dec 11, 1:21 pm)
Re: Something is broken in repack, David Miller, (Tue Dec 11, 1:24 pm)
Re: Something is broken in repack, Nicolas Pitre, (Tue Dec 11, 1:44 pm)
Re: Something is broken in repack, Andreas Ericsson, (Tue Dec 11, 4:26 pm)
Re: Something is broken in repack, Daniel Berlin, (Tue Dec 11, 1:28 pm)
Re: Something is broken in repack, Andreas Ericsson, (Tue Dec 11, 3:34 am)
Re: Something is broken in repack, Junio C Hamano, (Mon Dec 10, 10:55 pm)
Re: Something is broken in repack, Nicolas Pitre, (Mon Dec 10, 11:27 pm)
Re: Something is broken in repack, David Kastrup, (Tue Dec 11, 7:08 am)
Re: Something is broken in repack, Pierre Habouzit, (Tue Dec 11, 8:08 am)
Re: Something is broken in repack, David Kastrup, (Tue Dec 11, 8:18 am)
Re: Something is broken in repack, David Brown, (Fri Dec 7, 10:56 pm)
speck-geostationary