hedecai/fog

Multi-threaded version

Opened this issue · 6 comments

hi,I am looking forward to fog mt version,
the Benchmarks show MT version be even faster to others.
But the function “RasterPaintSerializer_init_mt” is empty in the source 
code ? 
fog mt is not implemented ? 

Original issue reported on code.google.com by weolar@gmail.com on 31 Aug 2011 at 5:00

Hi,

multi-threaded version in trunk is currently not implemented, but if you want 
to run benchmarks you can go backward in SVN, the tests are real, but using 
older Fog.

I'm going to add support for mt-rendering again, I needed only to improve core 
architecture before the main work (this is what I'm doing right now) is done. 
New multithreaded renderer will be based on better code (less synchronization 
between commands).

Original comment by kobalicek.petr on 31 Aug 2011 at 7:00

Adding as BUG so you can see when fixed

Original comment by kobalicek.petr on 31 Aug 2011 at 7:01

  • Changed state: Accepted
Oh,well,thank you!
When can  I  see the better architecture ?
I look forward to seeing :-)

Original comment by weolar@gmail.com on 31 Aug 2011 at 8:01

Thanks :)

Multithreaded version is coming, but I wanted to maximize performance per 
single-core before the mt-version. The reason is that I changed the 
rasterization process and it needs to be verified first. The multithreaded 
version I'm preparing should perform better than the old one, mainly because of 
less synchronization and less memory used by threads in total.

Original comment by kobalicek.petr on 9 Sep 2011 at 7:19

When can  we  see the Mt architecture ?

Original comment by xiubinhu...@gmail.com on 13 Sep 2012 at 7:35

As cited on the project page - multithreaded architecture was temporarily 
removed because I needed to refactor and improve the code. However if you are 
really curious and want to see it's possible to checkout older revision of Fog. 
I just tried to look at revision 600 and it's there.

Original comment by kobalicek.petr on 13 Sep 2012 at 8:14