Re: [PATCH] drm/radeon: Explicitly print GTT/VRAM offsets on test failure.

[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>
>
> Otherwise these would need to be painstakingly calculated looking at the source
> code.
>
> Signed-off-by: Michel Dänzer <michel.daenzer@xxxxxxx>

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

> ---
>
> NOTE: Only compile tested.
>
>  drivers/gpu/drm/radeon/radeon_test.c |   24 ++++++++++++++++++------
>  1 files changed, 18 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_test.c b/drivers/gpu/drm/radeon/radeon_test.c
> index 1ebd0fe..602fa35 100644
> --- a/drivers/gpu/drm/radeon/radeon_test.c
> +++ b/drivers/gpu/drm/radeon/radeon_test.c
> @@ -136,9 +136,15 @@ void radeon_test_moves(struct radeon_device *rdev)
>                     gtt_start++, vram_start++) {
>                        if (*vram_start != gtt_start) {
>                                DRM_ERROR("Incorrect GTT->VRAM copy %d: Got 0x%p, "
> -                                         "expected 0x%p (GTT map 0x%p-0x%p)\n",
> -                                         i, *vram_start, gtt_start, gtt_map,
> -                                         gtt_end);
> +                                         "expected 0x%p (GTT/VRAM offset "
> +                                         "0x%16llx/0x%16llx)\n",
> +                                         i, *vram_start, gtt_start,
> +                                         (unsigned long long)
> +                                         (gtt_addr - rdev->mc.gtt_start +
> +                                          (void*)gtt_start - gtt_map),
> +                                         (unsigned long long)
> +                                         (vram_addr - rdev->mc.vram_start +
> +                                          (void*)gtt_start - gtt_map));
>                                radeon_bo_kunmap(vram_obj);
>                                goto out_cleanup;
>                        }
> @@ -179,9 +185,15 @@ void radeon_test_moves(struct radeon_device *rdev)
>                     gtt_start++, vram_start++) {
>                        if (*gtt_start != vram_start) {
>                                DRM_ERROR("Incorrect VRAM->GTT copy %d: Got 0x%p, "
> -                                         "expected 0x%p (VRAM map 0x%p-0x%p)\n",
> -                                         i, *gtt_start, vram_start, vram_map,
> -                                         vram_end);
> +                                         "expected 0x%p (VRAM/GTT offset "
> +                                         "0x%16llx/0x%16llx)\n",
> +                                         i, *gtt_start, vram_start,
> +                                         (unsigned long long)
> +                                         (vram_addr - rdev->mc.vram_start +
> +                                          (void*)vram_start - vram_map),
> +                                         (unsigned long long)
> +                                         (gtt_addr - rdev->mc.gtt_start +
> +                                          (void*)vram_start - vram_map));
>                                radeon_bo_kunmap(gtt_obj[i]);
>                                goto out_cleanup;
>                        }
> --
> 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