drm/i915: Enable cmd parser to do secure batch promotion for aliasing ppgtt
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 14 Apr 2015 15:35:22 +0000 (17:35 +0200)
committerJani Nikula <jani.nikula@intel.com>
Thu, 30 Apr 2015 10:29:56 +0000 (13:29 +0300)
With the binding regression from the original full ppgtt patches
fixed we can throw the switch. Yay!

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90190
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
[Jani: tweaked commit title per Chris' suggestion]
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/i915_gem_execbuffer.c

index cfdc8c6073aa87501cc3a8f24be569554e8c5799..d2e21c549756acea95540ffeb2b4cfc022c22009 100644 (file)
@@ -1558,12 +1558,8 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
                 * dispatch_execbuffer implementations. We specifically
                 * don't want that set when the command parser is
                 * enabled.
-                *
-                * FIXME: with aliasing ppgtt, buffers that should only
-                * be in ggtt still end up in the aliasing ppgtt. remove
-                * this check when that is fixed.
                 */
-               if (USES_FULL_PPGTT(dev))
+               if (USES_PPGTT(dev))
                        dispatch_flags |= I915_DISPATCH_SECURE;
 
                exec_start = 0;