Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

Quoting Tvrtko Ursulin (2018-09-18 11:33:14) > > On 18/09/2018 11:03, Chris Wilson wrote: > > Quoting Chris Wilson (2018-09-18 11:02:09) > >> Quoting Tvrtko Ursulin (2018-09-18 10:59:11) > >>> > >>> On 18/09/2018 10:44, Chris Wilson wrote: > Quoting Tvrtko Ursulin (2018-09-18 10:38:44) >

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

On 18/09/2018 11:03, Chris Wilson wrote: Quoting Chris Wilson (2018-09-18 11:02:09) Quoting Tvrtko Ursulin (2018-09-18 10:59:11) On 18/09/2018 10:44, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-09-18 10:38:44) Can I say it is testing the i915_execbuffer2_set_context_id as well by

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

Quoting Chris Wilson (2018-09-18 11:02:09) > Quoting Tvrtko Ursulin (2018-09-18 10:59:11) > > > > On 18/09/2018 10:44, Chris Wilson wrote: > > > Quoting Tvrtko Ursulin (2018-09-18 10:38:44) > > >> Can I say it is testing the i915_execbuffer2_set_context_id as well by > > >> knowing underlying ABI

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

Quoting Tvrtko Ursulin (2018-09-18 10:59:11) > > On 18/09/2018 10:44, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2018-09-18 10:38:44) > >> > >> On 17/09/2018 16:52, Chris Wilson wrote: > >>> Quoting Tvrtko Ursulin (2018-09-17 16:46:18) > From: Tvrtko Ursulin > > Move a really

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

On 18/09/2018 10:44, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-09-18 10:38:44) On 17/09/2018 16:52, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-09-17 16:46:18) From: Tvrtko Ursulin Move a really small test that invalid context is rejected under the gem_ctx_exec umbrella. v2:

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

Quoting Tvrtko Ursulin (2018-09-18 10:38:44) > > On 17/09/2018 16:52, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2018-09-17 16:46:18) > >> From: Tvrtko Ursulin > >> > >> Move a really small test that invalid context is rejected under the > >> gem_ctx_exec umbrella. > >> > >> v2: > >> *

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

On 17/09/2018 16:52, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-09-17 16:46:18) From: Tvrtko Ursulin Move a really small test that invalid context is rejected under the gem_ctx_exec umbrella. v2: * And actually fix the test so it does what it claims. And add more variety in the

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t 1/2] tests/gem_ctx_bad_exec: Consolidate to gem_ctx_exec

Quoting Tvrtko Ursulin (2018-09-17 16:46:18) > From: Tvrtko Ursulin > > Move a really small test that invalid context is rejected under the > gem_ctx_exec umbrella. > > v2: > * And actually fix the test so it does what it claims. And add more >variety in the invalid context id's it tests