<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><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#c5">Comment # 5</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 <span class="vcard"><a class="email" href="mailto:illia.iorin@gmail.com" title="Illia <illia.iorin@gmail.com>"> <span class="fn">Illia</span></a>
</span></b>
        <pre>(In reply to Nanley Chery from <a href="show_bug.cgi?id=91433#c4">comment #4</a>)

<span class="quote">> 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?</span >

Test works correct. I've greped mesa and found only one mention  to
"pack_uint_Z_FLOAT32_X24S8" there is in "_mesa_get_pack_uint_z_func" and I've
edited its behavior. When  something calls  "_mesa_get_pack_uint_z_func" with
"MESA_FORMAT_Z32_FLOAT_S8X24_UINT" it will get "pack_uint_Z_FLOAT32" it works
like this in "_mesa_get_pack_float_z_func". 

I think we can  add comments to both functions if someone will look for  
"pack_uint_Z_FLOAT32_X24S8" or "pack_float_Z_FLOAT32_X24S8".

<span class="quote">> > 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.</span >
Thanks, I've found commit which implement sending this error.  It close related
 with our bug. I think we can painless revert this commit. it's done its work.

commit fceff14450e48a90778e0d1e79c13fa7a40631e6
Author: Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>>

    mesa: Add a _mesa_problem to document a piglit failure on i965.

    Having figured out what was going on with piglit fbo-depth copypixels
    GL_DEPTH_COMPONENT32...</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>