<p dir="ltr"><br>
On Oct 8, 2014 6:36 AM, "Iago Toral" <<a href="mailto:itoral@igalia.com">itoral@igalia.com</a>> wrote:<br>
><br>
> El 2014-10-07 21:46, Jason Ekstrand escribió:<br>
><br>
>> On Oct 7, 2014 12:04 PM, "Iago Toral Quiroga" <<a href="mailto:itoral@igalia.com">itoral@igalia.com</a>><br>
>> wrote:<br>
>>  ><br>
>>  > There is a comment warning about the fact that this is not doing<br>
>> what we<br>
>>  > expect, so fix it. This should be doing the same as<br>
>> unpack_B5G6R5_UNORM<br>
>>  > but swapping B and R.<br>
>>  > ---<br>
>>  ><br>
>>  > Jason started some work to auto-generate the format_pack.c and<br>
>>  > format_unpack.c files which I think has this fixed. I am continuing<br>
>> his<br>
>>  > work on this at the moment, but I guess it might make sense to fix<br>
>> this in<br>
>>  > the current code too while that work is on-going.<br>
>><br>
>> Not much time to reply right now, but I seem to recall there being a<br>
>> bit more to fix here.  What about packing and swrast's texel fetch<br>
>> implementation.  Are those OK for this format?<br>
>><br>
>>> No piglit regressions observed.<br>
>><br>
>><br>
>> On what drivers?  It might be good to test on swrast and llvmpipe. <br>
>> I'm not super-concerned there but we should at least try not to break<br>
>> it.<br>
>>  --Jason<br>
><br>
><br>
> That was on Intel, but you are right, at least classic swrast has some regressions with this.<br>
><br>
> For reference, I did not see any regressions on Gallium Softpipe but I could only run a small subset of tests with this driver (-t texture -t color -t format)  or otherwise it would hog my CPU and eventually crash my system. Could not test with llvmpipe, for some reason, even when I am bulding with llvm and I see the llvmpipe sources being built Mesa insists in using the softpipe driver at runtime...</p>
<p dir="ltr">I can give you the configure flags for testing llvmpipe if you'd like.  Yes, doing a full piglit run on llvmpipe or swrast requires a pretty beefy desktop and still takes quite a while.</p>
<p dir="ltr">> Since there are regressions on swrast at least I guess we should just drop this patch until we have a proper fix for all drivers.</p>
<p dir="ltr">I'm not as worried about llvmpipe because it fails about half the tests it attempts anyway.  I would like to know what's going on with swrast though.</p>
<p dir="ltr">Brian, any ideas?</p>
<p dir="ltr">--Jason</p>
<p dir="ltr">><br>
> Iago<br>
><br>
>>><br>
>>  >  src/mesa/main/format_unpack.c | 10 +++-------<br>
>>  >  1 file changed, 3 insertions(+), 7 deletions(-)<br>
>>  ><br>
>>  > diff --git a/src/mesa/main/format_unpack.c<br>
>> b/src/mesa/main/format_unpack.c<br>
>>  > index d5628a9..11b028c 100644<br>
>>  > --- a/src/mesa/main/format_unpack.c<br>
>>  > +++ b/src/mesa/main/format_unpack.c<br>
>>  > @@ -207,16 +207,12 @@ unpack_B5G6R5_UNORM(const void *src, GLfloat<br>
>> dst[][4], GLuint n)<br>
>>  >  static void<br>
>>  >  unpack_R5G6B5_UNORM(const void *src, GLfloat dst[][4], GLuint n)<br>
>>  >  {<br>
>>  > -   /* Warning: this function does not match the current Mesa<br>
>> definition<br>
>>  > -    * of MESA_FORMAT_R5G6B5_UNORM.<br>
>>  > -    */<br>
>>  >     const GLushort *s = ((const GLushort *) src);<br>
>>  >     GLuint i;<br>
>>  >     for (i = 0; i < n; i++) {<br>
>>  > -      GLuint t = (s[i] >> 8) | (s[i] << 8); /* byte swap */<br>
>>  > -      dst[i][RCOMP] = UBYTE_TO_FLOAT( ((t >> 8) & 0xf8) | ((t<br>
>>>><br>
>>>> 13) & 0x7) );<br>
>><br>
>>  > -      dst[i][GCOMP] = UBYTE_TO_FLOAT( ((t >> 3) & 0xfc) | ((t<br>
>>>><br>
>>>>   9) & 0x3) );<br>
>><br>
>>  > -      dst[i][BCOMP] = UBYTE_TO_FLOAT( ((t << 3) & 0xf8) | ((t<br>
>>>><br>
>>>>   2) & 0x7) );<br>
>><br>
>>  > +      dst[i][RCOMP] = ((s[i]      ) & 0x1f) * (1.0F /<br>
>> 31.0F);<br>
>>  > +      dst[i][GCOMP] = ((s[i] >> 5 ) & 0x3f) * (1.0F / 63.0F);<br>
>>  > +      dst[i][BCOMP] = ((s[i] >> 11) & 0x1f) * (1.0F / 31.0F);<br>
>>  >        dst[i][ACOMP] = 1.0F;<br>
>>  >     }<br>
>>  >  }<br>
>>  > --<br>
>>  > 1.9.1<br>
>>  ><br>
>>  > _______________________________________________<br>
>>  > mesa-dev mailing list<br>
>>  > <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
>>  > <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a> [1]<br>
>><br>
>> Links:<br>
>> ------<br>
>> [1] <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>