drm/radeon/benchmark: signedness bug in radeon_benchmark_move()
authorDan Carpenter <dan.carpenter@oracle.com>
Sat, 29 Oct 2011 07:21:28 +0000 (10:21 +0300)
committerDave Airlie <airlied@redhat.com>
Fri, 11 Nov 2011 11:16:12 +0000 (11:16 +0000)
radeon_benchmark_do_move() returns an int so "time" should be int
too.  Making it unsigned breaks the error handling.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/radeon/radeon_benchmark.c

index 5cafc90de7f8cec757fa5695db9865ccc621cabc..17e1a9b2d8fbf3e1dedce2923a7e5b726aea284c 100644 (file)
@@ -98,7 +98,7 @@ static void radeon_benchmark_move(struct radeon_device *rdev, unsigned size,
        struct radeon_bo *sobj = NULL;
        uint64_t saddr, daddr;
        int r, n;
-       unsigned int time;
+       int time;
 
        n = RADEON_BENCHMARK_ITERATIONS;
        r = radeon_bo_create(rdev, size, PAGE_SIZE, true, sdomain, &sobj);