All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing
@ 2019-09-23  7:41 ` Chris Wilson
  0 siblings, 0 replies; 5+ messages in thread
From: Chris Wilson @ 2019-09-23  7:41 UTC (permalink / raw)
  To: intel-gfx; +Cc: igt-dev

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

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* [igt-dev] [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing
@ 2019-09-23  7:41 ` Chris Wilson
  0 siblings, 0 replies; 5+ messages in thread
From: Chris Wilson @ 2019-09-23  7:41 UTC (permalink / raw)
  To: intel-gfx; +Cc: igt-dev

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

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for i915/gem_eio: Include non-context based tests for older kms testing
  2019-09-23  7:41 ` [igt-dev] " Chris Wilson
  (?)
@ 2019-09-23  9:25 ` Patchwork
  -1 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2019-09-23  9:25 UTC (permalink / raw)
  To: Chris Wilson; +Cc: igt-dev

== 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

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing
  2019-09-23  7:41 ` [igt-dev] " Chris Wilson
@ 2019-09-24  8:49   ` Andi Shyti
  -1 siblings, 0 replies; 5+ messages in thread
From: Andi Shyti @ 2019-09-24  8:49 UTC (permalink / raw)
  To: Chris Wilson; +Cc: igt-dev, intel-gfx

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();
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing
@ 2019-09-24  8:49   ` Andi Shyti
  0 siblings, 0 replies; 5+ messages in thread
From: Andi Shyti @ 2019-09-24  8:49 UTC (permalink / raw)
  To: Chris Wilson; +Cc: igt-dev, intel-gfx

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

^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-09-24  8:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-23  7:41 [PATCH i-g-t] i915/gem_eio: Include non-context based tests for older kms testing Chris Wilson
2019-09-23  7:41 ` [igt-dev] " Chris Wilson
2019-09-23  9:25 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
2019-09-24  8:49 ` [PATCH i-g-t] " Andi Shyti
2019-09-24  8:49   ` [igt-dev] [Intel-gfx] " Andi Shyti

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.