Re: [Intel-gfx] [PATCH] tests/gem_eio: Disable reset for wait subtests

2015-11-26 Thread Daniel Vetter
On Wed, Nov 25, 2015 at 04:34:13PM +, Chris Wilson wrote: > On Wed, Nov 25, 2015 at 04:29:01PM +, Chris Wilson wrote: > > On Wed, Nov 25, 2015 at 04:58:19PM +0100, Daniel Vetter wrote: > > > This testcase tries to validate -EIO behaviour by disabling gpu reset > > > support in the kernel.

Re: [Intel-gfx] [PATCH] tests/gem_eio: Disable reset for wait subtests

2015-11-26 Thread Chris Wilson
On Thu, Nov 26, 2015 at 09:36:14AM +0100, Daniel Vetter wrote: > On Wed, Nov 25, 2015 at 04:34:13PM +, Chris Wilson wrote: > > On Wed, Nov 25, 2015 at 04:29:01PM +, Chris Wilson wrote: > > > On Wed, Nov 25, 2015 at 04:58:19PM +0100, Daniel Vetter wrote: > > > > This testcase tries to

Re: [Intel-gfx] [PATCH] tests/gem_eio: Disable reset for wait subtests

2015-11-26 Thread Daniel Vetter
On Thu, Nov 26, 2015 at 10:03:01AM +, Chris Wilson wrote: > On Thu, Nov 26, 2015 at 09:36:14AM +0100, Daniel Vetter wrote: > > On Wed, Nov 25, 2015 at 04:34:13PM +, Chris Wilson wrote: > > > On Wed, Nov 25, 2015 at 04:29:01PM +, Chris Wilson wrote: > > > > On Wed, Nov 25, 2015 at

Re: [Intel-gfx] [PATCH] tests/gem_eio: Disable reset for wait subtests

2015-11-25 Thread Chris Wilson
On Wed, Nov 25, 2015 at 04:29:01PM +, Chris Wilson wrote: > On Wed, Nov 25, 2015 at 04:58:19PM +0100, Daniel Vetter wrote: > > This testcase tries to validate -EIO behaviour by disabling gpu reset > > support in the kernel. Except that the wait subtest forgot to do that, > > and therefore gets

[Intel-gfx] [PATCH] tests/gem_eio: Disable reset for wait subtests

2015-11-25 Thread Daniel Vetter
This testcase tries to validate -EIO behaviour by disabling gpu reset support in the kernel. Except that the wait subtest forgot to do that, and therefore gets a return value of 0 instead of the expected -EIO. With this fix gem_eio passes on a kernel with my fixes completely. Cc: Chris Wilson

Re: [Intel-gfx] [PATCH] tests/gem_eio: Disable reset for wait subtests

2015-11-25 Thread Chris Wilson
On Wed, Nov 25, 2015 at 04:58:19PM +0100, Daniel Vetter wrote: > This testcase tries to validate -EIO behaviour by disabling gpu reset > support in the kernel. Except that the wait subtest forgot to do that, > and therefore gets a return value of 0 instead of the expected -EIO. > Wrong. It was