| From 89cd67b326fa95872cc2b4524cd807128db6071d Mon Sep 17 00:00:00 2001 |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| Date: Mon, 1 Jul 2013 19:39:34 +0300 |
| Subject: drm/radeon: forever loop on error in radeon_do_test_moves() |
| |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| |
| commit 89cd67b326fa95872cc2b4524cd807128db6071d upstream. |
| |
| The error path does this: |
| |
| for (--i; i >= 0; --i) { |
| |
| which is a forever loop because "i" is unsigned. |
| |
| Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> |
| Signed-off-by: Alex Deucher <alexander.deucher@amd.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/gpu/drm/radeon/radeon_test.c | 4 ++-- |
| 1 file changed, 2 insertions(+), 2 deletions(-) |
| |
| --- a/drivers/gpu/drm/radeon/radeon_test.c |
| +++ b/drivers/gpu/drm/radeon/radeon_test.c |
| @@ -37,8 +37,8 @@ static void radeon_do_test_moves(struct |
| struct radeon_bo **gtt_obj = NULL; |
| struct radeon_fence *fence = NULL; |
| uint64_t gtt_addr, vram_addr; |
| - unsigned i, n, size; |
| - int r, ring; |
| + unsigned n, size; |
| + int i, r, ring; |
| |
| switch (flag) { |
| case RADEON_TEST_COPY_DMA: |
