Dear Alex,

Am Mittwoch, den 13.03.2013, 12:38 -0400 schrieb alexdeuc...@gmail.com:
> From: Alex Deucher <alexander.deuc...@amd.com>
> 
> Remove old comment and allow benchmarking moves within the
> same memory domain for both dma and blit methods.
>
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
> ---
>  drivers/gpu/drm/radeon/radeon_benchmark.c |    5 +----
>  1 files changed, 1 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/radeon/radeon_benchmark.c 
> b/drivers/gpu/drm/radeon/radeon_benchmark.c
> index a2f0c24..6e05a2e 100644
> --- a/drivers/gpu/drm/radeon/radeon_benchmark.c
> +++ b/drivers/gpu/drm/radeon/radeon_benchmark.c
> @@ -122,10 +122,7 @@ static void radeon_benchmark_move(struct radeon_device 
> *rdev, unsigned size,
>               goto out_cleanup;
>       }
>  
> -     /* r100 doesn't have dma engine so skip the test */
> -     /* also, VRAM-to-VRAM test doesn't make much sense for DMA */
> -     /* skip it as well if domains are the same */
> -     if ((rdev->asic->copy.dma) && (sdomain != ddomain)) {
> +     if (rdev->asic->copy.dma) {

why is the comment not valid anymore? What changed?

>               time = radeon_benchmark_do_move(rdev, size, saddr, daddr,
>                                               RADEON_BENCHMARK_COPY_DMA, n);
>               if (time < 0)


Thanks,

Paul

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to