Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes:
> After a GPU reset, we want to replay our queue of requests. However, the
> GPU reset clobbered the state and we only fixup the state for the
> guitly
s/guitly/guilty.
Reviewed-by: Mika Kuoppala <mika.kuoppala@xxxxxxxxx>
> request - and engines deemed innocent we try to leave untouched so that
> we recover as completely as possible. However, we need to clear the sw
> tracking of the ELSP ports even for innocent requests, so move the clear
> to the common path of init_hw (from reset_hw).
>
> Reported-by: Mika Kuoppala <mika.kuoppala@xxxxxxxxx>
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/intel_lrc.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
> index 0ea992ba2723..936f6f63f626 100644
> --- a/drivers/gpu/drm/i915/intel_lrc.c
> +++ b/drivers/gpu/drm/i915/intel_lrc.c
> @@ -1242,8 +1242,12 @@ static int gen8_init_common_ring(struct intel_engine_cs *engine)
>
> intel_engine_init_hangcheck(engine);
>
> - if (!execlists_elsp_idle(engine))
> + /* After a GPU reset, we may have requests to replay */
> + if (!execlists_elsp_idle(engine)) {
> + engine->execlist_port[0].count = 0;
> + engine->execlist_port[1].count = 0;
> execlists_submit_ports(engine);
> + }
>
> return 0;
> }
> @@ -1318,10 +1322,7 @@ static void reset_common_ring(struct intel_engine_cs *engine,
> memset(&port[1], 0, sizeof(port[1]));
> }
>
> - /* CS is stopped, and we will resubmit both ports on resume */
> GEM_BUG_ON(request->ctx != port[0].request->ctx);
> - port[0].count = 0;
> - port[1].count = 0;
>
> /* Reset WaIdleLiteRestore:bdw,skl as well */
> request->tail = request->wa_tail - WA_TAIL_DWORDS * sizeof(u32);
> --
> 2.9.3
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
