Hey, Op 25-06-15 om 11:25 schreef Liu, Hong: > atomic commit only frees the state if no error met, caller needs to > free the allocated state if atomic commit failed. > > Signed-off-by: Hong Liu <hong.liu@xxxxxxxxx> > --- > drivers/gpu/drm/drm_atomic.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c > index 6e3b78e..aa8ad4c 100644 > --- a/drivers/gpu/drm/drm_atomic.c > +++ b/drivers/gpu/drm/drm_atomic.c > @@ -1344,6 +1344,9 @@ retry: > } > } > > + if (ret && !(arg->flags & DRM_MODE_ATOMIC_TEST_ONLY)) > + goto fail; > + > drm_modeset_drop_locks(&ctx); > drm_modeset_acquire_fini(&ctx); "[PATCH 2/2] drm/atomic: Cleanup on error properly in the atomic ioctl." fixes it too. ~Maarten _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
