On 3 May 2012 15:59, Chad Versace <<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>> wrote: <div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 <div class="im"> 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. </div>Thanks for adding tests for the extra formats. <div class="im"> > --- > 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 </div>[snip] <div class="im"> > + * 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 </div>This comment needs to be updated.</blockquote><div> Whoops, you're right. I'll fix before pushing. </div></div>