Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-24 Thread Luben Tuikov
On 2020-11-24 12:40 p.m., Christian König wrote: > Am 24.11.20 um 18:11 schrieb Luben Tuikov: >> On 2020-11-24 2:50 a.m., Christian König wrote: >>> Am 24.11.20 um 02:12 schrieb Luben Tuikov: On 2020-11-23 3:06 a.m., Christian König wrote: > Am 23.11.20 um 06:37 schrieb Andrey Grodzovsky:

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-24 Thread Luben Tuikov
On 2020-11-24 12:17 p.m., Andrey Grodzovsky wrote: > > On 11/24/20 12:11 PM, Luben Tuikov wrote: >> On 2020-11-24 2:50 a.m., Christian König wrote: >>> Am 24.11.20 um 02:12 schrieb Luben Tuikov: On 2020-11-23 3:06 a.m., Christian König wrote: > Am 23.11.20 um 06:37 schrieb Andrey

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-24 Thread Christian König
Am 24.11.20 um 18:11 schrieb Luben Tuikov: On 2020-11-24 2:50 a.m., Christian König wrote: Am 24.11.20 um 02:12 schrieb Luben Tuikov: On 2020-11-23 3:06 a.m., Christian König wrote: Am 23.11.20 um 06:37 schrieb Andrey Grodzovsky: On 11/22/20 6:57 AM, Christian König wrote: Am 21.11.20 um

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-24 Thread Andrey Grodzovsky
On 11/24/20 12:11 PM, Luben Tuikov wrote: On 2020-11-24 2:50 a.m., Christian König wrote: Am 24.11.20 um 02:12 schrieb Luben Tuikov: On 2020-11-23 3:06 a.m., Christian König wrote: Am 23.11.20 um 06:37 schrieb Andrey Grodzovsky: On 11/22/20 6:57 AM, Christian König wrote: Am 21.11.20 um

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-24 Thread Luben Tuikov
On 2020-11-24 2:50 a.m., Christian König wrote: > Am 24.11.20 um 02:12 schrieb Luben Tuikov: >> On 2020-11-23 3:06 a.m., Christian König wrote: >>> Am 23.11.20 um 06:37 schrieb Andrey Grodzovsky: On 11/22/20 6:57 AM, Christian König wrote: > Am 21.11.20 um 06:21 schrieb Andrey Grodzovsky:

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-23 Thread Christian König
Am 24.11.20 um 02:12 schrieb Luben Tuikov: On 2020-11-23 3:06 a.m., Christian König wrote: Am 23.11.20 um 06:37 schrieb Andrey Grodzovsky: On 11/22/20 6:57 AM, Christian König wrote: Am 21.11.20 um 06:21 schrieb Andrey Grodzovsky: No point to try recovery if device is gone, it's meaningless.

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-23 Thread Luben Tuikov
On 2020-11-23 3:06 a.m., Christian König wrote: > Am 23.11.20 um 06:37 schrieb Andrey Grodzovsky: >> >> On 11/22/20 6:57 AM, Christian König wrote: >>> Am 21.11.20 um 06:21 schrieb Andrey Grodzovsky: No point to try recovery if device is gone, it's meaningless. >>> >>> I think that this

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-23 Thread Christian König
Am 23.11.20 um 06:37 schrieb Andrey Grodzovsky: On 11/22/20 6:57 AM, Christian König wrote: Am 21.11.20 um 06:21 schrieb Andrey Grodzovsky: No point to try recovery if device is gone, it's meaningless. I think that this should go into the device specific recovery function and not in the

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-22 Thread Andrey Grodzovsky
On 11/22/20 6:57 AM, Christian König wrote: Am 21.11.20 um 06:21 schrieb Andrey Grodzovsky: No point to try recovery if device is gone, it's meaningless. I think that this should go into the device specific recovery function and not in the scheduler. The timeout timer is rearmed here, so

Re: [PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-22 Thread Christian König
Am 21.11.20 um 06:21 schrieb Andrey Grodzovsky: No point to try recovery if device is gone, it's meaningless. I think that this should go into the device specific recovery function and not in the scheduler. Christian. Signed-off-by: Andrey Grodzovsky ---

[PATCH v3 07/12] drm/sched: Prevent any job recoveries after device is unplugged.

2020-11-20 Thread Andrey Grodzovsky
No point to try recovery if device is gone, it's meaningless. Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | 2 +- drivers/gpu/drm/etnaviv/etnaviv_sched.c | 3 ++- drivers/gpu/drm/lima/lima_sched.c | 3 ++- drivers/gpu/drm/panfrost/panfrost_job.c