[igt-dev] [PATCH i-g-t 1/2] tests/gem_exec_await: Relax the busy spinner

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Mon Nov 19 15:33:56 UTC 2018


On 19/11/2018 15:28, Chris Wilson wrote:
> Quoting Tvrtko Ursulin (2018-11-19 15:22:28)
>> From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
>>
>> Add some nop instructions between recursive batch buffer start calls to
>> give system some breathing room. Without these, especially when coupled
>> with memory pressure, false GPU hangs can be observed caused by the
>> inability of the chip to cope.
> 
> Doesn't seem to be required. And the machines most susceptible to timer
> errors due to busyspin have not show the issue.

With the memory pressure subtest, the second patch in this series, it 
was make it or break it to have the nops. Without them it was GPU hangs 
all around, and with them so far all clean.

Regards,

Tvrtko


More information about the igt-dev mailing list