[igt-dev] [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing

[igt-dev] [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing

[Chris Wilson]

As not every machine can use contexts, include a non-context reset stress
test to run in parallel to enabling/disabling pipes.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/i915/gem_eio.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tests/i915/gem_eio.c b/tests/i915/gem_eio.c
index e7f5d4ddb..52e83a5cf 100644
--- a/tests/i915/gem_eio.c
+++ b/tests/i915/gem_eio.c
@@ -870,8 +870,11 @@ static void test_kms(int i915, igt_display_t *dpy)
 	igt_fork(child, 1)
 		display_helper(dpy, shared);
 
-	test_reset_stress(i915, 0);
-	test_reset_stress(i915, TEST_WEDGE);
+	test_inflight(i915, 0);
+	if (gem_has_contexts(fd)) {
+		test_reset_stress(i915, 0);
+		test_reset_stress(i915, TEST_WEDGE);
+	}
 
 	*shared = 1;
 	igt_waitchildren();
-- 
2.23.0

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✗ GitLab.Pipeline: warning for i915/gem_eio: Include non-context based tests for older kms testing

[Patchwork]

== Series Details ==

Series: i915/gem_eio: Include non-context based tests for older kms testing
URL   : https://patchwork.freedesktop.org/series/67067/
State : warning

== Summary ==

Did not get list of undocumented tests for this run, something is wrong!

Other than that, pipeline status: FAILED.

see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/65499 for more details

== Logs ==

For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/65499
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing

[Andi Shyti]

Hi Chris,

On Mon, Sep 23, 2019 at 08:41:49AM +0100, Chris Wilson wrote:
> As not every machine can use contexts, include a non-context reset stress
> test to run in parallel to enabling/disabling pipes.

makes sense.

> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  tests/i915/gem_eio.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/i915/gem_eio.c b/tests/i915/gem_eio.c
> index e7f5d4ddb..52e83a5cf 100644
> --- a/tests/i915/gem_eio.c
> +++ b/tests/i915/gem_eio.c
> @@ -870,8 +870,11 @@ static void test_kms(int i915, igt_display_t *dpy)
>  	igt_fork(child, 1)
>  		display_helper(dpy, shared);
>  
> -	test_reset_stress(i915, 0);
> -	test_reset_stress(i915, TEST_WEDGE);
> +	test_inflight(i915, 0);
> +	if (gem_has_contexts(fd)) {

considering that in v2 I am sure you are going to provide an fd
of "some sort" :)

Reveiwed-by: Andi Shyti <andi.shyti@intel.com>

Thanks,
Andi

> +		test_reset_stress(i915, 0);
> +		test_reset_stress(i915, TEST_WEDGE);
> +	}
>  
>  	*shared = 1;
>  	igt_waitchildren();
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev