Commit 624bb0c0 authored by Eric Anholt's avatar Eric Anholt

drm/v3d: Delay the scheduler timeout if we're still making progress.

GTF-GLES2.gtf.GL.acos.acos_float_vert_xvary submits jobs that take 4
seconds at maximum resolution, but we still want to reset quickly if a
job is really hung.  Sample the CL's current address and the return
address (since we call into tile lists repeatedly) and if either has
changed then assume we've made progress.
Signed-off-by: default avatarEric Anholt <eric@anholt.net>
Cc: Lucas Stach <l.stach@pengutronix.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20180703170515.6298-1-eric@anholt.netReviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Acked-by: default avatarDaniel Vetter <daniel@ffwll.ch>
parent 408633d2
...@@ -184,6 +184,8 @@ struct v3d_job { ...@@ -184,6 +184,8 @@ struct v3d_job {
/* GPU virtual addresses of the start/end of the CL job. */ /* GPU virtual addresses of the start/end of the CL job. */
u32 start, end; u32 start, end;
u32 timedout_ctca, timedout_ctra;
}; };
struct v3d_exec_info { struct v3d_exec_info {
......
...@@ -222,6 +222,7 @@ ...@@ -222,6 +222,7 @@
#define V3D_CLE_CTNCA(n) (V3D_CLE_CT0CA + 4 * n) #define V3D_CLE_CTNCA(n) (V3D_CLE_CT0CA + 4 * n)
#define V3D_CLE_CT0RA 0x00118 #define V3D_CLE_CT0RA 0x00118
#define V3D_CLE_CT1RA 0x0011c #define V3D_CLE_CT1RA 0x0011c
#define V3D_CLE_CTNRA(n) (V3D_CLE_CT0RA + 4 * n)
#define V3D_CLE_CT0LC 0x00120 #define V3D_CLE_CT0LC 0x00120
#define V3D_CLE_CT1LC 0x00124 #define V3D_CLE_CT1LC 0x00124
#define V3D_CLE_CT0PC 0x00128 #define V3D_CLE_CT0PC 0x00128
......
...@@ -153,7 +153,25 @@ v3d_job_timedout(struct drm_sched_job *sched_job) ...@@ -153,7 +153,25 @@ v3d_job_timedout(struct drm_sched_job *sched_job)
struct v3d_job *job = to_v3d_job(sched_job); struct v3d_job *job = to_v3d_job(sched_job);
struct v3d_exec_info *exec = job->exec; struct v3d_exec_info *exec = job->exec;
struct v3d_dev *v3d = exec->v3d; struct v3d_dev *v3d = exec->v3d;
enum v3d_queue job_q = job == &exec->bin ? V3D_BIN : V3D_RENDER;
enum v3d_queue q; enum v3d_queue q;
u32 ctca = V3D_CORE_READ(0, V3D_CLE_CTNCA(job_q));
u32 ctra = V3D_CORE_READ(0, V3D_CLE_CTNRA(job_q));
/* If the current address or return address have changed, then
* the GPU has probably made progress and we should delay the
* reset. This could fail if the GPU got in an infinite loop
* in the CL, but that is pretty unlikely outside of an i-g-t
* testcase.
*/
if (job->timedout_ctca != ctca || job->timedout_ctra != ctra) {
job->timedout_ctca = ctca;
job->timedout_ctra = ctra;
schedule_delayed_work(&job->base.work_tdr,
job->base.sched->timeout);
return;
}
mutex_lock(&v3d->reset_lock); mutex_lock(&v3d->reset_lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment