[cairo] SSE support in pixman... Ooops

[André Tupinambá]

Ooops,

I wrote the code, check, recheck and check again, ran tests and perfs
and think "That's right, it's working". Sent the patch and went to
sleep...

Now, 3 am, I just wake up with "oh my god, I found a bug"... And a big one.

Please ignore the previous patch and apply this new one. Ignores the
perf-sse2-* too, look at the attachments for the correct ones, they
stills good (5x faster than MMX code).

Now I can go to sleep...

Regards,

André Tupinambá

On Mon, Mar 10, 2008 at 11:20 PM, André Tupinambá <andrelrt at gmail.com> wrote:
> Hi everyone,
>
>  I just subscribe this mailing list and saw some archives about SSE(2)
>  optimization in pixman library. Are this (sub)project running?
>
>  I'm studing the pixman some days and I think that I can help too. So I
>  wrote a proof of concept changing the fbCompositeSrc_8888x8888mmx to
>  use SSE2 instructions.
>
>  In attachment there are the patch and some cairo-perf's results. I
>  don't know much about the autoconf tool so to compile the code I
>  needed to run the configure like this:
>
>  CFLAGS=-msse2 ./configure
>
>  And them change the config.h file to include this:
>
>  #define USE_SSE 1
>
>  The firsts tests results are REALLY good, but I'm not sure that's
>  works fine and not introduce some bad pixels in the result images. The
>  perf was runned using:
>
>  taskset 2 ./cairo-perf -i 500 paint_image_rgba_over > perf-sse2-run1
>
>  Best Regards
>
>  André Tupinambá
>
>  PS: If you saw anything weird in my English, please forgive me. I'm
>  from Brazil and English ain't my first language. Well, the comments
>  are LGPL too, fell free to fix them :)
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pixman-mmx.c.patch
Type: application/octet-stream
Size: 5771 bytes
Desc: not available
Url : http://lists.cairographics.org/archives/cairo/attachments/20080311/003f90eb/attachment.obj 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: perf-sse2-run1
Type: application/octet-stream
Size: 1205 bytes
Desc: not available
Url : http://lists.cairographics.org/archives/cairo/attachments/20080311/003f90eb/attachment-0001.obj 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: perf-sse2-run2
Type: application/octet-stream
Size: 1205 bytes
Desc: not available
Url : http://lists.cairographics.org/archives/cairo/attachments/20080311/003f90eb/attachment-0002.obj