<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 7, 2015 at 2:09 PM, Oded Gabbay <span dir="ltr"><<a href="mailto:oded.gabbay@gmail.com" target="_blank">oded.gabbay@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Mon, Sep 7, 2015 at 2:04 PM, Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>> wrote:<br>
> On Sun,  6 Sep 2015 18:27:07 +0300<br>
> Oded Gabbay <<a href="mailto:oded.gabbay@gmail.com">oded.gabbay@gmail.com</a>> wrote:<br>
><br>
>> This patch-set contains optimizations for two existing VMX fast-paths and a new<br>
>> VMX fast-path function.<br>
>><br>
>> The optimization ideas came from Siarhei's recent implementation of over_n_8888<br>
>> VMX fast path (see <a href="http://lists.freedesktop.org/archives/pixman/2015-September/003951.html" rel="noreferrer" target="_blank">http://lists.freedesktop.org/archives/pixman/2015-September/003951.html</a>).<br>
>><br>
>> The new function I added is actually one that I already implemented a couple<br>
>> of months ago, but it produced conflicting results regarding the performance.<br>
>> However, I now optimized it and it now shows considerable performance<br>
>> improvement over the non-vmx path.<br>
>><br>
>> The last patch removes many helper functions that caused the less than stellar<br>
>> performance the current fast-paths provide. I removed them as I don't want<br>
>> anyone to try and use them, because there are much better alternatives, as<br>
>> I've demonstrated with this patch-set.<br>
>><br>
>> Thanks,<br>
>><br>
>>       Oded<br>
>><br>
>> Oded Gabbay (4):<br>
>>   vmx: optimize scaled_nearest_scanline_vmx_8888_8888_OVER<br>
>>   vmx: optimize vmx_composite_over_n_8888_8888_ca<br>
>>   vmx: implement fast path vmx_composite_over_n_8_8888<br>
>>   vmx: Remove unused expensive functions<br>
>><br>
>>  pixman/pixman-vmx.c | 439 ++++++++++++++++++----------------------------------<br>
>>  1 file changed, 150 insertions(+), 289 deletions(-)<br>
>><br>
><br>
> Hi Oded,<br>
><br>
> nice diffstat. :-)<br>
><br>
> This series is:<br>
> Acked-by: Pekka Paalanen <<a href="mailto:pekka.paalanen@collabora.co.uk">pekka.paalanen@collabora.co.uk</a>><br>
><br>
> I did notice a few minor issues. Patch 1 has a dereference before<br>
> NULL-check, and you sometimes forget the space before an opening<br>
> parenthesis.<br>
><br>
> I suppose there is no danger of regressing operations you didn't<br>
> touch? ;-)<br>
><br>
><br>
> Thanks,<br>
> pq<br>
<br>
</div></div>HI Pekka,<br>
I run cario benchmark (trimmed) and there was *no* regression.<br>
I don't think optimizing some fast-paths affects other, non-related,<br>
fast-paths. And, of course, I don't think it has *any* impact on non<br>
POWER systems.<br>
However, if someone thinks of a specific other function I need to<br>
check for regression, I'm open for suggestions :)<br>
<span class=""><font color="#888888"><br>
         Oded<br>
</font></span></blockquote></div><br></div><div class="gmail_extra"><div class="gmail_default" style="font-size:small">​It bugged me that there was no change, neither up nor down in cairo benchmark.</div><div class="gmail_default" style="font-size:small">So I rechecked it and I had a wrong setup - cairo used the system-installed pixman instead of my pixman.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">After fixing that, I saw several modest speedups for this patch series:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default">Speedups</div><div class="gmail_default">========</div><div class="gmail_default">image        t-firefox-scrolling  1232.30 (1237.81 0.40%) -> 1080.17 (1097.06 0.99%):  1.14x speedup</div><div class="gmail_default">image        t-gnome-terminal-vim  613.86 (615.04 0.12%) -> 549.73 (551.32 0.13%):  1.12x speedup<br></div><div class="gmail_default">image        t-evolution  405.54 (412.06 0.81%) -> 370.57 (379.11 1.89%):  1.09x speedup<br></div><div class="gmail_default">image        t-gvim  653.02 (655.16 0.16%) -> 615.31 (618.40 1.68%):  1.06x speedup<br></div><div class="gmail_default">image        t-firefox-talos-gfx  919.31 (926.31 0.36%) -> 867.05 (870.01 0.35%):  1.06x speedup<br></div><div class="gmail_default" style="font-size:small">​</div><div class="gmail_default" style="font-size:small">I'll add it into the last commit of this patch-set for future references.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">         Oded</div><br></div><img width="0" height="0" class="mailtrack-img" src="https://mailtrack.io/trace/mail/433ecd7788e001e7ef9a0a93a1acf2caea78a929.png"></div>