diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-10-29 10:21:28 +0300 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2011-11-11 11:16:12 +0000 |
| commit | bfba16582600ab2c75dc39250a2b8f3b2a42da11 (patch) | |
| tree | f1043efacdf8e96ed6d965d1d1012d897487ad01 /drivers/gpu | |
| parent | 8f4ff2b06afcd6f151868474a432c603057eaf56 (diff) | |
| download | linux-bfba16582600ab2c75dc39250a2b8f3b2a42da11.tar.bz2 | |
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>
Diffstat (limited to 'drivers/gpu')
| -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); |