Re: [PATCH 2/2] drm/scheduler: add last_scheduled dependency handling

2018-08-06 Thread Christian König
Am 06.08.2018 um 15:23 schrieb Nayan Deshmukh: Hi Christian, Good patch. But it might lead to bad performance when we reschedule when the hardware queue of the engine on which the last job was pushed is not full. On Mon, Aug 6, 2018 at 5:49 PM Christian König

Re: [PATCH 2/2] drm/scheduler: add last_scheduled dependency handling

2018-08-06 Thread Nayan Deshmukh
Hi Christian, Good patch. But it might lead to bad performance when we reschedule when the hardware queue of the engine on which the last job was pushed is not full. On Mon, Aug 6, 2018 at 5:49 PM Christian König < ckoenig.leichtzumer...@gmail.com> wrote: > This fixes accessing the last_schedule

[PATCH 2/2] drm/scheduler: add last_scheduled dependency handling

2018-08-06 Thread Christian König
This fixes accessing the last_scheduled fence from multiple threads as well as makes it easier to move entities between schedulers. Signed-off-by: Christian König --- drivers/gpu/drm/scheduler/gpu_scheduler.c | 34 +-- 1 file changed, 28 insertions(+), 6 deletions(-)