Bug 44962

Summary: [bisected i965] oglc depth-stencil(advanced.packed_depth_stencil_border) regressed
Product: Mesa Reporter: fangxun <xunx.fang>
Component: Drivers/DRI/i965Assignee: Kenneth Graunke <kenneth>
Status: CLOSED WONTFIX QA Contact:
Severity: major    
Priority: high CC: eric
Version: git   
Hardware: All   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:

Description fangxun 2012-01-19 20:04:33 UTC 
System Environment:
--------------------------
Arch:           i386
Platform:       Sugarbay
Libdrm:         (master)2.4.30-1-g66518ab5653cfdc840cd69e7b653ec05df060584
Mesa:           (8.0)c85402aba91755808729fadf57a9f92285f4e61a
Xserver:                (server-1.11-branch)xorg-server-1.11.3
Xf86_video_intel:(master)2.17.0-478-g35f81005f91d294e61bb4ced7cbddd1a76ccb324
Kernel:         (drm-intel-fixes) 8109021313c7a3d8947677391ce6ab9cd0bb1d28

Bug detailed description:
------------------------- 
It failed on Ironlake and Sandybridge. 
Bisect shows 96db07e28c165610b809a9fd88410a82a24a893e is the first bad commit.
commit 96db07e28c165610b809a9fd88410a82a24a893e
Author:     Eric Anholt <eric@anholt.net>
AuthorDate: Mon Oct 17 14:34:52 2011 -0700
Commit:     Eric Anholt <eric@anholt.net>
CommitDate: Wed Oct 26 12:42:23 2011 -0700

    intel: Enable stripping of texture borders.

    This replaces software rendering of textures with the deprecated
    1-pixel border (which is always bad, since mipmapping is rather broken
    in swrast, and GLSL 1.30 is unsupported) with hardware rendering that
    just pretends there was never a border (so you have potential seams on
    apps that actually intentionally used the 1-pixel borders, but correct
    rendering otherwise).

    This doesn't regress any piglit tests on gen6 (since the texwrap
    border/bordercolor cases already failed due to broken border color
    handling), but regresses texwrap border cases on original gen4 since
    those end up sampling the border color instead of the border pixels.
    It's a small price to pay for not thinking about texture borders any
    more.

    Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
    Reviewed-by: Brian Paul <brianp@vmware.com>

Reproduce steps:
----------------
1. start X
2. ./oglconform z -s -suite all -v 2 -test depth-stencil advanced.packed_depth_stencil_border
Comment 1 Ian Romanick 2012-01-23 10:35:28 UTC 
This is either fixed by Ken's border color patch (posted to the mesa-dev list) or we don't care.
Comment 2 Ian Romanick 2012-02-01 08:55:00 UTC 
Ken's patches don't seem to have fixed the test.  The output of the test says:

"Test creates a 2D texture 64x64 texels with non zero border. Texture is loaded to the driver with packed depth stencil internal format and defined base format: DEPTH_STENCIL_EXT.  Texture is placed on the polygon. After rendering image is compared with expected texture.  Border color should be the same as specified in texture environment settings"

Based on that, I don't think we care about this regression, and I'm removing it from the release tracker.

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.