Hi Lucas, On Mon, Aug 24, 2020 at 8:02 AM Lucas Stach <l.stach@xxxxxxxxxxxxxx> wrote: > > 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.) Just a nit: the correct syntax for the Fixes line is: Fixes: 135517d3565b ("drm/scheduler: Avoid accessing freed bad job.") _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel