[Bug 101704] [regression][bisected] glReadPixels() from pbuffer failing in Android CTS camera tests

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=101704

            Bug ID: 101704
           Summary: [regression][bisected] glReadPixels() from pbuffer
                    failing in Android CTS camera tests
           Product: Mesa
           Version: git
          Hardware: x86-64 (AMD64)
                OS: Linux (All)
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Drivers/DRI/i965
          Assignee: intel-3d-bugs at lists.freedesktop.org
          Reporter: tfiga at chromium.org
        QA Contact: intel-3d-bugs at lists.freedesktop.org

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

https://android.googlesource.com/platform/frameworks/base/+/master/core/java/android/hardware/camera2/legacy/SurfaceTextureRenderer.java#796

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

https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/framebuffer.c#n831
https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/main/readpix.c#n1078


Bisected to:

b7153c3e9f9d2d430b0338313587a00e531e4800 is the first bad commit
commit b7153c3e9f9d2d430b0338313587a00e531e4800
Author: Kenneth Graunke <kenneth at whitecape.org>
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
https://bugs.freedesktop.org/show_bug.cgi?id=101558 .

If you have any ideas, I'm open to help in further testing.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-3d-bugs/attachments/20170706/3a4bf1c1/attachment.html>