Re: [PATCH 1/2] drm/radeon: stop the leaks in cik_ib_test

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

 



On Mon, Oct 14, 2013 at 5:32 AM, Christian König
<deathsimple@xxxxxxxxxxx> wrote:
> From: Christian König <christian.koenig@xxxxxxx>
>
> Stop leaking IB memory and scratch register space when the test fails.
>
> Signed-off-by: Christian König <christian.koenig@xxxxxxx>

Both patches applied.

Thanks!

Alex

> ---
>  drivers/gpu/drm/radeon/cik.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/radeon/cik.c b/drivers/gpu/drm/radeon/cik.c
> index b874ccd..8f393df 100644
> --- a/drivers/gpu/drm/radeon/cik.c
> +++ b/drivers/gpu/drm/radeon/cik.c
> @@ -3182,6 +3182,7 @@ int cik_ib_test(struct radeon_device *rdev, struct radeon_ring *ring)
>         r = radeon_ib_get(rdev, ring->idx, &ib, NULL, 256);
>         if (r) {
>                 DRM_ERROR("radeon: failed to get ib (%d).\n", r);
> +               radeon_scratch_free(rdev, scratch);
>                 return r;
>         }
>         ib.ptr[0] = PACKET3(PACKET3_SET_UCONFIG_REG, 1);
> @@ -3198,6 +3199,8 @@ int cik_ib_test(struct radeon_device *rdev, struct radeon_ring *ring)
>         r = radeon_fence_wait(ib.fence, false);
>         if (r) {
>                 DRM_ERROR("radeon: fence wait failed (%d).\n", r);
> +               radeon_scratch_free(rdev, scratch);
> +               radeon_ib_free(rdev, &ib);
>                 return r;
>         }
>         for (i = 0; i < rdev->usec_timeout; i++) {
> --
> 1.8.1.2
>
_______________________________________________
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