Bug 77162

Summary: [IVB/HSW/BYT/BDW Bisected]Piglit spec_glsl-1.50_execution_geometry_point-size-out fails
Product: piglit Reporter: lu hua <huax.lu>
Component: testsAssignee: Jordan Justen <jljusten>
Status: VERIFIED FIXED QA Contact: Eric Anholt <eric>
Severity: normal    
Priority: high    
Version: unspecified   
Hardware: All   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:

Description lu hua 2014-04-08 02:55:00 UTC 
System Environment:
--------------------------
Platform: Ivybridge
Libdrm:		(master)libdrm-2.4.52-10-g1cb5fc706cf5913d9dee33c939f65a23ecea17a5
Mesa:		(master)159cec9dec2320d821d388c0df8140c7705b6e0b
Xserver:	(master)xorg-server-1.15.99.901-204-g901fbfbbbd71c0d82080957f8ba09eebbc786f2b
Xf86_video_intel:(master)2.99.911-44-g3310ee89c1f1a663de5f5b12b8125809a213996f
Cairo:		(master)bb17403622e9ea474017e0039af8ee48b2b285db
Libva:		(staging)960352aaf1e11891a533ce2c98b16edde24d6851
Libva_intel_driver:(staging)150f67c67bd92cd201b75a92388fe3a63b00cd8a
Kernel:	(drm-intel-nightly) 7cd8b823493acc320c7db9112de9d4e21c6dbd11

Bug detailed description:
-----------------------------
It fails on Ivybridge, Haswell, Baytrail and Broadwell with Mesa master branch, and works well on Mesa 10.1 branch.
Following cases also fail and have same bisect commit:
spec_glsl-1.50_execution_geometry_clip-distance-out-values
spec_glsl-1.50_execution_geometry_max-input-components
spec_glsl-1.50_execution_geometry_primitive-id-out
spec_glsl-1.50_execution_gs-redeclares-both-pervertex-blocks
spec_glsl-1.50_execution_interface-vs-named-to-gs-array
spec_glsl-1.50_execution_redeclare-pervertex-out-subset-gs
spec_glsl-1.50_execution_redeclare-pervertex-subset-vs-to-gs

Bisect shows: c20596b85f666de470648decbac20b7643a50570 is the first bad commit
commit c20596b85f666de470648decbac20b7643a50570
Author:     Jordan Justen <jordan.l.justen@intel.com>
AuthorDate: Thu Apr 3 10:22:21 2014 -0700
Commit:     Jordan Justen <jordan.l.justen@intel.com>
CommitDate: Thu Apr 3 10:37:12 2014 -0700

    Revert "shader_runner: always enable VAO for gl >= 31"

    This reverts commit b28cdc887aefb1e766719e1f74d457f77bdf9a18.

    b28cdc88 causes many tests to crash on AMD and NVIDIA binary
    drivers.

    One known example is tests/shaders/glsl-fs-any.shader_test

    Reported-by: "Fu, Ping" <Ping.Fu@amd.com>
    Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>

output:
Probe color at (185,125) 
Expected: 1.000000 1.000000 1.000000 1.000000 
Observed: 0.000000 0.000000 0.000000 0.000000 
Probe color at (190,125) 
Expected: 1.000000 1.000000 1.000000 1.000000 
Observed: 0.000000 0.000000 0.000000 0.000000 
PIGLIT: {'result': 'fail' } 


Reproduce steps:
---------------------------- 
1. xinit
2. ./bin/shader_runner /GFX/Test/Piglit/piglit/tests/spec/glsl-1.50/execution/geometry/point-size-out.shader_test -auto
Comment 1 lu hua 2014-05-07 01:16:41 UTC 
It also happens on mesa 10.2 branch.
Comment 2 Jordan Justen 2014-05-14 07:48:59 UTC 
Should be fixed by
"shader_runner: enable VAO for gl >= 31 before glDrawArrays"
which is on the piglit list.
Comment 3 Jordan Justen 2014-05-16 21:13:36 UTC 
This should be fixed with piglit commit
b8de17e99fdc0edcd40a437d478ba56322b6c323
Comment 4 lu hua 2014-05-19 02:23:32 UTC 
Verified.Fixed.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.