[PATCH] drm/amdgpu: increase timeout of IB test

zhoucm1 david1.zhou at amd.com
Tue Jul 26 07:28:42 UTC 2016


CQE has found timeout when they cherry-pick your timeout patch.
I also found ib_test could be timeout after gpu reset.
1s maybe too long for a simple testing command, but I just think that 
doesn't matter to judge timeout.

Regards,
David

On 2016年07月26日 15:24, Christian König wrote:
> Am 26.07.2016 um 07:57 schrieb Chunming Zhou:
>> we should give enough time to IB test.
>>
>> Change-Id: I92bfbe9b3aa35083f41baed8663907abfa15c8e6
>> Signed-off-by: Chunming Zhou <David1.Zhou at amd.com>
>
> Do we really need more than 10ms for an IB test? A whole second sounds 
> awful long when we need to do 10+ tests currently.
>
> Christian.
>
>> ---
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c 
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
>> index 050062e..a31d7ef 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
>> @@ -33,7 +33,7 @@
>>   #include "amdgpu.h"
>>   #include "atom.h"
>>   -#define AMDGPU_IB_TEST_TIMEOUT    msecs_to_jiffies(10)
>> +#define AMDGPU_IB_TEST_TIMEOUT    msecs_to_jiffies(1000)
>>     /*
>>    * IB
>
>



More information about the amd-gfx mailing list