Re: Fw: [PATCH] drm/amdgpu: add amdgpu_timeout_ring_* file to debugfs

2023-06-15 Thread Alex Deucher
On Thu, Jun 15, 2023 at 5:14 AM Christian König wrote: > > Am 15.06.23 um 10:55 schrieb Nicolai Hähnle: > > On Thu, Jun 15, 2023 at 9:47 AM Christian König > > wrote: > Uh, that's very dangerous what you do here and wouldn't work in a whole > bunch of cases. > >>> Please elaborate: *wha

Re: Fw: [PATCH] drm/amdgpu: add amdgpu_timeout_ring_* file to debugfs

2023-06-15 Thread Christian König
Am 15.06.23 um 10:55 schrieb Nicolai Hähnle: On Thu, Jun 15, 2023 at 9:47 AM Christian König wrote: Uh, that's very dangerous what you do here and wouldn't work in a whole bunch of cases. Please elaborate: *what* case doesn't work? The core memory management can wait at any time for the GPU r

Re: Fw: [PATCH] drm/amdgpu: add amdgpu_timeout_ring_* file to debugfs

2023-06-15 Thread Nicolai Hähnle
On Thu, Jun 15, 2023 at 9:47 AM Christian König wrote: > >> Uh, that's very dangerous what you do here and wouldn't work in a whole > >> bunch of cases. > > Please elaborate: *what* case doesn't work? > > The core memory management can wait at any time for the GPU reset to finish. > > If we set th

Re: Fw: [PATCH] drm/amdgpu: add amdgpu_timeout_ring_* file to debugfs

2023-06-15 Thread Christian König
Am 14.06.23 um 21:20 schrieb Nicolai Hähnle: Hi Christian, Report the per-ring timeout in milliseconds and allow users to adjust the timeout dynamically. This can be useful for debugging, e.g. to more easily test whether a submission genuinely hangs or is just taking very long, and to temporari

Re: Fw: [PATCH] drm/amdgpu: add amdgpu_timeout_ring_* file to debugfs

2023-06-14 Thread Nicolai Hähnle
Hi Christian, > > Report the per-ring timeout in milliseconds and allow users to adjust > > the timeout dynamically. This can be useful for debugging, e.g. to more > > easily test whether a submission genuinely hangs or is just taking very > > long, and to temporarily disable GPU recovery so that