Re: [Intel-gfx] [PATCH igt 2/2] lib: Remove illegal instructions from hang injection

2017-08-08 Thread Daniel Vetter
On Mon, Aug 07, 2017 at 04:33:40PM +0300, Mika Kuoppala wrote: > Chris Wilson writes: > > > The idea behind using an illegal instruction was to hang the GPU must > > faster than simply using the recursive batch. However, we stopped doing > > so on gen8+ as the CS parser

Re: [Intel-gfx] [PATCH igt 2/2] lib: Remove illegal instructions from hang injection

2017-08-07 Thread Mika Kuoppala
Chris Wilson writes: > The idea behind using an illegal instruction was to hang the GPU must > faster than simply using the recursive batch. However, we stopped doing > so on gen8+ as the CS parser was much laxer and allowed the illegal > command through but still

[Intel-gfx] [PATCH igt 2/2] lib: Remove illegal instructions from hang injection

2017-08-07 Thread Chris Wilson
The idea behind using an illegal instruction was to hang the GPU must faster than simply using the recursive batch. However, we stopped doing so on gen8+ as the CS parser was much laxer and allowed the illegal command through but still interpreted the packet length (jumping over the recursive