All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/i915/guc: Clear preempt status before use
@ 2018-10-12 15:25 Chris Wilson
  2018-10-12 15:58 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
                   ` (7 more replies)
  0 siblings, 8 replies; 9+ messages in thread
From: Chris Wilson @ 2018-10-12 15:25 UTC (permalink / raw)
  To: intel-gfx

---
 drivers/gpu/drm/i915/intel_guc_submission.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_guc_submission.c b/drivers/gpu/drm/i915/intel_guc_submission.c
index eae668442ebe..4c45da01ea1c 100644
--- a/drivers/gpu/drm/i915/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/intel_guc_submission.c
@@ -654,11 +654,10 @@ static void complete_preempt_context(struct intel_engine_cs *engine)
 	if (inject_preempt_hang(execlists))
 		return;
 
+	wait_for_guc_preempt_report(engine);
+
 	execlists_cancel_port_requests(execlists);
 	execlists_unwind_incomplete_requests(execlists);
-
-	wait_for_guc_preempt_report(engine);
-	intel_write_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX, 0);
 }
 
 /**
@@ -726,6 +725,9 @@ static bool __guc_dequeue(struct intel_engine_cs *engine)
 			int prio = execlists->queue_priority;
 
 			if (__execlists_need_preempt(prio, port_prio(port))) {
+				intel_write_status_page(engine,
+							I915_GEM_HWS_PREEMPT_INDEX,
+							0);
 				execlists_set_active(execlists,
 						     EXECLISTS_ACTIVE_PREEMPT);
 				queue_work(engine->i915->guc.preempt_wq,
@@ -816,8 +818,7 @@ static void guc_submission_tasklet(unsigned long data)
 	}
 
 	if (execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT) &&
-	    intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX) ==
-	    GUC_PREEMPT_FINISHED)
+	    intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX))
 		complete_preempt_context(engine);
 
 	if (!execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT))
-- 
2.19.1

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
@ 2018-10-12 15:58 ` Patchwork
  2018-10-12 16:20 ` ✓ Fi.CI.BAT: success " Patchwork
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2018-10-12 15:58 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/guc: Clear preempt status before use
URL   : https://patchwork.freedesktop.org/series/50936/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
61c98e4e5b38 drm/i915/guc: Clear preempt status before use
-:8: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:44: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 1 errors, 1 warnings, 0 checks, 31 lines checked

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* ✓ Fi.CI.BAT: success for drm/i915/guc: Clear preempt status before use
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
  2018-10-12 15:58 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
@ 2018-10-12 16:20 ` Patchwork
  2018-10-12 16:33 ` [PATCH] " Chris Wilson
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2018-10-12 16:20 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/guc: Clear preempt status before use
URL   : https://patchwork.freedesktop.org/series/50936/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4978 -> Patchwork_10442 =

== Summary - SUCCESS ==

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/50936/revisions/1/mbox/

== Possible new issues ==

  Here are the unknown changes that may have been introduced in Patchwork_10442:

  === IGT changes ===

    ==== Warnings ====

    igt@drv_selftest@live_gem:
      {fi-apl-guc}:       INCOMPLETE (fdo#106693) -> DMESG-WARN

    
== Known issues ==

  Here are the changes found in Patchwork_10442 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@drv_selftest@live_coherency:
      fi-gdg-551:         PASS -> DMESG-FAIL (fdo#107164)

    igt@drv_selftest@live_guc:
      {fi-apl-guc}:       NOTRUN -> DMESG-WARN (fdo#107258)

    igt@kms_flip@basic-flip-vs-modeset:
      fi-skl-6700hq:      PASS -> DMESG-WARN (fdo#105998)

    igt@kms_pipe_crc_basic@hang-read-crc-pipe-b:
      fi-byt-clapper:     PASS -> FAIL (fdo#107362, fdo#103191)

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
      fi-icl-u:           NOTRUN -> INCOMPLETE (fdo#107713)

    
    ==== Possible fixes ====

    igt@drv_module_reload@basic-reload:
      fi-blb-e6850:       INCOMPLETE (fdo#107718) -> PASS

    igt@kms_pipe_crc_basic@nonblocking-crc-pipe-b-frame-sequence:
      fi-byt-clapper:     FAIL (fdo#107362, fdo#103191) -> PASS +1

    
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
  fdo#105998 https://bugs.freedesktop.org/show_bug.cgi?id=105998
  fdo#106693 https://bugs.freedesktop.org/show_bug.cgi?id=106693
  fdo#107164 https://bugs.freedesktop.org/show_bug.cgi?id=107164
  fdo#107258 https://bugs.freedesktop.org/show_bug.cgi?id=107258
  fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
  fdo#107713 https://bugs.freedesktop.org/show_bug.cgi?id=107713
  fdo#107718 https://bugs.freedesktop.org/show_bug.cgi?id=107718


== Participating hosts (44 -> 42) ==

  Additional (1): fi-icl-u 
  Missing    (3): fi-ilk-m540 fi-byt-squawks fi-bsw-cyan 


== Build changes ==

    * Linux: CI_DRM_4978 -> Patchwork_10442

  CI_DRM_4978: ca98b2681a49a1417f8157af2d94a4f2d0bd0e47 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4674: 93871c6fb3c25e5d350c9faf36ded917174214de @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10442: 61c98e4e5b38f51aae2ca7542d105354fc24f8ec @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

61c98e4e5b38 drm/i915/guc: Clear preempt status before use

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10442/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] drm/i915/guc: Clear preempt status before use
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
  2018-10-12 15:58 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
  2018-10-12 16:20 ` ✓ Fi.CI.BAT: success " Patchwork
