Re: [patch] drm/radeon: forever loop on error in radeon_do_test_moves()

2013-10-02 Thread Alex Deucher
On Mon, Jul 1, 2013 at 12:39 PM, Dan Carpenter wrote: > The error path does this: > > for (--i; i >= 0; --i) { > > which is a forever loop because "i" is unsigned. > > Signed-off-by: Dan Carpenter Sorry I missed this one. Applied. Thanks! Alex > > diff --git a/drivers/gpu/drm/radeon/

[patch] drm/radeon: forever loop on error in radeon_do_test_moves()

2013-07-01 Thread Dan Carpenter
The error path does this: for (--i; i >= 0; --i) { which is a forever loop because "i" is unsigned. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/radeon/radeon_test.c b/drivers/gpu/drm/radeon/radeon_test.c index f4d6bce..12e8099 100644 --- a/drivers/gpu/drm/radeon/radeon_t

[patch] drm/radeon: forever loop on error in radeon_do_test_moves()

2013-07-01 Thread Dan Carpenter
The error path does this: for (--i; i >= 0; --i) { which is a forever loop because "i" is unsigned. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/radeon/radeon_test.c b/drivers/gpu/drm/radeon/radeon_test.c index f4d6bce..12e8099 100644 --- a/drivers/gpu/drm/radeon/radeon_t