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 > >
