From: Dan Carpenter Date: Sat, 29 Oct 2011 07:21:28 +0000 (+0300) Subject: drm/radeon/benchmark: signedness bug in radeon_benchmark_move() X-Git-Tag: v3.12-rc1~4381^2~21 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bfba16582600ab2c75dc39250a2b8f3b2a42da11;p=kernel%2Fkernel-generic.git 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 Signed-off-by: Dave Airlie --- diff --git a/drivers/gpu/drm/radeon/radeon_benchmark.c b/drivers/gpu/drm/radeon/radeon_benchmark.c index 5cafc90..17e1a9b 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);