Re: [PATCH 2/2] drm/scheduler: remove timeout work_struct from drm_sched_job

2018-09-20 Thread Christian König
Am 20.09.2018 um 13:25 schrieb Nayan Deshmukh: On Wed, Sep 19, 2018, 9:31 PM Christian König mailto:christian.koe...@amd.com>> wrote: Am 18.09.2018 um 18:17 schrieb Nayan Deshmukh: > having a delayed work item per job is redundant as we only need one > per scheduler to track the

Re: [PATCH 2/2] drm/scheduler: remove timeout work_struct from drm_sched_job

2018-09-20 Thread Nayan Deshmukh
On Wed, Sep 19, 2018, 9:31 PM Christian König wrote: > Am 18.09.2018 um 18:17 schrieb Nayan Deshmukh: > > having a delayed work item per job is redundant as we only need one > > per scheduler to track the time out the currently executing job. > > Well that looks simpler than I thought it would be

Re: [PATCH 2/2] drm/scheduler: remove timeout work_struct from drm_sched_job

2018-09-19 Thread Christian König
Am 19.09.2018 um 17:39 schrieb Michel Dänzer: On 2018-09-19 2:30 p.m., Christian König wrote: Am 18.09.2018 um 18:17 schrieb Nayan Deshmukh: having a delayed work item per job is redundant as we only need one per scheduler to track the time out the currently executing job. Well that looks simp

Re: [PATCH 2/2] drm/scheduler: remove timeout work_struct from drm_sched_job

2018-09-19 Thread Michel Dänzer
On 2018-09-19 2:30 p.m., Christian König wrote: > Am 18.09.2018 um 18:17 schrieb Nayan Deshmukh: >> having a delayed work item per job is redundant as we only need one >> per scheduler to track the time out the currently executing job. > > Well that looks simpler than I thought it would be. > > B

Re: [PATCH 2/2] drm/scheduler: remove timeout work_struct from drm_sched_job

2018-09-19 Thread Christian König
Am 18.09.2018 um 18:17 schrieb Nayan Deshmukh: having a delayed work item per job is redundant as we only need one per scheduler to track the time out the currently executing job. Well that looks simpler than I thought it would be. But it shows the next problem that the timeout and the complet

[PATCH 2/2] drm/scheduler: remove timeout work_struct from drm_sched_job

2018-09-18 Thread Nayan Deshmukh
having a delayed work item per job is redundant as we only need one per scheduler to track the time out the currently executing job. Signed-off-by: Nayan Deshmukh Suggested-by: Christian König --- drivers/gpu/drm/scheduler/sched_main.c | 16 +--- include/drm/gpu_scheduler.h