]> git.baikalelectronics.ru Git - kernel.git/commit
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)
commitcefca9ac3fc837e0e03e3104f4bbfac904324f34
treef1d6b1b4d72ee860a401807af217ea5474b4e3c1
parent727ee898f19f8fb5848a82be755f381bd1c4be4b
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: f348fdccc541 ("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