Hi all, Am Montag, den 24.08.2020, 11:11 -0300 schrieb Fabio Estevam: > 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.") I've added this patch with the above fixed and Russell's T-b to my etnaviv/fixes branch. Regards, Lucas _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel