Re: [PATCH v2] drm/scheduler: Fix job cleanup without timeout handler

2019-05-21 Thread Koenig, Christian
Am 21.05.19 um 16:13 schrieb Alex Deucher: > [CAUTION: External Email] > > On Tue, May 21, 2019 at 2:48 AM Koenig, Christian > wrote: >> Am 21.05.19 um 01:16 schrieb Erico Nunes: >>> [CAUTION: External Email] >>> >>> After "5918045c4ed4 drm/scheduler: rework job destruction", jobs are >>> only del

Re: [PATCH v2] drm/scheduler: Fix job cleanup without timeout handler

2019-05-21 Thread Alex Deucher
On Tue, May 21, 2019 at 2:48 AM Koenig, Christian wrote: > > Am 21.05.19 um 01:16 schrieb Erico Nunes: > > [CAUTION: External Email] > > > > After "5918045c4ed4 drm/scheduler: rework job destruction", jobs are > > only deleted when the timeout handler is able to be cancelled > > successfully. > >

Re: [PATCH v2] drm/scheduler: Fix job cleanup without timeout handler

2019-05-21 Thread Qiang Yu
On Tue, May 21, 2019 at 8:17 PM Erico Nunes wrote: > > On Tue, May 21, 2019 at 8:47 AM Koenig, Christian > wrote: > > > > Am 21.05.19 um 01:16 schrieb Erico Nunes: > > > [CAUTION: External Email] > > > > > > After "5918045c4ed4 drm/scheduler: rework job destruction", jobs are > > > only deleted w

Re: [PATCH v2] drm/scheduler: Fix job cleanup without timeout handler

2019-05-21 Thread Koenig, Christian
Am 21.05.19 um 14:16 schrieb Erico Nunes: > [CAUTION: External Email] > > On Tue, May 21, 2019 at 8:47 AM Koenig, Christian > wrote: >> Am 21.05.19 um 01:16 schrieb Erico Nunes: >>> [CAUTION: External Email] >>> >>> After "5918045c4ed4 drm/scheduler: rework job destruction", jobs are >>> only dele

Re: [PATCH v2] drm/scheduler: Fix job cleanup without timeout handler

2019-05-21 Thread Erico Nunes
On Tue, May 21, 2019 at 8:47 AM Koenig, Christian wrote: > > Am 21.05.19 um 01:16 schrieb Erico Nunes: > > [CAUTION: External Email] > > > > After "5918045c4ed4 drm/scheduler: rework job destruction", jobs are > > only deleted when the timeout handler is able to be cancelled > > successfully. > >

Re: [PATCH v2] drm/scheduler: Fix job cleanup without timeout handler

2019-05-20 Thread Koenig, Christian
Am 21.05.19 um 01:16 schrieb Erico Nunes: > [CAUTION: External Email] > > After "5918045c4ed4 drm/scheduler: rework job destruction", jobs are > only deleted when the timeout handler is able to be cancelled > successfully. > > In case no timeout handler is running (timeout == MAX_SCHEDULE_TIMEOUT),

[PATCH v2] drm/scheduler: Fix job cleanup without timeout handler

2019-05-20 Thread Erico Nunes
After "5918045c4ed4 drm/scheduler: rework job destruction", jobs are only deleted when the timeout handler is able to be cancelled successfully. In case no timeout handler is running (timeout == MAX_SCHEDULE_TIMEOUT), job cleanup would be skipped which may result in memory leaks. Add the handling