All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Wilson <chris@chris-wilson.co.uk>
To: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] [PATCH 1/7] drm/i915/gt: Ensure that all new contexts clear STOP_RING
Date: Sun, 29 Dec 2019 18:31:47 +0000	[thread overview]
Message-ID: <20191229183153.3719869-1-chris@chris-wilson.co.uk> (raw)

Set up the RING_MI_NODE in new contexts to clear the STOP_RING bit, just
in case they find it still set after a reset (as they are the first
contexts to be run).

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/gt/intel_lrc.c | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_lrc.c b/drivers/gpu/drm/i915/gt/intel_lrc.c
index 4fb70a7716e3..723914a41ef4 100644
--- a/drivers/gpu/drm/i915/gt/intel_lrc.c
+++ b/drivers/gpu/drm/i915/gt/intel_lrc.c
@@ -3030,10 +3030,8 @@ static void reset_csb_pointers(struct intel_engine_cs *engine)
 			       &execlists->csb_status[reset_value]);
 }
 
-static void __execlists_reset_reg_state(const struct intel_context *ce,
-					const struct intel_engine_cs *engine)
+static void __reset_stop_ring(u32 *regs, const struct intel_engine_cs *engine)
 {
-	u32 *regs = ce->lrc_reg_state;
 	int x;
 
 	x = lrc_ring_mi_mode(engine);
@@ -3043,6 +3041,14 @@ static void __execlists_reset_reg_state(const struct intel_context *ce,
 	}
 }
 
+static void __execlists_reset_reg_state(const struct intel_context *ce,
+					const struct intel_engine_cs *engine)
+{
+	u32 *regs = ce->lrc_reg_state;
+
+	__reset_stop_ring(regs, engine);
+}
+
 static void __execlists_reset(struct intel_engine_cs *engine, bool stalled)
 {
 	struct intel_engine_execlists * const execlists = &engine->execlists;
@@ -4043,6 +4049,8 @@ static void execlists_init_reg_state(u32 *regs,
 			     INTEL_GEN(engine->i915) >= 12 ?
 			     GEN12_CTX_BB_PER_CTX_PTR :
 			     CTX_BB_PER_CTX_PTR);
+
+	__reset_stop_ring(regs, engine);
 }
 
 static int
-- 
2.25.0.rc0

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

             reply	other threads:[~2019-12-29 18:32 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-29 18:31 Chris Wilson [this message]
2019-12-29 18:31 ` [Intel-gfx] [PATCH 2/7] drm/i915/gt: Avoid using tag 0 for the very first submission Chris Wilson
2019-12-30 13:43   ` Matthew Auld
2019-12-29 18:31 ` [Intel-gfx] [PATCH 3/7] drm/i915/gt: Avoid using the GPU before initialisation Chris Wilson
2019-12-30 13:58   ` Matthew Auld
2019-12-29 18:31 ` [Intel-gfx] [PATCH 4/7] drm/i915/gt: Do not restore invalid RS state Chris Wilson
2019-12-30 14:46   ` Matthew Auld
2019-12-29 18:31 ` [Intel-gfx] [PATCH 5/7] drm/i915/gt: Ignore stale context state upon resume Chris Wilson
2019-12-30 16:06   ` Matthew Auld
2019-12-30 16:12     ` Chris Wilson
2019-12-29 18:31 ` [Intel-gfx] [PATCH 6/7] drm/i915/gt: Discard stale context state from across idling Chris Wilson
2019-12-30 16:12   ` Matthew Auld
2019-12-29 18:31 ` [Intel-gfx] [PATCH 7/7] drm/i915/gt: Always poison the kernel_context image before unparking Chris Wilson
2019-12-30 16:16   ` Matthew Auld
2019-12-29 19:20 ` [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/7] drm/i915/gt: Ensure that all new contexts clear STOP_RING Patchwork
2019-12-29 20:50 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2019-12-29 22:27 ` [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/7] drm/i915/gt: Ensure that all new contexts clear STOP_RING (rev2) Patchwork
2019-12-30  0:12 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2019-12-30 11:01 ` [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/7] drm/i915/gt: Ensure that all new contexts clear STOP_RING (rev3) Patchwork
2019-12-30 13:33 ` [Intel-gfx] [PATCH 1/7] drm/i915/gt: Ensure that all new contexts clear STOP_RING Matthew Auld
2019-12-30 14:03 ` [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/7] drm/i915/gt: Ensure that all new contexts clear STOP_RING (rev4) Patchwork
2019-12-30 21:07 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191229183153.3719869-1-chris@chris-wilson.co.uk \
    --to=chris@chris-wilson.co.uk \
    --cc=intel-gfx@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.