Hey, Op 05-08-15 om 15:08 schreef Daniel Vetter: > We want to make sure that no one tries to acquire more locks and > states, and ww mutexes provide debug facilities for that. So use them. > > Cc: Rob Clark <robdcl...@gmail.com> > Cc: Maarten Lankhorst <maarten.lankho...@linux.intel.com> > Signed-off-by: Daniel Vetter <daniel.vet...@intel.com> > --- > drivers/gpu/drm/drm_atomic.c | 2 ++ > 1 file changed, 2 insertions(+) I like the idea, played with the thought myself, but I think it might need to be slightly less strict for transitional drivers.
~Maarten > diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c > index 422183e7ee7d..5ea609f270ea 100644 > --- a/drivers/gpu/drm/drm_atomic.c > +++ b/drivers/gpu/drm/drm_atomic.c > @@ -1268,6 +1268,8 @@ int drm_atomic_check_only(struct drm_atomic_state > *state) > } > } > > + ww_acquire_done(&state->acquire_ctx->ww_ctx); > + > return ret; > } > EXPORT_SYMBOL(drm_atomic_check_only); _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
