| Summary: | [SNB bisected]3 shad-interactions subcases failed on GT1, with VS threads increasing | ||
|---|---|---|---|
| Product: | Mesa | Reporter: | fangxun <xunx.fang> |
| Component: | Drivers/DRI/i965 | Assignee: | Zou Nan hai <nanhai.zou> |
| Status: | VERIFIED FIXED | QA Contact: | |
| Severity: | major | ||
| Priority: | high | CC: | idr, kenneth, nanhai.zou |
| Version: | git | ||
| Hardware: | All | ||
| OS: | Linux (All) | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
|
Description
fangxun
2011-03-27 18:40:11 UTC
According to Chris Wilson, max VS threads is only 24 on GT1. See: http://lists.freedesktop.org/archives/intel-gfx/2011-March/009828.html. If that is true, and it seems believable, the fix is pretty clear. Still failed with the patch. I have reverted the patch from 7.10. I'm leaving the bug open because it still exists on master. This needs a real fix ASAP. commit 50dccfdbefcf0a5c35668287508916a3a74da4c8 Author: Ian Romanick <ian.d.romanick@intel.com> Date: Tue Apr 5 16:14:39 2011 -0700 Revert "i965: bump VS thread number to 60 on SNB" Increasing the number of VS threads beyond 1 causes some regressions in vertex shader tests on Sugar Bay GT1 systems. This reverts commit c21a44463a402a886bf58d0e689d5a098eda1ff4. References: https://bugs.freedesktop.org/show_bug.cgi?id=35730 This should be fixed on master by: commit a019dd0d6e5bba00e8ee7818e004ee42ca507102 Author: Kenneth Graunke <kenneth@whitecape.org> Date: Sun Apr 3 00:57:30 2011 -0700 i965: Fix null register use in Sandybridge implied move resolution. Fixes regressions caused by commit 9a21bc6401, namely GPU hangs when running gnome-shell or compiz (Mesa bugs #35820 and #35853). I incorrectly refactored the case that dealt with ARF_NULL; even in that case, the source register needs to be changed to the MRF. NOTE: This is a candidate for the 7.10 branch (if 9a21bc6401 is cherry-picked, take this one too). commit 9d60a7ce08a67eb8b79c60f829d090ba4a37ed7e Author: Kenneth Graunke <kenneth@whitecape.org> Date: Tue Mar 29 05:49:40 2011 -0700 i965: Resolve implied moves in brw_dp_READ_4_vs_relative. Fixes piglit test glsl-vs-arrays-3 on Sandybridge, as well as garbage rendering in 3DMarkMobileES 2.0's Taiji demo and GLBenchmark 2.0's Egypt and PRO demos. NOTE: This a candidate for stable release branches. It depends on commit 9a21bc640188e4078075b9f8e6701853a4f0bbe4. commit 9a21bc640188e4078075b9f8e6701853a4f0bbe4 Author: Kenneth Graunke <kenneth@whitecape.org> Date: Wed Mar 16 14:09:17 2011 -0700 i965: Refactor Sandybridge implied move handling. This was open-coded in three different places, and more are necessary. Extract this into a function so it can be reused. Unfortunately, not all variations were the same: in particular, one set compression control and checked that the source register was not ARF_NULL. This seemed like a good idea, so all cases now do so. Verified with mesa master commit: a26121f37530619610a78a5fbe5ef87e44047fda |
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.