Re: [PATCH] drm/radeon: avoid leaking const ib (not used yet on si and newer GPU)

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

 



On Wed, May 2, 2012 at 4:24 PM,  <j.glisse@xxxxxxxxx> wrote:
> From: Jerome Glisse <jglisse@xxxxxxxxxx>
>
> Signed-off-by: Jerome Glisse <jglisse@xxxxxxxxxx>

Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

> ---
>  drivers/gpu/drm/radeon/radeon_cs.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_cs.c b/drivers/gpu/drm/radeon/radeon_cs.c
> index 82f2e7b0..b3800cb 100644
> --- a/drivers/gpu/drm/radeon/radeon_cs.c
> +++ b/drivers/gpu/drm/radeon/radeon_cs.c
> @@ -161,6 +161,8 @@ int radeon_cs_parser_init(struct radeon_cs_parser *p, void *data)
>        /* get chunks */
>        INIT_LIST_HEAD(&p->validated);
>        p->idx = 0;
> +       p->ib = NULL;
> +       p->const_ib = NULL;
>        p->chunk_ib_idx = -1;
>        p->chunk_relocs_idx = -1;
>        p->chunk_flags_idx = -1;
> @@ -318,6 +320,9 @@ static void radeon_cs_parser_fini(struct radeon_cs_parser *parser, int error)
>        kfree(parser->chunks);
>        kfree(parser->chunks_array);
>        radeon_ib_free(parser->rdev, &parser->ib);
> +       if (parser->const_ib) {
> +               radeon_ib_free(parser->rdev, &parser->const_ib);
> +       }
>  }
>
>  static int radeon_cs_ib_chunk(struct radeon_device *rdev,
> --
> 1.7.7.6
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
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