On Wed, Nov 28, 2018 at 09:35:47AM +0100, Daniel Vetter wrote: > 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. Ohh, that's a much better idea! > -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 -- Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
