Use a thread for ppm exporter

[GoogleCodeExporter]

Currently the gource processing is completely blocked while waiting to
write a frame to the output stream. On a system with more than 1 CPU core
you can get better throughput by using a separate thread to write the
output. A rudimentary patch is provided as an example. It works for me but
you may want to dress it up a bit.

Original issue reported on code.google.com by Highland...@gmail.com on 18 Jan 2010 at 7:28

Attachments:

• dif.txt

[GoogleCodeExporter]

Good idea.

FYI ppm.cpp has changed a bit (not released yet) as I improved the performance 
by
doing less writes, so the patch will need some merging.

http://github.com/acaudwell/Gource/commit/7857f6fe0e5dd3e25034f7d6b31ec83fd3564e
14

Original comment by acaudw...@gmail.com on 18 Jan 2010 at 7:42

• Added labels: Type-Other
• Removed labels: Type-Defect

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Here's a patch against current git. It's a bit more C++ oriented...

Original comment by Highland...@gmail.com on 18 Jan 2010 at 8:40

Attachments:

• 0001-Use-a-thread-for-the-exporter.patch

[GoogleCodeExporter]

Nice. I will try it out later this week.

Original comment by acaudw...@gmail.com on 18 Jan 2010 at 8:48

• Changed state: Accepted

[GoogleCodeExporter]

Hi. I've merged your code into this branch:

http://github.com/acaudwell/Gource/tree/ppmthread

http://gource.googlecode.com/files/gource-0.24-beta1.tar.gz

I've changed the code to use the SDL_thread API since that is more portable (the
SDL_thread API is essentially identical). I made a few other changes like adding
double buffering, and doing the pixel flipping on the video card.

In my tests its about 2fps faster (11fps vs 9fps) with the thread than without 
using
this command line:

./gource --stop-at-end --output-ppm-stream - | ffmpeg -y -b 3000K -r 60 -f 
image2pipe
-vcodec ppm -i - -vcodec libx264 gource.mp4

(running against Gource.git)

It'd be good if you could have a look over changes before I add this to the 
master.

Cheers

Original comment by acaudw...@gmail.com on 28 Jan 2010 at 3:02

[GoogleCodeExporter]

I've updated the beta:

http://gource.googlecode.com/files/gource-0.24-beta2.tar.gz

Original comment by acaudw...@gmail.com on 31 Jan 2010 at 10:30

[GoogleCodeExporter]

Original comment by acaudw...@gmail.com on 31 Jan 2010 at 10:37

• Changed state: Fixed