<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - piglit.spec.arb_internalformat_query2.samples and num_sample_counts pname checks"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=104794#c1">Comment # 1</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - piglit.spec.arb_internalformat_query2.samples and num_sample_counts pname checks"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=104794">bug 104794</a>
              from <span class="vcard"><a class="email" href="mailto:apuentes@igalia.com" title="Antia Puentes <apuentes@igalia.com>"> <span class="fn">Antia Puentes</span></a>
</span></b>
        <pre>This should be fixed now.

The commit originating the regressions was reverted by me, however, in issue 
<a href="https://github.com/KhronosGroup/OpenGL-API/issues/32">https://github.com/KhronosGroup/OpenGL-API/issues/32</a> it was clarified that the
commit implemented the correct behaviour.

Commit is now restored in Mesa master and the regressions fixed in Piglit.

** Mesa **

commit e1623b303ccc7a880e34bddef7231dcf230826b3
Author: Juan A. Suarez Romero <<a href="mailto:jasuarez@igalia.com">jasuarez@igalia.com</a>>
Date:   Mon Jan 15 10:58:50 2018 +0000

    mesa: add missing RGB9_E5 format in _mesa_base_fbo_format

    RGB9_E5 should be accepted by RenderbufferStorage if the
    EXT_texture_shared_exponent is exposed. It is left to the
    implementations to return GL_FRAMEBUFFER_UNSUPPORTED_EXT
    when checking the framebuffer completeness if they do not
    support rendering in this format.

    Discussed in:
    <a href="https://github.com/KhronosGroup/OpenGL-API/issues/32">https://github.com/KhronosGroup/OpenGL-API/issues/32</a>

    This fixes KHR-GL45.internalformat.renderbuffer.rgb9_e5

    v2: Added more info to the commit message (Antia)

    Reviewed-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>>
    Reviewed-by: Antia Puentes <<a href="mailto:apuentes@igalia.com">apuentes@igalia.com</a>>

** Piglit **

commit 5f11fdff7b329eb5712eed98f783a6164410336f
Author: Antia Puentes <<a href="mailto:apuentes@igalia.com">apuentes@igalia.com</a>>
Date:   Tue Feb 20 13:55:13 2018 +0100

    internalformat_query*: fix SAMPLES pname checks for RGB9_E5

    SAMPLES and NUM_SAMPLE_COUNTS queries accept internalformats which
    are defined as color-, depth- or stencil-renderable. RGB9_E5 is marked
    as non color-renderable in OpenGL 4.6, however if the
    EXT_texture_shared_exponent extension is exposed it must be considered
    as such. The later was discussed in:
    <a href="https://github.com/KhronosGroup/OpenGL-API/issues/32">https://github.com/KhronosGroup/OpenGL-API/issues/32</a>

    v2: use 'tex' as shortname for texture instead of 'text' (Piñeiro)

    Bugzilla: <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - piglit.spec.arb_internalformat_query2.samples and num_sample_counts pname checks"
   href="show_bug.cgi?id=104794">https://bugs.freedesktop.org/show_bug.cgi?id=104794</a>

    Reviewed-by: Alejandro Piñeiro <<a href="mailto:apinheiro@igalia.com">apinheiro@igalia.com</a>></pre>
        </div>
      </p>


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

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