@ 2018-10-12 16:33 ` Chris Wilson
  2018-10-12 16:40 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev2) Patchwork
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: Chris Wilson @ 2018-10-12 16:33 UTC (permalink / raw)
  To: intel-gfx

---
 drivers/gpu/drm/i915/intel_guc_submission.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_guc_submission.c b/drivers/gpu/drm/i915/intel_guc_submission.c
index eae668442ebe..c257eceef862 100644
--- a/drivers/gpu/drm/i915/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/intel_guc_submission.c
@@ -610,11 +610,13 @@ static void inject_preempt_context(struct work_struct *work)
 	data[5] = guc->execbuf_client->stage_id;
 	data[6] = intel_guc_ggtt_offset(guc, guc->shared_data);
 
+	local_bh_disable();
 	if (WARN_ON(intel_guc_send(guc, data, ARRAY_SIZE(data)))) {
 		execlists_clear_active(&engine->execlists,
 				       EXECLISTS_ACTIVE_PREEMPT);
 		tasklet_schedule(&engine->execlists.tasklet);
 	}
+	local_bh_enable();
 }
 
 /*
@@ -654,11 +656,10 @@ static void complete_preempt_context(struct intel_engine_cs *engine)
 	if (inject_preempt_hang(execlists))
 		return;
 
+	wait_for_guc_preempt_report(engine);
+
 	execlists_cancel_port_requests(execlists);
 	execlists_unwind_incomplete_requests(execlists);
-
-	wait_for_guc_preempt_report(engine);
-	intel_write_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX, 0);
 }
 
 /**
@@ -726,6 +727,9 @@ static bool __guc_dequeue(struct intel_engine_cs *engine)
 			int prio = execlists->queue_priority;
 
 			if (__execlists_need_preempt(prio, port_prio(port))) {
+				intel_write_status_page(engine,
+							I915_GEM_HWS_PREEMPT_INDEX,
+							0);
 				execlists_set_active(execlists,
 						     EXECLISTS_ACTIVE_PREEMPT);
 				queue_work(engine->i915->guc.preempt_wq,
@@ -816,8 +820,7 @@ static void guc_submission_tasklet(unsigned long data)
 	}
 
 	if (execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT) &&
-	    intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX) ==
-	    GUC_PREEMPT_FINISHED)
+	    intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX))
 		complete_preempt_context(engine);
 
 	if (!execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT))
-- 
2.19.1

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev2)
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
                   ` (2 preceding siblings ...)
  2018-10-12 16:33 ` [PATCH] " Chris Wilson
@ 2018-10-12 16:40 ` Patchwork
  2018-10-12 16:58 ` ✓ Fi.CI.BAT: success " Patchwork
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2018-10-12 16:40 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/guc: Clear preempt status before use (rev2)
URL   : https://patchwork.freedesktop.org/series/50936/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
9aca37eb0037 drm/i915/guc: Clear preempt status before use
-:8: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:58: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 1 errors, 1 warnings, 0 checks, 44 lines checked

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* ✓ Fi.CI.BAT: success for drm/i915/guc: Clear preempt status before use (rev2)
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
                   ` (3 preceding siblings ...)
  2018-10-12 16:40 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev2) Patchwork
