Re: [PATCH] drm/radeon: Take IH ring into account for test size calculation.

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

 



2011/8/19 Michel Dänzer <michel@xxxxxxxxxxx>:
> From: Michel Dänzer <michel.daenzer@xxxxxxx>
>
> Signed-off-by: Michel Dänzer <michel.daenzer@xxxxxxx>

Should probably also take into account the additional CP rings we
allocate on cayman.

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

> ---
>  drivers/gpu/drm/radeon/radeon_test.c |   10 +++++++---
>  1 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_test.c b/drivers/gpu/drm/radeon/radeon_test.c
> index dee4a0c..1ebd0fe 100644
> --- a/drivers/gpu/drm/radeon/radeon_test.c
> +++ b/drivers/gpu/drm/radeon/radeon_test.c
> @@ -40,10 +40,14 @@ void radeon_test_moves(struct radeon_device *rdev)
>        size = 1024 * 1024;
>
>        /* Number of tests =
> -        * (Total GTT - IB pool - writeback page - ring buffer) / test size
> +        * (Total GTT - IB pool - writeback page - ring buffers) / test size
>         */
> -       n = ((u32)(rdev->mc.gtt_size - RADEON_IB_POOL_SIZE*64*1024 - RADEON_GPU_PAGE_SIZE -
> -            rdev->cp.ring_size)) / size;
> +       n = rdev->mc.gtt_size - RADEON_IB_POOL_SIZE*64*1024 - rdev->cp.ring_size;
> +       if (rdev->wb.wb_obj)
> +               n -= RADEON_GPU_PAGE_SIZE;
> +       if (rdev->ih.ring_obj)
> +               n -= rdev->ih.ring_size;
> +       n /= size;
>
>        gtt_obj = kzalloc(n * sizeof(*gtt_obj), GFP_KERNEL);
>        if (!gtt_obj) {
> --
> 1.7.5.4
>
>
> _______________________________________________
> 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