drm/i915: Remove a bad BUG_ON in the fence management code.
authorEric Anholt <eric@anholt.net>
Thu, 4 Jun 2009 11:18:14 +0000 (11:18 +0000)
committerEric Anholt <eric@anholt.net>
Thu, 4 Jun 2009 11:43:09 +0000 (11:43 +0000)
This could be triggered by a gtt mapping fault on 965 that decides to
remove the fence from another object that happens to be active currently.
Since the other object doesn't rely on the fence reg for its execution, we
don't wait for it to finish.  We'll soon be not waiting on 915 most of the
time as well, so just drop the BUG_ON.

Signed-off-by: Eric Anholt <eric@anholt.net>
drivers/gpu/drm/i915/i915_gem.c

index 670d12881468297e8e0af0fe40222c3f54f5cb2f..39f5c658ef5e7c9754444672e86c166cd12a0191 100644 (file)
@@ -2260,9 +2260,6 @@ try_again:
                        goto try_again;
                }
 
-               BUG_ON(old_obj_priv->active ||
-                      (reg->obj->write_domain & I915_GEM_GPU_DOMAINS));
-
                /*
                 * Zap this virtual mapping so we can set up a fence again
                 * for this object next time we need it.