@ 2018-10-12 16:58 ` Patchwork
  2018-10-12 17:06 ` [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2018-10-12 16:58 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/guc: Clear preempt status before use (rev2)
URL   : https://patchwork.freedesktop.org/series/50936/
State : success

== Summary ==

= CI Bug Log - changes from CI_DRM_4978 -> Patchwork_10443 =

== Summary - SUCCESS ==

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/50936/revisions/2/mbox/

== Possible new issues ==

  Here are the unknown changes that may have been introduced in Patchwork_10443:

  === IGT changes ===

    ==== Possible regressions ====

    igt@drv_selftest@live_hangcheck:
      {fi-apl-guc}:       NOTRUN -> DMESG-WARN +1

    igt@pm_rpm@module-reload:
      {fi-apl-guc}:       PASS -> DMESG-WARN +3

    
    ==== Warnings ====

    igt@drv_selftest@live_gem:
      {fi-apl-guc}:       INCOMPLETE (fdo#106693) -> DMESG-WARN

    
== Known issues ==

  Here are the changes found in Patchwork_10443 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@drv_module_reload@basic-reload:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#106248, fdo#106725)

    igt@drv_selftest@live_guc:
      {fi-apl-guc}:       NOTRUN -> DMESG-WARN (fdo#107258)

    igt@kms_flip@basic-flip-vs-modeset:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#106000)

    igt@kms_flip@basic-flip-vs-wf_vblank:
      fi-glk-j4005:       PASS -> FAIL (fdo#100368)

    igt@kms_flip@basic-plain-flip:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#106097)

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
      fi-byt-clapper:     PASS -> FAIL (fdo#103191, fdo#107362)

    
    ==== Possible fixes ====

    igt@drv_module_reload@basic-reload:
      fi-blb-e6850:       INCOMPLETE (fdo#107718) -> PASS

    igt@kms_frontbuffer_tracking@basic:
      fi-byt-clapper:     FAIL (fdo#103167) -> PASS

    igt@kms_pipe_crc_basic@nonblocking-crc-pipe-b-frame-sequence:
      fi-byt-clapper:     FAIL (fdo#103191, fdo#107362) -> PASS +1

    
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
  fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
  fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
  fdo#106000 https://bugs.freedesktop.org/show_bug.cgi?id=106000
  fdo#106097 https://bugs.freedesktop.org/show_bug.cgi?id=106097
  fdo#106248 https://bugs.freedesktop.org/show_bug.cgi?id=106248
  fdo#106693 https://bugs.freedesktop.org/show_bug.cgi?id=106693
  fdo#106725 https://bugs.freedesktop.org/show_bug.cgi?id=106725
  fdo#107258 https://bugs.freedesktop.org/show_bug.cgi?id=107258
  fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
  fdo#107718 https://bugs.freedesktop.org/show_bug.cgi?id=107718


== Participating hosts (44 -> 41) ==

  Additional (1): fi-icl-u 
  Missing    (4): fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-pnv-d510 


== Build changes ==

    * Linux: CI_DRM_4978 -> Patchwork_10443

  CI_DRM_4978: ca98b2681a49a1417f8157af2d94a4f2d0bd0e47 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4674: 93871c6fb3c25e5d350c9faf36ded917174214de @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10443: 9aca37eb003750967e9cb11fa086446bea8cc7a6 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

9aca37eb0037 drm/i915/guc: Clear preempt status before use

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10443/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] drm/i915/guc: Clear preempt status before use
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
                   ` (4 preceding siblings ...)
  2018-10-12 16:58 ` ✓ Fi.CI.BAT: success " Patchwork
@ 2018-10-12 17:06 ` Chris Wilson
  2018-10-12 17:17 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev3) Patchwork
  2018-10-12 17:33 ` ✗ Fi.CI.BAT: failure " Patchwork
  7 siblings, 0 replies; 9+ messages in thread
From: Chris Wilson @ 2018-10-12 17:06 UTC (permalink / raw)
  To: intel-gfx

---
 drivers/gpu/drm/i915/intel_guc_submission.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_guc_submission.c b/drivers/gpu/drm/i915/intel_guc_submission.c
index eae668442ebe..e1d5463a04e6 100644
--- a/drivers/gpu/drm/i915/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/intel_guc_submission.c
@@ -610,11 +610,13 @@ static void inject_preempt_context(struct work_struct *work)
 	data[5] = guc->execbuf_client->stage_id;
 	data[6] = intel_guc_ggtt_offset(guc, guc->shared_data);
 
+	tasklet_disable(&engine->execlists.tasklet);
 	if (WARN_ON(intel_guc_send(guc, data, ARRAY_SIZE(data)))) {
 		execlists_clear_active(&engine->execlists,
 				       EXECLISTS_ACTIVE_PREEMPT);
 		tasklet_schedule(&engine->execlists.tasklet);
 	}
+	tasklet_enable(&engine->execlists.tasklet);
 }
 
 /*
@@ -654,11 +656,10 @@ static void complete_preempt_context(struct intel_engine_cs *engine)
 	if (inject_preempt_hang(execlists))
 		return;
 
+	wait_for_guc_preempt_report(engine);
+
 	execlists_cancel_port_requests(execlists);
 	execlists_unwind_incomplete_requests(execlists);
-
-	wait_for_guc_preempt_report(engine);
-	intel_write_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX, 0);
 }
 
 /**
@@ -726,6 +727,9 @@ static bool __guc_dequeue(struct intel_engine_cs *engine)
 			int prio = execlists->queue_priority;
 
 			if (__execlists_need_preempt(prio, port_prio(port))) {
+				intel_write_status_page(engine,
+							I915_GEM_HWS_PREEMPT_INDEX,
+							0);
 				execlists_set_active(execlists,
 						     EXECLISTS_ACTIVE_PREEMPT);
 				queue_work(engine->i915->guc.preempt_wq,
@@ -816,8 +820,7 @@ static void guc_submission_tasklet(unsigned long data)
 	}
 
 	if (execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT) &&
-	    intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX) ==
-	    GUC_PREEMPT_FINISHED)
+	    intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX))
 		complete_preempt_context(engine);
 
 	if (!execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT))
-- 
2.19.1

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev3)
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
                   ` (5 preceding siblings ...)
  2018-10-12 17:06 ` [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
@ 2018-10-12 17:17 ` Patchwork
  2018-10-12 17:33 ` ✗ Fi.CI.BAT: failure " Patchwork
  7 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2018-10-12 17:17 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/guc: Clear preempt status before use (rev3)
URL   : https://patchwork.freedesktop.org/series/50936/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
769878ff15d5 drm/i915/guc: Clear preempt status before use
-:8: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one

-:58: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 1 errors, 1 warnings, 0 checks, 44 lines checked

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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* ✗ Fi.CI.BAT: failure for drm/i915/guc: Clear preempt status before use (rev3)
  2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
                   ` (6 preceding siblings ...)
  2018-10-12 17:17 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev3) Patchwork
@ 2018-10-12 17:33 ` Patchwork
  7 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2018-10-12 17:33 UTC (permalink / raw)
  To: Chris Wilson; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/guc: Clear preempt status before use (rev3)
URL   : https://patchwork.freedesktop.org/series/50936/
State : failure

== Summary ==

= CI Bug Log - changes from CI_DRM_4978 -> Patchwork_10444 =

== Summary - FAILURE ==

  Serious unknown changes coming with Patchwork_10444 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_10444, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/50936/revisions/3/mbox/

== Possible new issues ==

  Here are the unknown changes that may have been introduced in Patchwork_10444:

  === IGT changes ===

    ==== Possible regressions ====

    igt@pm_rpm@module-reload:
      fi-glk-j4005:       PASS -> FAIL

    
    ==== Warnings ====

    igt@drv_selftest@live_gem:
      {fi-apl-guc}:       INCOMPLETE (fdo#106693) -> DMESG-WARN

    
== Known issues ==

  Here are the changes found in Patchwork_10444 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@drv_module_reload@basic-reload:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#106248, fdo#106725)

    igt@drv_selftest@live_guc:
      {fi-apl-guc}:       NOTRUN -> DMESG-WARN (fdo#107258)

    igt@drv_selftest@live_sanitycheck:
      fi-glk-j4005:       PASS -> DMESG-WARN (fdo#107726)

    igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
      fi-byt-clapper:     PASS -> FAIL (fdo#103191, fdo#107362)

    igt@kms_pipe_crc_basic@nonblocking-crc-pipe-b:
      fi-ilk-650:         PASS -> DMESG-WARN (fdo#106387)

    igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
      fi-blb-e6850:       PASS -> INCOMPLETE (fdo#107718)

    
    ==== Possible fixes ====

    igt@kms_pipe_crc_basic@nonblocking-crc-pipe-b-frame-sequence:
      fi-byt-clapper:     FAIL (fdo#103191, fdo#107362) -> PASS +1

    
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
  fdo#106248 https://bugs.freedesktop.org/show_bug.cgi?id=106248
  fdo#106387 https://bugs.freedesktop.org/show_bug.cgi?id=106387
  fdo#106693 https://bugs.freedesktop.org/show_bug.cgi?id=106693
  fdo#106725 https://bugs.freedesktop.org/show_bug.cgi?id=106725
  fdo#107258 https://bugs.freedesktop.org/show_bug.cgi?id=107258
  fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
  fdo#107718 https://bugs.freedesktop.org/show_bug.cgi?id=107718
  fdo#107726 https://bugs.freedesktop.org/show_bug.cgi?id=107726


== Participating hosts (44 -> 41) ==

  Missing    (3): fi-ilk-m540 fi-byt-squawks fi-bsw-cyan 


== Build changes ==

    * Linux: CI_DRM_4978 -> Patchwork_10444

  CI_DRM_4978: ca98b2681a49a1417f8157af2d94a4f2d0bd0e47 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4674: 93871c6fb3c25e5d350c9faf36ded917174214de @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_10444: 769878ff15d589cc2486cb729f6a4ae6877f5a0e @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

769878ff15d5 drm/i915/guc: Clear preempt status before use

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_10444/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2018-10-12 17:33 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-12 15:25 [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
2018-10-12 15:58 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2018-10-12 16:20 ` ✓ Fi.CI.BAT: success " Patchwork
2018-10-12 16:33 ` [PATCH] " Chris Wilson
2018-10-12 16:40 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev2) Patchwork
2018-10-12 16:58 ` ✓ Fi.CI.BAT: success " Patchwork
2018-10-12 17:06 ` [PATCH] drm/i915/guc: Clear preempt status before use Chris Wilson
2018-10-12 17:17 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915/guc: Clear preempt status before use (rev3) Patchwork
2018-10-12 17:33 ` ✗ Fi.CI.BAT: failure " Patchwork

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.