drm/radeon/kms: remove benchmarks shorter than one page
authorIlija Hadzic <ihadzic@research.bell-labs.com>
Tue, 31 Jan 2012 14:35:25 +0000 (09:35 -0500)
committerDave Airlie <airlied@redhat.com>
Fri, 3 Feb 2012 09:32:30 +0000 (09:32 +0000)
copy_blit operation works only on integral number of pages
so benchmarks shorter than one page size (4K) do not make sense

v2: use RADEON_GPU_PAGE_SIZE instead of "magic" 1024 number and
    sweep sizes between 1 * <page_size> to 16K * <page_size> doubling
    the size in each iteration; we get the same coverage, as
    in the original benchmark, but guarantee integer multiples
    of page size

v3: add whitespace between '*' operator per review received from
    zajec5@gmail.com

Signed-off-by: Ilija Hadzic <ihadzic@research.bell-labs.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/radeon/radeon_benchmark.c

index 815f234..58cee89 100644 (file)
@@ -208,22 +208,22 @@ void radeon_benchmark(struct radeon_device *rdev, int test_number)
                break;
        case 3:
                /* GTT to VRAM, buffer size sweep, powers of 2 */
-               for (i = 1; i <= 65536; i <<= 1)
-                       radeon_benchmark_move(rdev, i*1024,
+               for (i = 1; i <= 16384; i <<= 1)
+                       radeon_benchmark_move(rdev, i * RADEON_GPU_PAGE_SIZE,
                                              RADEON_GEM_DOMAIN_GTT,
                                              RADEON_GEM_DOMAIN_VRAM);
                break;
        case 4:
                /* VRAM to GTT, buffer size sweep, powers of 2 */
-               for (i = 1; i <= 65536; i <<= 1)
-                       radeon_benchmark_move(rdev, i*1024,
+               for (i = 1; i <= 16384; i <<= 1)
+                       radeon_benchmark_move(rdev, i * RADEON_GPU_PAGE_SIZE,
                                              RADEON_GEM_DOMAIN_VRAM,
                                              RADEON_GEM_DOMAIN_GTT);
                break;
        case 5:
                /* VRAM to VRAM, buffer size sweep, powers of 2 */
-               for (i = 1; i <= 65536; i <<= 1)
-                       radeon_benchmark_move(rdev, i*1024,
+               for (i = 1; i <= 16384; i <<= 1)
+                       radeon_benchmark_move(rdev, i * RADEON_GPU_PAGE_SIZE,
                                              RADEON_GEM_DOMAIN_VRAM,
                                              RADEON_GEM_DOMAIN_VRAM);
                break;