Mesa (master): i965: Fix Sandybridge regression introduced by workaround-free math.

Kenneth Graunke kwg at kemper.freedesktop.org
Mon Nov 7 20:19:20 UTC 2011


Module: Mesa
Branch: master
Commit: dceb202297b39220fbbcb41267077fb3ff8d137a
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=dceb202297b39220fbbcb41267077fb3ff8d137a

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Mon Nov  7 12:07:44 2011 -0800

i965: Fix Sandybridge regression introduced by workaround-free math.

Commit a73c65c5342bf41fa0dfefe7daa9197ce6a11db4 had a typo which
accidentally enabled the workaround-free Gen7 code on Gen6.

Fixes GPU hangs in anything using pow() or integer division/modulus.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/i965/brw_fs_emit.cpp |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp b/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
index b613fb0..ad4fae2 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
@@ -833,7 +833,7 @@ fs_visitor::generate_code()
       case SHADER_OPCODE_INT_QUOTIENT:
       case SHADER_OPCODE_INT_REMAINDER:
       case SHADER_OPCODE_POW:
-	 if (intel->gen >= 6) {
+	 if (intel->gen >= 7) {
 	    generate_math2_gen7(inst, dst, src[0], src[1]);
 	 } else if (intel->gen == 6) {
 	    generate_math2_gen6(inst, dst, src[0], src[1]);




More information about the mesa-commit mailing list