On Wed, 21 Aug 2019 at 16:59, Chris Wilson <[email protected]> wrote: > > Avoid calling i915_vma_put_fence() by using our alternate paths that > bind a secondary vma avoiding the original fenced vma. For the few > instances where we need to release the fence (i.e. on binding when the > GGTT range becomes invalid), replace the put_fence with a revoke_fence. > > Signed-off-by: Chris Wilson <[email protected]>
<snip> > @@ -557,20 +553,16 @@ i915_gem_gtt_pwrite_fast(struct drm_i915_gem_object > *obj, > wakeref = intel_runtime_pm_get(rpm); > } > > - vma = i915_gem_object_ggtt_pin(obj, NULL, 0, 0, > - PIN_MAPPABLE | > - PIN_NONBLOCK /* NOWARN */ | > - PIN_NOEVICT); > + vma = ERR_PTR(-ENODEV); > + if (i915_gem_object_is_tiled(obj)) > + vma = i915_gem_object_ggtt_pin(obj, NULL, 0, 0, Hmm? Reviewed-by: Matthew Auld <[email protected]> _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
