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

Alex Deucher alexdeucher at gmail.com
Wed May 2 13:30:44 PDT 2012


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

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  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 at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel


More information about the dri-devel mailing list