path: root/drivers/gpu/drm/msm/dsi
diff options
authorSteven Price <>2019-10-09 10:44:55 +0100
committerRob Herring <>2019-10-15 11:38:22 -0500
commit5b3ec8134f5f9fa1ed0a538441a495521078bbee (patch)
treef6ef968ec57b77e4c8f44d388b4cc427a1ae347b /drivers/gpu/drm/msm/dsi
parenteda6d764ac33d9ce8d656fa381f05b3feae09085 (diff)
drm/panfrost: Handle resetting on timeout better
Panfrost uses multiple schedulers (one for each slot, so 2 in reality), and on a timeout has to stop all the schedulers to safely perform a reset. However more than one scheduler can trigger a timeout at the same time. This race condition results in jobs being freed while they are still in use. When stopping other slots use cancel_delayed_work_sync() to ensure that any timeout started for that slot has completed. Also use mutex_trylock() to obtain reset_lock. This means that only one thread attempts the reset, the other threads will simply complete without doing anything (the first thread will wait for this in the call to cancel_delayed_work_sync()). While we're here and since the function is already dependent on sched_job not being NULL, let's remove the unnecessary checks. Fixes: aa20236784ab ("drm/panfrost: Prevent concurrent resets") Tested-by: Neil Armstrong <> Signed-off-by: Steven Price <> Cc: Signed-off-by: Rob Herring <> Link:
Diffstat (limited to 'drivers/gpu/drm/msm/dsi')
0 files changed, 0 insertions, 0 deletions