[Pixman] [PATCH 2/2] MIPS: DSPr2: Added bilinear over_8888_8_8888 fast path.
nlukic at mips.com
Wed May 23 09:57:32 PDT 2012
I added more explanation in the commit message for that commit.
Sorry for the late reply. If you think that anything is missing in the commit message, please tell me, and I'll update it.
From: pixman-bounces+nlukic=mips.com at lists.freedesktop.org [mailto:pixman-bounces+nlukic=mips.com at lists.freedesktop.org] On Behalf Of Søren Sandmann
Sent: Wednesday, May 23, 2012 6:51 AM
To: Siarhei Siamashka
Cc: pixman at lists.freedesktop.org; nemanja.lukic at rt-rk.com
Subject: Re: [Pixman] [PATCH 2/2] MIPS: DSPr2: Added bilinear over_8888_8_8888 fast path.
Siarhei Siamashka <siarhei.siamashka at gmail.com> writes:
> On Wed, May 16, 2012 at 12:37 AM, Siarhei Siamashka
> <siarhei.siamashka at gmail.com> wrote:
>> On Mon, May 14, 2012 at 9:17 PM, Nemanja Lukic <nemanja.lukic at rt-rk.com> wrote:
>>> Is this small improvement worth making this code vulnerable to endian issues?
>> If you are already satisfied with this level of performance, then it's
>> probably fine for now.
> By the way, I really mean it :) In my opinion, it is generally enough
> that the patches are useful for something and do not cause
> regressions. If implementing additional performance tweaks may take
> too much time, then they can be added later. But it is also important
> to realize that there is still some room for improvement and not to
> drop the optimization work half-way.
> Also maybe you have noticed that pixman-0.26.0 is about to be released
> next week:
> We still need to either fix the bug which causes the test suite
> failure for MIPS DSP ASE. Or at least disable problematic
> optimizations for this stable release.
Yeah, unless someone who understands the fix here:
comes up with a commit message, I'll just revert the optimization before
Pixman mailing list
Pixman at lists.freedesktop.org
More information about the Pixman