On Sat, Sep 14, 2013 at 03:03:18PM -0700, Ben Widawsky wrote: > @@ -1117,8 +1109,11 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data, > * batch" bit. Hence we need to pin secure batches into the global gtt. > * hsw should have this fixed, but let's be paranoid and do it > * unconditionally for now. */ > - if (flags & I915_DISPATCH_SECURE && !batch_obj->has_global_gtt_mapping) > - i915_gem_gtt_bind_object(batch_obj, batch_obj->cache_level); > + if (flags & I915_DISPATCH_SECURE && > + !batch_obj->has_global_gtt_mapping) { > + struct i915_vma *vma = i915_gem_obj_to_vma(batch_obj, vm); > + vm->bind_vma(vma, batch_obj->cache_level, GLOBAL_BIND); > + } This should be ggtt rather than vm? -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx