[PATCH] drm/amdgpu: record error code when ring test failed

Christian König deathsimple at vodafone.de
Tue Aug 30 15:04:23 UTC 2016


Am 30.08.2016 um 16:24 schrieb Deucher, Alexander:
>> -----Original Message-----
>> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
>> Of Chunming Zhou
>> Sent: Tuesday, August 30, 2016 5:59 AM
>> To: amd-gfx at lists.freedesktop.org
>> Cc: Zhou, David(ChunMing)
>> Subject: [PATCH] drm/amdgpu: record error code when ring test failed
>>
>> Change-Id: I3a59f602a4d5ec42c8c184daa14eb8194b0dab9e
>> Signed-off-by: Chunming Zhou <David1.Zhou at amd.com>
>> ---
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 5 +++--
>>   1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
>> index f5810f7..8c17888 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
>> @@ -280,7 +280,7 @@ void amdgpu_ib_pool_fini(struct amdgpu_device
>> *adev)
>>   int amdgpu_ib_ring_tests(struct amdgpu_device *adev)
>>   {
>>   	unsigned i;
>> -	int r;
>> +	int r, ret = 0;
>>
>>   	for (i = 0; i < AMDGPU_MAX_RINGS; ++i) {
>>   		struct amdgpu_ring *ring = adev->rings[i];
>> @@ -301,10 +301,11 @@ int amdgpu_ib_ring_tests(struct amdgpu_device
>> *adev)
>>   			} else {
>>   				/* still not good, but we can live with it */
>>   				DRM_ERROR("amdgpu: failed testing IB on
>> ring %d (%d).\n", i, r);
>> +				ret = r;
> Hmm, I think that was intentional so as not to fail completely even if some of the engines aren't working.

Yeah, I've had the same concern so I double checked it. The driver just 
prints an additional error message and continuous with the startup.

In general I think it makes sense to return an error here, cause then we 
can easily identify cases where we need to fallback to a full engine reset.

Christian.

> Alex
>
>>   			}
>>   		}
>>   	}
>> -	return 0;
>> +	return ret;
>>   }
>>
>>   /*
>> --
>> 1.9.1
>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx




More information about the amd-gfx mailing list