On Wed, May 31, 2017 at 01:33:55PM +0100, Lionel Landwerlin wrote: > static void > -free_oa_buffer(struct drm_i915_private *i915) > +free_sseu_buffer(struct drm_i915_private *i915) > { > - mutex_lock(&i915->drm.struct_mutex); > + i915_gem_object_unpin_map(i915->perf.sseu_buffer.vma->obj); > + i915_vma_unpin(i915->perf.sseu_buffer.vma); > + i915_gem_object_put(i915->perf.sseu_buffer.vma->obj); unpin followed by put is common enough that it is wrapped into i915_vam_unpin_and_release(). -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx