On Tue, Nov 27, 2018 at 05:46:39PM -0500, Sean Paul wrote: > From: Sean Paul <seanpaul@xxxxxxxxxxxx> > > drm_atomic_state_put doesn't require any locking, and this makes things > easier for switching to modeset_lock_all helpers in a future patch > > Cc: Daniel Vetter <daniel@xxxxxxxx> > Signed-off-by: Sean Paul <seanpaul@xxxxxxxxxxxx> > --- > drivers/gpu/drm/drm_atomic_helper.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c > index fe8dd8aa4ae40..15a75b9f185fa 100644 > --- a/drivers/gpu/drm/drm_atomic_helper.c > +++ b/drivers/gpu/drm/drm_atomic_helper.c > @@ -3286,9 +3286,10 @@ int drm_atomic_helper_resume(struct drm_device *dev, > drm_modeset_backoff(&ctx); > } > > - drm_atomic_state_put(state); > + state->acquire_ctx = NULL; If you want to clear this I think this should be done in drm_atomic_helper_commit_duplicated_state(), for symmetry reasons. And in a separate patch. Just moving the _put looks fine and has my r-b. -Daniel > drm_modeset_drop_locks(&ctx); > drm_modeset_acquire_fini(&ctx); > + drm_atomic_state_put(state); > > return err; > } > -- > Sean Paul, Software Engineer, Google / Chromium OS > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel