Quoting Animesh Manna (2019-08-21 07:32:31) > Freed the gem object after completion of dsb workload. > > Cc: Shashank Sharma <shashank.sharma@xxxxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > Signed-off-by: Animesh Manna <animesh.manna@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_dsb.c | 23 +++++++++++++++++++++++ > drivers/gpu/drm/i915/display/intel_dsb.h | 1 + > 2 files changed, 24 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_dsb.c b/drivers/gpu/drm/i915/display/intel_dsb.c > index 7e0a9b37f702..bfb138952f61 100644 > --- a/drivers/gpu/drm/i915/display/intel_dsb.c > +++ b/drivers/gpu/drm/i915/display/intel_dsb.c > @@ -234,3 +234,26 @@ void intel_dsb_commit(struct intel_dsb *dsb) > dsb->free_pos = 0; > intel_dsb_disable_engine(dsb); > } > + > +void intel_dsb_put(struct intel_dsb *dsb) > +{ > + struct intel_crtc *crtc; > + struct drm_i915_private *i915; > + struct i915_vma *vma; > + > + if (!dsb) > + return; > + > + crtc = dsb->crtc; > + i915 = to_i915(crtc->base.dev); > + > + if (dsb->cmd_buf) { > + vma = dsb->vma; > + mutex_lock(&i915->drm.struct_mutex); > + crtc->dsb_in_use--; That is complete garbage. This dsb just happens to be crtc->dsb_in_use? > + i915_gem_object_unpin_map(vma->obj); > + i915_vma_unpin_and_release(&vma, 0); i915_vma_unpin_and_release(vma, I915_VMA_RELEASE_MAP); Does not require struct_mutex. Put this with the allocator. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx