Re: [PATCH 1/2] drm/msm/atomic: Clean up planes in the error paths of .atomic_commit()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 27, 2015 at 7:39 AM, Laurent Pinchart
<laurent.pinchart@xxxxxxxxxxxxxxxx> wrote:
> When the .atomic_commit() handler fails, clean up planes previoulsy
> prepared by drm_atomic_helper_prepare_planes() with a call to
> drm_atomic_helper_cleanup_planes().

Thanks, I'll send that w/ my pull req for 4.2

BR,
-R

>
> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> ---
>  drivers/gpu/drm/msm/msm_atomic.c | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/msm_atomic.c b/drivers/gpu/drm/msm/msm_atomic.c
> index 5b192128cda2..55d4973c476a 100644
> --- a/drivers/gpu/drm/msm/msm_atomic.c
> +++ b/drivers/gpu/drm/msm/msm_atomic.c
> @@ -187,8 +187,10 @@ int msm_atomic_commit(struct drm_device *dev,
>                 return ret;
>
>         c = commit_init(state);
> -       if (!c)
> -               return -ENOMEM;
> +       if (!c) {
> +               ret = -ENOMEM;
> +               goto error;
> +       }
>
>         /*
>          * Figure out what crtcs we have:
> @@ -221,7 +223,7 @@ int msm_atomic_commit(struct drm_device *dev,
>         ret = start_atomic(dev->dev_private, c->crtc_mask);
>         if (ret) {
>                 kfree(c);
> -               return ret;
> +               goto error;
>         }
>
>         /*
> @@ -265,4 +267,8 @@ int msm_atomic_commit(struct drm_device *dev,
>         complete_commit(c);
>
>         return 0;
> +
> +error:
> +       drm_atomic_helper_cleanup_planes(dev, state);
> +       return ret;
>  }
> --
> Regards,
>
> Laurent Pinchart
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux