Summary: | [SNB Bisected]Ogles2conform GL2ExtensionTests/depth_texture_cube_map/depth_texture_cube_map.test fail | ||
---|---|---|---|
Product: | Mesa | Reporter: | lu hua <huax.lu> |
Component: | Drivers/DRI/i965 | Assignee: | Paul Berry <stereotype441> |
Status: | VERIFIED FIXED | QA Contact: | |
Severity: | major | ||
Priority: | high | CC: | idr, xunx.fang |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: | output |
It fails on 9.1 branch with following commit(Bug 64662): commit 496996010561463bf0f2e2a7632978c548065b98 Author: Chad Versace <chad.versace@linux.intel.com> Date: Fri Apr 5 15:18:00 2013 -0700 intel: Allocate hiz in intel_renderbuffer_move_to_temp() When moving the renderbuffer to a new miptree, we neglected to allocate the hiz buffer for the new miptree. Oops. Fixes all Piglit depthstencil-render-miplevels tests from crash to pass on Sandybridge. Note: This is a candidate for the 9.1 branch. Reviewed-by: Eric Anholt <eric@anholt.net> Reviewed-by: Paul Berry <stereotype441@gmail.com> Signed-off-by: Chad Versace <chad.versace@linux.intel.com> (cherry picked from commit aa391976dfa12479185d9eeed1f2a0b4dce6c49b) Fixed by commit 46ea8041074df79561f9771e2ecf198f2cbd088f Author: Paul Berry <stereotype441@gmail.com> Date: Thu May 16 14:12:15 2013 -0700 intel: Do a depth resolve before copying images between miptrees. When intel_finalize_mipmap_tree() calls intel_miptree_copy_teximage() to reassemble a depth miptree that has been broken apart into pieces (to deal with misalignment of levels/layers within the miptree), it just copies the depth data, not the HiZ data. This is reasonable, since the alignment restrictions of HiZ are a large part of the reason why the miptree had to be broken apart in the first place. However, in order for the depth copy to be sufficient, we need to do a depth resolve first, to make sure any deferred depth writes that are in the HiZ buffer get performed. Fixes https://bugs.freedesktop.org/show_bug.cgi?id=64662 and https://bugs.freedesktop.org/show_bug.cgi?id=64659. NOTE: This is a candidate for stable release branches. Reviewed-by: Chad Versace <chad.versace@linux.intel.com> 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.
Created attachment 79374 [details] output System Environment: -------------------------- Arch: i386 Platform: Sandybridge Libdrm: (master)2.4.44-9-g96e90aabc4c0238de2f2d245899f991a3b996587 Mesa:(master)013424678e2604a45c420ec403e6f12b82446b24 Xserver:(master)xorg-server-1.14.99.1-82-g7e97166c4fc38076b6224e4dbdd632b43444149f Xf86_video_intel:(master)2.21.6-41-gf57a65c35268f215c17f1a02b3af50a6678ce3c1 Cairo:(master)631bf299256e11a17511977f357e0353fb5615f7 Libva: (staging)0139a11ba23f8b2aa0d2ed36336c4df23d54390c Libva_intel_driver:(staging)0a5cfdab47c788569feb0a3e4293906687523f88 Kernel: (drm-intel-nightly) 675fa342aa3a54af0a81f23809fff9590dfb6fc1 Bug detailed description: ------------------------- It fails on sandybridge with mesa master branch and mesa 9.1 branch. Bisect shows:d641a01d981f7de600ea50fa53c85e34fc2e3174 is the first bad commit. commit d641a01d981f7de600ea50fa53c85e34fc2e3174 Author: Eric Anholt Date: Mon May 6 14:12:56 2013 -0700 i965: Add support for Y-tiled blits on gen6+. Reviewed-by: Kenneth Graunke Reviewed-by: Ian Romanick Reproduce steps: ---------------- 1. xinit 2. ./GTF -width=64 -height=64 -run=GL2ExtensionTests/depth_texture_cube_map/depth_texture_cube_map.test