projects
/
profile
/
ivi
/
kernel-x86-ivi.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
8f4ff2b
)
drm/radeon/benchmark: signedness bug in radeon_benchmark_move()
author
Dan Carpenter
<dan.carpenter@oracle.com>
Sat, 29 Oct 2011 07:21:28 +0000
(10:21 +0300)
committer
Dave 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
patch
|
blob
|
history
diff --git
a/drivers/gpu/drm/radeon/radeon_benchmark.c
b/drivers/gpu/drm/radeon/radeon_benchmark.c
index
5cafc90
..
17e1a9b
100644
(file)
--- 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);