<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [regression][bisected] glReadPixels() from pbuffer failing in Android CTS camera tests"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101704">101704</a>
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>[regression][bisected] glReadPixels() from pbuffer failing in Android CTS camera tests
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>Mesa
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>git
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>x86-64 (AMD64)
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>Linux (All)
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>normal
          </td>
        </tr>

        <tr>
          <th>Priority</th>
          <td>medium
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>Drivers/DRI/i965
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>intel-3d-bugs@lists.freedesktop.org
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>tfiga@chromium.org
          </td>
        </tr>

        <tr>
          <th>QA Contact</th>
          <td>intel-3d-bugs@lists.freedesktop.org
          </td>
        </tr></table>
      <p>
        <div>
        <pre>Looks like there is a regression affecting glReadPixels() from pbuffer on
Android, which manifests in following Android 7.1 CTS tests:

android.hardware.camera2.cts.MultiViewTest#testDualTextureViewAndImageReaderPreview
android.hardware.camera2.cts.MultiViewTest#testTextureViewPreviewWithImageReader
android.hardware.camera2.cts.PerformanceTest#testCameraLaunch
android.hardware.camera2.cts.PerformanceTest#testSingleCapture
android.hardware.camera2.cts.RobustnessTest#testMandatoryOutputCombinations
android.hardware.camera2.cts.SurfaceViewPreviewTest#testSurfaceSet

Log of a failed test attached. I tracked down the exact failure to be a
GL_INVALID_OPERATION at

<a href="https://android.googlesource.com/platform/frameworks/base/+/master/core/java/android/hardware/camera2/legacy/SurfaceTextureRenderer.java#796">https://android.googlesource.com/platform/frameworks/base/+/master/core/java/android/hardware/camera2/legacy/SurfaceTextureRenderer.java#796</a>

The error is being generated twice, once by _mesa_get_color_read_format() and
then by _mesa_ReadnPixelsARB():

<a href="https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/framebuffer.c#n831">https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/framebuffer.c#n831</a>
<a href="https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/readpix.c#n1078">https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/readpix.c#n1078</a>


Bisected to:

b7153c3e9f9d2d430b0338313587a00e531e4800 is the first bad commit
commit b7153c3e9f9d2d430b0338313587a00e531e4800
Author: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>>
Date:   Fri Jun 9 12:57:09 2017 -0700

    i965: Call intel_prepare_render() from intel_update_state()

Bisect log attached. Revert from top of the branch also fixed the issue.

Might be related to the problem observed in
<a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [regression][bisected] MPV playing video via opengl "randomly" results in only part of the window / screen being rendered with Mesa GIT."
   href="show_bug.cgi?id=101558">https://bugs.freedesktop.org/show_bug.cgi?id=101558</a> .

If you have any ideas, I'm open to help in further testing.</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>