Re: [PATCH] gitweb: Use config file or file for repository owner's name.

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

 



On Wed, Jan 30, 2008 at 07:06:44PM -0800, Junio C Hamano wrote:
> If the config reading interface is too slow (somebody has to
> bench it on a large set of repositories), maybe we would need to

I made a simple bench as follows.

dd= 'dd if=/dev/zero of=/home/bruno/dds/$i bs=1M count=400000'
Machine: 4*opteron 2.8GHz
     32GB ram
     14*750GB RAID0 sata2 /home
     
Generated a 1000 projects [ too much?! git.debian.org has 668]
For each test a 'echo 2 > /proc/sys/vm/drop_caches' was done before running
it.

command: time gitweb.cgi > /dev/null

-----------------  -----------------  ------------  -----------------
With Project List   NO Project List      LoadAvg     description way
-----------------  -----------------  ------------  -----------------
->0*dd             ->0*dd
   1m0.851s           1m18.651s        0.78  0.70     description file
   1m1.511s           0m55.051s        0.83  0.35     gitweb.description
               
->2*dd             ->2*dd
   21m0.899s          17m19.706s       8.21  6.48     description file
   16m29.455s         13m36.602s       5.90  5.95     gitweb.description
                   
->4*dd             ->4*dd
   23m6.781s          26m51.544s       10.81 12       description file
   20m57.249s         26m32.704s       11.50 12.55    gitweb.description


My test was simple =) But we can get some conclusions on it.
Running a git-config -z -l for each git repository is not a problem, as we
obtain same speed on tests (only loosing on very high IO's) [maybe i should
run with 8*dd or 80...].

After that, having a gitweb.owner might not be a problem as we parse all
config file once.

Running the "git-for-each-ref" is the big killer [waaaw =P]. We could store
timestamp at gitweb.lastchange, which is updated by a hook?! Or store it in
some way[ i will bench with if stored].
-- 
Bruno Ribas - ribas@xxxxxxxxxxxx
http://web.inf.ufpr.br/ribas
C3SL: http://www.c3sl.ufpr.br 
-
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]

  Powered by Linux