diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-10-29 11:21:28 +0400 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2011-11-11 15:16:12 +0400 |
| commit | bfba16582600ab2c75dc39250a2b8f3b2a42da11 (patch) | |
| tree | f1043efacdf8e96ed6d965d1d1012d897487ad01 | |
| parent | 8f4ff2b06afcd6f151868474a432c603057eaf56 (diff) | |
| download | linux-bfba16582600ab2c75dc39250a2b8f3b2a42da11.tar.xz | |
drm/radeon/benchmark: signedness bug in radeon_benchmark_move()
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>
| -rw-r--r-- | drivers/gpu/drm/radeon/radeon_benchmark.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_benchmark.c b/drivers/gpu/drm/radeon/radeon_benchmark.c index 5cafc90de7f8..17e1a9b2d8fb 100644 --- a/drivers/gpu/drm/radeon/radeon_benchmark.c +++ b/drivers/gpu/drm/radeon/radeon_benchmark.c @@ -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); |