Bug 14799

Summary: [i915 sRGB texture] sRGB texture enabled but not actually working
Product: Mesa Reporter: Shuang He <shuang.he>
Component: Drivers/DRI/i915Assignee: Kristian Høgsberg <krh>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: high CC: dri-devel
Version: unspecified   
Hardware: Other   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments: test case

Description Shuang He 2008-03-03 22:29:22 UTC 
Created attachment 14798 [details]
test case

System Environment:
--------------------------

--Platform: i915

--Architecture(32-bit,64-bit,compatiblity): all

--2D driver:
commit 4a42b01f5ee5a673716d6959dfe0e693b037eb48 

--3D driver:
commit dd1d66fc4ab5d7064113a2017a431c3461598b91

--Xserver:
commit 06c968acbf6771d5fed4b12ebd400fea791ea498 

--Drm
commit d6098db1409e8ee45052920d3acdd3b6f2cb80aa

--Kernel:
2.6.23.1


Bug detailed description:
-------------------------
This issue make glean test case occluQry failed on i915.
OpenGL GL_EXT_texture_sRGB seems enabled by following commit, but it's
actually not working:

commit dd1d66fc4ab5d7064113a2017a431c3461598b91
Author: Kristian Høgsberg <krh@redhat.com>
Date:   Fri Feb 22 18:06:19 2008 -0500

    intel: Merge intel_context.c from i915 and i965.



Reproduce steps:
----------------
1. start X
2. compile and run attached test case


Current result:
----------------
OpenGL GL_EXT_texture_sRGB seems enabled by commit 
but it's actually not working


Expected result:
----------------
GL_EXT_texture_sRGB should work when it's enabled
Comment 1 Kristian Høgsberg 2008-03-07 10:48:42 UTC 
Just committed this:

commit 1e6943cf5592186c3721dfefbdcf1a519a79bd8f
Author: Kristian Høgsberg <krh@redhat.com>
Date:   Fri Mar 7 13:45:09 2008 -0500

    [intel] Only enable GL_EXT_texture_sRGB on i965.
    
    Fixes #14799.

Should fix this issue.  There may be similar problems with GL_NV_point_sprite, GL_ARB_texture_non_power_of_two, GL_NV_texture_rectangle, GL_EXT_texture_rectangle, GL_ARB_point_sprite, GL_ARB_point_parameters, and GL_EXT_blend_logic_op, though I do think i915 supports those.
Comment 2 Shuang He 2008-03-07 21:05:05 UTC 
it's fixed
Comment 3 Shuang He 2008-03-07 21:05:23 UTC 
verified, thanks, krh

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.