<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_ASSIGNED " title="ASSIGNED - piglit.spec.arb_depth_buffer_float.fbo-depth-gl_depth_component32f-copypixels fails" href="https://bugs.freedesktop.org/show_bug.cgi?id=91433#c4">Comment # 4</a> on <a class="bz_bug_link bz_status_ASSIGNED " title="ASSIGNED - piglit.spec.arb_depth_buffer_float.fbo-depth-gl_depth_component32f-copypixels fails" href="https://bugs.freedesktop.org/show_bug.cgi?id=91433">bug 91433</a> from <a class="email" href="mailto:nanleychery@gmail.com" title="Nanley Chery <nanleychery@gmail.com>"> Nanley Chery</a> <pre>(In reply to Illia from <a href="show_bug.cgi?id=91433#c3">comment #3</a>) > <a href="https://patchwork.freedesktop.org/patch/249252/">https://patchwork.freedesktop.org/patch/249252/</a> fixes fail. When problem was > reported intelCopyPixels() was using do_blit_copypixels(). Now > intelCopyPixels() uses _mesa_meta_CopyPixels(). And pack_uint_Z_FLOAT32 > doesn't work correctly. > I found your v2 (<a href="https://patchwork.freedesktop.org/patch/249295/">https://patchwork.freedesktop.org/patch/249295/</a>). Good catch. I don't think we can delete pack_uint_Z_FLOAT32_X24S8 however, we need to copy the S8 component of the buffer. I'm not sure why this test case still works with your patch: $ bin/fbo-depth copypixels GL_DEPTH32F_STENCIL8 -auto Maybe we need to adjust the piglit test? > I'm looking for a way of not receiving this message(Mesa 10.7.0-devel > implementation error: Incorrectly writing swrast's integer depth values to > MESA_FORMAT_Z_FLOAT32 depth buffer) You can find the code which emits this error by grepping for part of the message in the tree. I like using `git grep` personally.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> <li>You are the assignee for the bug.</li> </ul> </body> </html>