* [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume
@ 2019-12-26 0:56 Chris Wilson
2019-12-26 0:56 ` [Intel-gfx] [PATCH 2/5] drm/i915/gt: Discard stale context state from across idling Chris Wilson
` (5 more replies)
0 siblings, 6 replies; 7+ messages in thread
From: Chris Wilson @ 2019-12-26 0:56 UTC (permalink / raw)
To: intel-gfx
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/gt/intel_lrc.c | 10 ++++++++++
drivers/gpu/drm/i915/gt/intel_ring_submission.c | 2 +-
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_lrc.c b/drivers/gpu/drm/i915/gt/intel_lrc.c
index 4fb70a7716e3..2eeb95c61a48 100644
--- a/drivers/gpu/drm/i915/gt/intel_lrc.c
+++ b/drivers/gpu/drm/i915/gt/intel_lrc.c
@@ -2494,6 +2494,8 @@ static int execlists_context_alloc(struct intel_context *ce)
static void execlists_context_reset(struct intel_context *ce)
{
+ GEM_BUG_ON(!intel_context_is_pinned(ce));
+
/*
* Because we emit WA_TAIL_DWORDS there may be a disparity
* between our bookkeeping in ce->ring->head and ce->ring->tail and
@@ -2511,7 +2513,15 @@ static void execlists_context_reset(struct intel_context *ce)
* simplicity, we just zero everything out.
*/
intel_ring_reset(ce->ring, 0);
+
+ memset(ce->lrc_reg_state, 0, PAGE_SIZE);
+ execlists_init_reg_state(ce->lrc_reg_state,
+ ce, ce->engine, ce->ring, false);
__execlists_update_reg_state(ce, ce->engine);
+ ce->lrc_reg_state[CTX_CONTEXT_CONTROL] |=
+ _MASKED_BIT_ENABLE(CTX_CTRL_ENGINE_CTX_RESTORE_INHIBIT);
+
+ ce->lrc_desc |= CTX_DESC_FORCE_RESTORE;
}
static const struct intel_context_ops execlists_context_ops = {
diff --git a/drivers/gpu/drm/i915/gt/intel_ring_submission.c b/drivers/gpu/drm/i915/gt/intel_ring_submission.c
index 81f872f9ef03..2caf07b3ec22 100644
--- a/drivers/gpu/drm/i915/gt/intel_ring_submission.c
+++ b/drivers/gpu/drm/i915/gt/intel_ring_submission.c
@@ -1347,7 +1347,7 @@ static int ring_context_pin(struct intel_context *ce)
static void ring_context_reset(struct intel_context *ce)
{
- intel_ring_reset(ce->ring, 0);
+ intel_ring_reset(ce->ring, ce->ring->emit);
}
static const struct intel_context_ops ring_context_ops = {
--
2.24.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Intel-gfx] [PATCH 2/5] drm/i915/gt: Discard stale context state from across idling
2019-12-26 0:56 [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
@ 2019-12-26 0:56 ` Chris Wilson
2019-12-26 0:56 ` [Intel-gfx] [PATCH 3/5] drm/i915/gt: Always poison the kernel_context image before unparking Chris Wilson
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Chris Wilson @ 2019-12-26 0:56 UTC (permalink / raw)
To: intel-gfx
Before we idle, on parking, we switch to the kernel context such that we
have a scratch context loaded while the GPU idle, protecting any
precious user state. Be paranoid and assume that the idle state may have
been trashed, and reset the kernel_context image after idling.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Imre Deak <imre.deak@intel.com>
---
drivers/gpu/drm/i915/gt/intel_engine_pm.c | 6 ++++++
drivers/gpu/drm/i915/gt/intel_gt_pm.c | 8 --------
2 files changed, 6 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_engine_pm.c b/drivers/gpu/drm/i915/gt/intel_engine_pm.c
index 010620b78202..b9444ce29a3a 100644
--- a/drivers/gpu/drm/i915/gt/intel_engine_pm.c
+++ b/drivers/gpu/drm/i915/gt/intel_engine_pm.c
@@ -20,6 +20,7 @@ static int __engine_unpark(struct intel_wakeref *wf)
{
struct intel_engine_cs *engine =
container_of(wf, typeof(*engine), wakeref);
+ struct intel_context *ce;
void *map;
ENGINE_TRACE(engine, "\n");
@@ -34,6 +35,11 @@ static int __engine_unpark(struct intel_wakeref *wf)
if (!IS_ERR_OR_NULL(map))
engine->pinned_default_state = map;
+ /* Discard stale context state from across idling */
+ ce = engine->kernel_context;
+ if (ce)
+ ce->ops->reset(ce);
+
if (engine->unpark)
engine->unpark(engine);
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_pm.c b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
index 45b68a17da4d..46a0aed2d5d5 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt_pm.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
@@ -215,16 +215,8 @@ int intel_gt_resume(struct intel_gt *gt)
intel_llc_enable(>->llc);
for_each_engine(engine, gt, id) {
- struct intel_context *ce;
-
intel_engine_pm_get(engine);
- ce = engine->kernel_context;
- if (ce) {
- GEM_BUG_ON(!intel_context_is_pinned(ce));
- ce->ops->reset(ce);
- }
-
engine->serial++; /* kernel context lost */
err = engine->resume(engine);
--
2.24.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Intel-gfx] [PATCH 3/5] drm/i915/gt: Always poison the kernel_context image before unparking
2019-12-26 0:56 [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
2019-12-26 0:56 ` [Intel-gfx] [PATCH 2/5] drm/i915/gt: Discard stale context state from across idling Chris Wilson
@ 2019-12-26 0:56 ` Chris Wilson
2019-12-26 0:57 ` [Intel-gfx] [PATCH 4/5] drm/i915/gt: Restore coarse power gating for gen9 Chris Wilson
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Chris Wilson @ 2019-12-26 0:56 UTC (permalink / raw)
To: intel-gfx
Keep scrubbing the kernel_context image with poison before we reset it
in order to demonstrate that we will be resilient in the case where it
is accidentally overwritten on idle.
Suggested-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Imre Deak <imre.deak@intel.com>
---
drivers/gpu/drm/i915/gt/intel_engine_pm.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/drivers/gpu/drm/i915/gt/intel_engine_pm.c b/drivers/gpu/drm/i915/gt/intel_engine_pm.c
index b9444ce29a3a..86970bbff8bb 100644
--- a/drivers/gpu/drm/i915/gt/intel_engine_pm.c
+++ b/drivers/gpu/drm/i915/gt/intel_engine_pm.c
@@ -35,6 +35,20 @@ static int __engine_unpark(struct intel_wakeref *wf)
if (!IS_ERR_OR_NULL(map))
engine->pinned_default_state = map;
+ /* Poison the kernel_context image to verify we never fully trust it */
+ if (IS_ENABLED(CONFIG_DRM_I915_DEBUG_GEM) &&
+ engine->kernel_context->state) {
+ struct drm_i915_gem_object *obj =
+ engine->kernel_context->state->obj;
+
+ map = i915_gem_object_pin_map(obj, i915_coherent_map_type(engine->i915));
+ if (!IS_ERR(map)) {
+ memset(map, POISON_INUSE, obj->base.size);
+ i915_gem_object_flush_map(obj);
+ i915_gem_object_unpin_map(obj);
+ }
+ }
+
/* Discard stale context state from across idling */
ce = engine->kernel_context;
if (ce)
--
2.24.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Intel-gfx] [PATCH 4/5] drm/i915/gt: Restore coarse power gating for gen9
2019-12-26 0:56 [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
2019-12-26 0:56 ` [Intel-gfx] [PATCH 2/5] drm/i915/gt: Discard stale context state from across idling Chris Wilson
2019-12-26 0:56 ` [Intel-gfx] [PATCH 3/5] drm/i915/gt: Always poison the kernel_context image before unparking Chris Wilson
@ 2019-12-26 0:57 ` Chris Wilson
2019-12-26 0:57 ` [Intel-gfx] [PATCH 5/5] drm/i915/gt: Apply sanitiization just before resume Chris Wilson
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Chris Wilson @ 2019-12-26 0:57 UTC (permalink / raw)
To: intel-gfx; +Cc: Eero Tamminen
Believe in the power of the sacrificial kernel context and restore
coarse power gating in the belief that the issue affects only the
context idling on another engine -- i.e. only affects our scratch
kernel_context.
Closes: https://gitlab.freedesktop.org/drm/intel/issues/846
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Imre Deak <imre.deak@intel.com>
Cc: Eero Tamminen <eero.t.tamminen@intel.com>
Cc: Jon Bloomfield <jon.bloomfield@intel.com>
---
drivers/gpu/drm/i915/gt/intel_rc6.c | 9 ++-------
drivers/gpu/drm/i915/gt/uc/intel_guc.c | 2 +-
drivers/gpu/drm/i915/i915_drv.h | 4 ----
3 files changed, 3 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_rc6.c b/drivers/gpu/drm/i915/gt/intel_rc6.c
index 9e303c29d6e3..c47b02e2e029 100644
--- a/drivers/gpu/drm/i915/gt/intel_rc6.c
+++ b/drivers/gpu/drm/i915/gt/intel_rc6.c
@@ -175,13 +175,8 @@ static void gen9_rc6_enable(struct intel_rc6 *rc6)
GEN6_RC_CTL_RC6_ENABLE |
rc6_mode;
- /*
- * WaRsDisableCoarsePowerGating:skl,cnl
- * - Render/Media PG need to be disabled with RC6.
- */
- if (!NEEDS_WaRsDisableCoarsePowerGating(rc6_to_i915(rc6)))
- set(uncore, GEN9_PG_ENABLE,
- GEN9_RENDER_PG_ENABLE | GEN9_MEDIA_PG_ENABLE);
+ set(uncore, GEN9_PG_ENABLE,
+ GEN9_RENDER_PG_ENABLE | GEN9_MEDIA_PG_ENABLE);
}
static void gen8_rc6_enable(struct intel_rc6 *rc6)
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
index 5d00a3b2d914..cfa6dde0fc63 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
@@ -496,7 +496,7 @@ int intel_guc_sample_forcewake(struct intel_guc *guc)
action[0] = INTEL_GUC_ACTION_SAMPLE_FORCEWAKE;
/* WaRsDisableCoarsePowerGating:skl,cnl */
- if (!HAS_RC6(dev_priv) || NEEDS_WaRsDisableCoarsePowerGating(dev_priv))
+ if (IS_GEN(dev_priv, 9))
action[1] = 0;
else
/* bit 0 and 1 are for Render and Media domain separately */
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index b7f122dccdca..776d4e4b9435 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -1656,10 +1656,6 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915,
#define NEEDS_RC6_CTX_CORRUPTION_WA(dev_priv) \
(IS_BROADWELL(dev_priv) || IS_GEN(dev_priv, 9))
-/* WaRsDisableCoarsePowerGating:skl,cnl */
-#define NEEDS_WaRsDisableCoarsePowerGating(dev_priv) \
- IS_GEN_RANGE(dev_priv, 9, 10)
-
#define HAS_GMBUS_IRQ(dev_priv) (INTEL_GEN(dev_priv) >= 4)
#define HAS_GMBUS_BURST_READ(dev_priv) (INTEL_GEN(dev_priv) >= 10 || \
IS_GEMINILAKE(dev_priv) || \
--
2.24.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Intel-gfx] [PATCH 5/5] drm/i915/gt: Apply sanitiization just before resume
2019-12-26 0:56 [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
` (2 preceding siblings ...)
2019-12-26 0:57 ` [Intel-gfx] [PATCH 4/5] drm/i915/gt: Restore coarse power gating for gen9 Chris Wilson
@ 2019-12-26 0:57 ` Chris Wilson
2019-12-26 1:46 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/5] drm/i915/gt: Ignore stale context state upon resume Patchwork
2019-12-26 2:37 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
5 siblings, 0 replies; 7+ messages in thread
From: Chris Wilson @ 2019-12-26 0:57 UTC (permalink / raw)
To: intel-gfx
Bring sanitization completely underneath the umbrella of intel_gt, and
perform it exclusively after suspend and before the next resume.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/gt/intel_gt.c | 2 --
drivers/gpu/drm/i915/gt/intel_gt_pm.c | 15 +++------------
drivers/gpu/drm/i915/gt/intel_gt_pm.h | 2 --
drivers/gpu/drm/i915/i915_drv.c | 2 --
drivers/gpu/drm/i915/selftests/i915_gem.c | 2 --
5 files changed, 3 insertions(+), 20 deletions(-)
diff --git a/drivers/gpu/drm/i915/gt/intel_gt.c b/drivers/gpu/drm/i915/gt/intel_gt.c
index e4f7cc7dd88a..8d5adac97b75 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt.c
@@ -38,8 +38,6 @@ void intel_gt_init_early(struct intel_gt *gt, struct drm_i915_private *i915)
void intel_gt_init_hw_early(struct intel_gt *gt, struct i915_ggtt *ggtt)
{
gt->ggtt = ggtt;
-
- intel_gt_sanitize(gt, false);
}
static void init_unused_ring(struct intel_gt *gt, u32 base)
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_pm.c b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
index 46a0aed2d5d5..bcd8a9806ed2 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt_pm.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
@@ -126,17 +126,7 @@ static bool reset_engines(struct intel_gt *gt)
return __intel_gt_reset(gt, ALL_ENGINES) == 0;
}
-/**
- * intel_gt_sanitize: called after the GPU has lost power
- * @gt: the i915 GT container
- * @force: ignore a failed reset and sanitize engine state anyway
- *
- * Anytime we reset the GPU, either with an explicit GPU reset or through a
- * PCI power cycle, the GPU loses state and we must reset our state tracking
- * to match. Note that calling intel_gt_sanitize() if the GPU has not
- * been reset results in much confusion!
- */
-void intel_gt_sanitize(struct intel_gt *gt, bool force)
+static void gt_sanitize(struct intel_gt *gt, bool force)
{
struct intel_engine_cs *engine;
enum intel_engine_id id;
@@ -201,6 +191,7 @@ int intel_gt_resume(struct intel_gt *gt)
intel_uncore_forcewake_get(gt->uncore, FORCEWAKE_ALL);
intel_rc6_sanitize(>->rc6);
+ gt_sanitize(gt, true);
/* Only when the HW is re-initialised, can we replay the requests */
err = intel_gt_init_hw(gt);
@@ -307,7 +298,7 @@ void intel_gt_suspend_late(struct intel_gt *gt)
intel_llc_disable(>->llc);
}
- intel_gt_sanitize(gt, false);
+ gt_sanitize(gt, false);
GT_TRACE(gt, "\n");
}
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_pm.h b/drivers/gpu/drm/i915/gt/intel_gt_pm.h
index 4a9e48c12bd4..60f0e2fbe55c 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt_pm.h
+++ b/drivers/gpu/drm/i915/gt/intel_gt_pm.h
@@ -51,8 +51,6 @@ void intel_gt_pm_init_early(struct intel_gt *gt);
void intel_gt_pm_init(struct intel_gt *gt);
void intel_gt_pm_fini(struct intel_gt *gt);
-void intel_gt_sanitize(struct intel_gt *gt, bool force);
-
void intel_gt_suspend_prepare(struct intel_gt *gt);
void intel_gt_suspend_late(struct intel_gt *gt);
int intel_gt_resume(struct intel_gt *gt);
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
index 59525094d0e3..9f75e03368c4 100644
--- a/drivers/gpu/drm/i915/i915_drv.c
+++ b/drivers/gpu/drm/i915/i915_drv.c
@@ -1817,8 +1817,6 @@ static int i915_drm_resume(struct drm_device *dev)
disable_rpm_wakeref_asserts(&dev_priv->runtime_pm);
- intel_gt_sanitize(&dev_priv->gt, true);
-
ret = i915_ggtt_enable_hw(dev_priv);
if (ret)
DRM_ERROR("failed to re-enable GGTT\n");
diff --git a/drivers/gpu/drm/i915/selftests/i915_gem.c b/drivers/gpu/drm/i915/selftests/i915_gem.c
index b37fc53973cc..78f36faf2bbe 100644
--- a/drivers/gpu/drm/i915/selftests/i915_gem.c
+++ b/drivers/gpu/drm/i915/selftests/i915_gem.c
@@ -124,8 +124,6 @@ static void pm_resume(struct drm_i915_private *i915)
* that runtime-pm just works.
*/
with_intel_runtime_pm(&i915->runtime_pm, wakeref) {
- intel_gt_sanitize(&i915->gt, false);
-
i915_gem_restore_gtt_mappings(i915);
i915_gem_restore_fences(&i915->ggtt);
--
2.24.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/5] drm/i915/gt: Ignore stale context state upon resume
2019-12-26 0:56 [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
` (3 preceding siblings ...)
2019-12-26 0:57 ` [Intel-gfx] [PATCH 5/5] drm/i915/gt: Apply sanitiization just before resume Chris Wilson
@ 2019-12-26 1:46 ` Patchwork
2019-12-26 2:37 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
5 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-12-26 1:46 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/5] drm/i915/gt: Ignore stale context state upon resume
URL : https://patchwork.freedesktop.org/series/71386/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
f9ba897e2ceb drm/i915/gt: Ignore stale context state upon resume
-:7: WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one
total: 0 errors, 1 warnings, 0 checks, 31 lines checked
9e6f35bd6180 drm/i915/gt: Discard stale context state from across idling
20a97072d51f drm/i915/gt: Always poison the kernel_context image before unparking
615fd746ceef drm/i915/gt: Restore coarse power gating for gen9
e73bb2efdf11 drm/i915/gt: Apply sanitiization just before resume
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/5] drm/i915/gt: Ignore stale context state upon resume
2019-12-26 0:56 [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
` (4 preceding siblings ...)
2019-12-26 1:46 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/5] drm/i915/gt: Ignore stale context state upon resume Patchwork
@ 2019-12-26 2:37 ` Patchwork
5 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-12-26 2:37 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: series starting with [1/5] drm/i915/gt: Ignore stale context state upon resume
URL : https://patchwork.freedesktop.org/series/71386/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_7635 -> Patchwork_15925
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_15925 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_15925, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_15925:
### IGT changes ###
#### Possible regressions ####
* igt@gem_ctx_switch@rcs0:
- fi-hsw-4770: [PASS][1] -> [FAIL][2] +3 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-hsw-4770/igt@gem_ctx_switch@rcs0.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-hsw-4770/igt@gem_ctx_switch@rcs0.html
* igt@gem_exec_create@basic:
- fi-hsw-4770r: NOTRUN -> [FAIL][3] +5 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-hsw-4770r/igt@gem_exec_create@basic.html
* igt@gem_exec_fence@basic-busy-default:
- fi-hsw-4770r: NOTRUN -> [TIMEOUT][4] +1 similar issue
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-hsw-4770r/igt@gem_exec_fence@basic-busy-default.html
* igt@gem_exec_fence@basic-wait-default:
- fi-hsw-4770: [PASS][5] -> [TIMEOUT][6] +1 similar issue
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-hsw-4770/igt@gem_exec_fence@basic-wait-default.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-hsw-4770/igt@gem_exec_fence@basic-wait-default.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-skl-6770hq: [PASS][7] -> [DMESG-WARN][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_pm_rpm@module-reload:
- fi-skl-6770hq: [PASS][9] -> [FAIL][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live_requests:
- fi-bdw-5557u: NOTRUN -> [INCOMPLETE][11]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-bdw-5557u/igt@i915_selftest@live_requests.html
* igt@runner@aborted:
- fi-hsw-4770: NOTRUN -> [FAIL][12]
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-hsw-4770/igt@runner@aborted.html
#### Warnings ####
* igt@i915_module_load@reload-with-fault-injection:
- fi-skl-6600u: [INCOMPLETE][13] ([i915#671] / [i915#69]) -> [DMESG-WARN][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-skl-6600u/igt@i915_module_load@reload-with-fault-injection.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-skl-6600u/igt@i915_module_load@reload-with-fault-injection.html
- fi-skl-lmem: [INCOMPLETE][15] ([i915#671]) -> [DMESG-WARN][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html
Known issues
------------
Here are the changes found in Patchwork_15925 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_suspend@basic-s0:
- fi-cml-s: [PASS][17] -> [FAIL][18] ([fdo#103375])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-cml-s/igt@gem_exec_suspend@basic-s0.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-cml-s/igt@gem_exec_suspend@basic-s0.html
* igt@i915_selftest@live_execlists:
- fi-kbl-soraka: [PASS][19] -> [DMESG-FAIL][20] ([i915#656])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-kbl-soraka/igt@i915_selftest@live_execlists.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-kbl-soraka/igt@i915_selftest@live_execlists.html
* igt@i915_selftest@live_gem_contexts:
- fi-cfl-guc: [PASS][21] -> [INCOMPLETE][22] ([fdo#106070] / [i915#424])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-cfl-guc/igt@i915_selftest@live_gem_contexts.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-cfl-guc/igt@i915_selftest@live_gem_contexts.html
#### Possible fixes ####
* igt@gem_close_race@basic-threads:
- fi-byt-j1900: [TIMEOUT][23] ([i915#816]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-byt-j1900/igt@gem_close_race@basic-threads.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-byt-j1900/igt@gem_close_race@basic-threads.html
* igt@gem_exec_suspend@basic-s3:
- fi-cml-s: [DMESG-WARN][25] ([fdo#111764]) -> [PASS][26]
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-cml-s/igt@gem_exec_suspend@basic-s3.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-cml-s/igt@gem_exec_suspend@basic-s3.html
* igt@i915_module_load@reload-with-fault-injection:
- fi-bxt-dsi: [INCOMPLETE][27] ([fdo#103927]) -> [PASS][28]
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_pm_rpm@module-reload:
- fi-icl-u3: [DMESG-WARN][29] ([i915#109]) -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-icl-u3/igt@i915_pm_rpm@module-reload.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-icl-u3/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live_workarounds:
- fi-icl-dsi: [INCOMPLETE][31] ([i915#140]) -> [PASS][32]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-icl-dsi/igt@i915_selftest@live_workarounds.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-icl-dsi/igt@i915_selftest@live_workarounds.html
#### Warnings ####
* igt@i915_module_load@reload-with-fault-injection:
- fi-kbl-x1275: [DMESG-WARN][33] ([i915#62] / [i915#92]) -> [INCOMPLETE][34] ([i915#879])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][35] ([i915#62] / [i915#92] / [i915#95]) -> [DMESG-WARN][36] ([i915#62] / [i915#92]) +5 similar issues
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-kbl-x1275/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_cursor_legacy@basic-flip-before-cursor-legacy:
- fi-kbl-x1275: [DMESG-WARN][37] ([i915#62] / [i915#92]) -> [DMESG-WARN][38] ([i915#62] / [i915#92] / [i915#95]) +1 similar issue
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7635/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/fi-kbl-x1275/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#106070]: https://bugs.freedesktop.org/show_bug.cgi?id=106070
[fdo#111735]: https://bugs.freedesktop.org/show_bug.cgi?id=111735
[fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764
[i915#109]: https://gitlab.freedesktop.org/drm/intel/issues/109
[i915#140]: https://gitlab.freedesktop.org/drm/intel/issues/140
[i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
[i915#656]: https://gitlab.freedesktop.org/drm/intel/issues/656
[i915#671]: https://gitlab.freedesktop.org/drm/intel/issues/671
[i915#69]: https://gitlab.freedesktop.org/drm/intel/issues/69
[i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816
[i915#879]: https://gitlab.freedesktop.org/drm/intel/issues/879
[i915#92]: https://gitlab.freedesktop.org/drm/intel/issues/92
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
Participating hosts (51 -> 38)
------------------------------
Additional (3): fi-hsw-4770r fi-icl-u2 fi-bdw-5557u
Missing (16): fi-ilk-m540 fi-kbl-7560u fi-hsw-4200u fi-hsw-peppy fi-byt-squawks fi-bsw-cyan fi-bwr-2160 fi-tgl-y fi-ctg-p8600 fi-gdg-551 fi-ivb-3770 fi-bsw-kefka fi-blb-e6850 fi-byt-n2820 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_7635 -> Patchwork_15925
CI-20190529: 20190529
CI_DRM_7635: 4d981e3511a45c244b31dedf719e74733ad81969 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5352: 0586d205f651674e575351c2d5a7d0760716c9f1 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_15925: e73bb2efdf112d6f4b0ef4433bd1017bbf6c76fe @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
e73bb2efdf11 drm/i915/gt: Apply sanitiization just before resume
615fd746ceef drm/i915/gt: Restore coarse power gating for gen9
20a97072d51f drm/i915/gt: Always poison the kernel_context image before unparking
9e6f35bd6180 drm/i915/gt: Discard stale context state from across idling
f9ba897e2ceb drm/i915/gt: Ignore stale context state upon resume
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_15925/index.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-12-26 2:37 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-26 0:56 [Intel-gfx] [PATCH 1/5] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
2019-12-26 0:56 ` [Intel-gfx] [PATCH 2/5] drm/i915/gt: Discard stale context state from across idling Chris Wilson
2019-12-26 0:56 ` [Intel-gfx] [PATCH 3/5] drm/i915/gt: Always poison the kernel_context image before unparking Chris Wilson
2019-12-26 0:57 ` [Intel-gfx] [PATCH 4/5] drm/i915/gt: Restore coarse power gating for gen9 Chris Wilson
2019-12-26 0:57 ` [Intel-gfx] [PATCH 5/5] drm/i915/gt: Apply sanitiization just before resume Chris Wilson
2019-12-26 1:46 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/5] drm/i915/gt: Ignore stale context state upon resume Patchwork
2019-12-26 2:37 ` [Intel-gfx] ✗ 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.