drm/msm/a6xx: Clear shadow on suspend
authorRob Clark <robdclark@chromium.org>
Tue, 10 Nov 2020 18:23:06 +0000 (10:23 -0800)
committerRob Clark <robdclark@chromium.org>
Tue, 10 Nov 2020 19:05:23 +0000 (11:05 -0800)
commite8b0b994c3a5881f0648d53f90435120089c56ad
treea4064b74ce1c33fc5d7036a4f0c55e6905817830
parent06d65ba3db67618c0afd70a72df516dace49e7e7
drm/msm/a6xx: Clear shadow on suspend

Clear the shadow rptr on suspend.  Otherwise, when we resume, we can
have a stale value until CP_WHERE_AM_I executes.  If we suspend near
the ringbuffer wraparound point, this can lead to a chicken/egg
situation where we are waiting for ringbuffer space to write the
CP_WHERE_AM_I (or CP_INIT) packet, because we mistakenly believe that
the ringbuffer is full (due to stale rptr value in the shadow).

Fixes errors like:

  [drm:adreno_wait_ring [msm]] *ERROR* timeout waiting for space in ringbuffer 0

in the resume path.

Fixes: d3a569fccfa0 ("drm/msm: a6xx: Use WHERE_AM_I for eligible targets")
Signed-off-by: Rob Clark <robdclark@chromium.org>
drivers/gpu/drm/msm/adreno/a6xx_gpu.c