[Piglit] [PATCH v2] Test rendering to various miplevels of depth and stencil textures.

[Paul Berry]

On 3 May 2012 15:59, Chad Versace <chad.versace at linux.intel.com> wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 05/02/2012 02:30 PM, Paul Berry wrote:
> > This patch adds a new test, "depthstencil-render-miplevels", which
> > verifies that we can render to all miplevels of depth and stencil
> > textures.  It exposes two bugs in Mesa's current i965 driver:
> >
> > - Miptree offsets aren't being calculated correctly for HiZ and
> >   separate stencil buffers, causing miplevels to overlap in those
> >   buffers.
> >
> > - When binding the same texture to the GL_DEPTH_ATTACHMENT and
> >   GL_STENCIL_ATTACHMENT points, Mesa is trying to share internal data
> >   structures even if the miplevels don't match; as a result,
> >   transitioning from one miplevel to another doesn't work.
> >
> > The test is capable of running at any texture size (including
> > non-power-of-two texture sizes).  However, because of some GPU hang
> > bugs in Mesa's current i965 driver, I've limited it to test at a
> > texture size of 1024x1024 for now.
> >
> > v2: Added comment "/* Anonymous namespace */" on the final closing
> > brace.  Adjusted the naming scheme for the subtests to allow
> > additional format combinations to be added in the future.  Also added
> > format combinations involving z32f and z32f_s8.
>
> Thanks for adding tests for the extra formats.
>
> > ---
> >  tests/all.tests                                   |   19 +
> >  tests/texturing/CMakeLists.gl.txt                 |    1 +
> >  tests/texturing/depthstencil-render-miplevels.cpp |  404
> +++++++++++++++++++++
> >  3 files changed, 424 insertions(+), 0 deletions(-)
> >  create mode 100644 tests/texturing/depthstencil-render-miplevels.cpp
>
>
> [snip]
>
> > + * Usage: depthstencil-render-miplevels <texture_size>
> <buffer_combination>
> > + *
> > + *  buffer_combination:          buffer attachments:
> > + *  stencil                      stencil->DEPTH_STENCIL
> > + *  depth_x                      depth->DEPTH_STENCIL
> > + *  depth                        depth->DEPTH_COMPONENT
> > + *  d16                          depth->DEPTH_COMPONENT16
> > + *  depth_x_and_stencil          depth->DEPTH_STENCIL,
> stencil->DEPTH_STENCIL
> > + *  stencil_and_depth_x          (as above, but stencil attached first)
> > + *  depth_and_stencil            depth->DEPTH_COMPONENT,
> stencil->DEPTH_STENCIL
> > + *  stencil_and_depth            (as above, but stencil attached first)
> > + *  depth_stencil_shared         depth->DEPTH_STENCIL<-stencil
> > + *  stencil_depth_shared         (as above, but stencil attached first)
> > + *  depth_stencil_single_binding depth_stencil->DEPTH_STENCIL
>
> This comment needs to be updated.


Whoops, you're right.  I'll fix before pushing.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20120507/3661b5d4/attachment.html>