On Fri, Oct 2, 2015 at 4:47 PM, Ian Romanick <i...@freedesktop.org> wrote:
> On 10/01/2015 12:15 PM, Ilia Mirkin wrote:
>> On Thu, Oct 1, 2015 at 3:12 PM, Ian Romanick <i...@freedesktop.org> wrote:
>>> I'm just
>>> wondering because Mesa doesn't support that extension. How is this even
>>> being hit?
>>
>> See 81d2fd91a90 (mesa: add NV_read_{depth,stencil,depth_stencil} extensions)
>
> Okay, that's weird. I must have had some old branch checked out at the
> time. I did 'grep -r NV_read_depth src/', and it came back empty. Now
> I just get to be irritated that we enabled THREE extensions for which we
> have ZERO tests... and least one is clearly completely broken. :(
>
> I guess now I at least have something concrete to point to then next
> time I object to enabling an ES extension that "just" allows some
> desktop functionality. ;)
I believe Rob tested at least some of it with qapitrace[1], as
otherwise there was no way to get access to the data in a
renderbuffer, which can be quite useful for debugging. Not all of us
have your level of familiarity with how GL works, but how will we
learn without making some mistakes? :)
No matter how many tests we might have, they'll always leave
*something* out. The fact that there are no tests at all for this ext
isn't great, of course. But there are also no functional tests for
{ARB,AMD}_conservative_depth and probably a number of others.
-ilia
1.
https://github.com/apitrace/apitrace/commit/7ad1d34c97cb0d2f07a381d7664f1ba0f1be410b.patch
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
