drm/etnaviv: always start/stop scheduler in timeout processing
authorLucas Stach <l.stach@pengutronix.de>
Mon, 24 Aug 2020 10:55:37 +0000 (12:55 +0200)
committerLucas Stach <l.stach@pengutronix.de>
Mon, 24 Aug 2020 15:21:21 +0000 (17:21 +0200)
commit50248a3ec0f5e5debd18033eb2a29f0b793a7000
treef1d6b1b4d72ee860a401807af217ea5474b4e3c1
parent2c5bf028ef34745e7b3fe768f9c9355ecc7df101
drm/etnaviv: always start/stop scheduler in timeout processing

The drm scheduler currently expects that the stop/start sequence is always
executed in the timeout handling, as the job at the head of the hardware
execution list is always removed from the ring mirror before the driver
function is called and only inserted back into the list when starting the
scheduler.

This adds some unnecessary overhead if the timeout handler determines
that the GPU is still executing jobs normally and just wished to extend
the timeout, but a better solution requires a major rearchitecture of the
scheduler, which is not applicable as a fix.

Fixes: 135517d3565b ("drm/scheduler: Avoid accessing freed bad job.")
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Tested-by: Russell King <rmk+kernel@armlinux.org.uk>
drivers/gpu/drm/etnaviv/etnaviv_sched.c