On 31.07.2012 16:47, Michel Dänzer wrote:
On Die, 2012-07-31 at 13:48 +0200, Christian König wrote:
Don't return success if scheduling the IB fails, otherwise we end up with an oops in ttm_eu_fence_buffer_objects.
Signed-off-by: Christian König deathsimple@vodafone.de Reviewed-by: Alex Deucher alexander.deucher@amd.com Cc: stable@vger.kernel.org
drivers/gpu/drm/radeon/radeon_cs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/radeon/radeon_cs.c b/drivers/gpu/drm/radeon/radeon_cs.c index 142f894..17238f4 100644 --- a/drivers/gpu/drm/radeon/radeon_cs.c +++ b/drivers/gpu/drm/radeon/radeon_cs.c @@ -377,7 +377,7 @@ static int radeon_cs_ib_chunk(struct radeon_device *rdev, if (r) { DRM_ERROR("Failed to schedule IB !\n"); }
- return 0;
return r; }
static int radeon_bo_vm_update_pte(struct radeon_cs_parser *parser,
AFAICT this fix is already in Linus' tree (commit 93bf888c5c730605e3470f5d2381f296eda88d79).
Your right, but why the heck isn't it in drm-fixes?
Christian.