Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Am 24.04.2018 um 17:30 schrieb Andrey Grodzovsky: If the ring is hanging for some reason allow to recover the waiting by sending fatal signal. Originally-by: David Panariti Signed-off-by: Andrey Grodzovsky ---

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Am 24.04.2018 um 17:30 schrieb Andrey Grodzovsky: If the ring is hanging for some reason allow to recover the waiting by sending fatal signal. Originally-by: David Panariti Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 14 ++ 1 file changed, 10

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/25/2018 04:55 PM, Eric W. Biederman wrote: Andrey Grodzovsky writes: On 04/24/2018 12:30 PM, Eric W. Biederman wrote: "Panariti, David" writes: Andrey Grodzovsky writes: Kind of

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/25/2018 04:55 PM, Eric W. Biederman wrote: Andrey Grodzovsky writes: On 04/24/2018 12:30 PM, Eric W. Biederman wrote: "Panariti, David" writes: Andrey Grodzovsky writes: Kind of dma_fence_wait_killable, except that we don't have such API (maybe worth adding ?) Depends on how

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Andrey Grodzovsky writes: > On 04/24/2018 12:30 PM, Eric W. Biederman wrote: >> "Panariti, David" writes: >> >>> Andrey Grodzovsky writes: Kind of dma_fence_wait_killable, except that we don't have such API

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Andrey Grodzovsky writes: > On 04/24/2018 12:30 PM, Eric W. Biederman wrote: >> "Panariti, David" writes: >> >>> Andrey Grodzovsky writes: Kind of dma_fence_wait_killable, except that we don't have such API (maybe worth adding ?) >>> Depends on how many places it would be called, or

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/24/2018 12:30 PM, Eric W. Biederman wrote: "Panariti, David" writes: Andrey Grodzovsky writes: Kind of dma_fence_wait_killable, except that we don't have such API (maybe worth adding ?) Depends on how many places it would be

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/24/2018 12:30 PM, Eric W. Biederman wrote: "Panariti, David" writes: Andrey Grodzovsky writes: Kind of dma_fence_wait_killable, except that we don't have such API (maybe worth adding ?) Depends on how many places it would be called, or think it might be called. Can always factor

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/24/2018 12:14 PM, Eric W. Biederman wrote: Andrey Grodzovsky writes: If the ring is hanging for some reason allow to recover the waiting by sending fatal signal. Originally-by: David Panariti Signed-off-by: Andrey Grodzovsky

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/24/2018 12:14 PM, Eric W. Biederman wrote: Andrey Grodzovsky writes: If the ring is hanging for some reason allow to recover the waiting by sending fatal signal. Originally-by: David Panariti Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 14

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

"Panariti, David" writes: > Andrey Grodzovsky writes: >> Kind of dma_fence_wait_killable, except that we don't have such API >> (maybe worth adding ?) > Depends on how many places it would be called, or think it might be called. > Can always

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

"Panariti, David" writes: > Andrey Grodzovsky writes: >> Kind of dma_fence_wait_killable, except that we don't have such API >> (maybe worth adding ?) > Depends on how many places it would be called, or think it might be called. > Can always factor on the 2nd time it's needed. > Factoring,

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

:19 AM To: Panariti, David; linux-kernel@vger.kernel.org; amd-...@lists.freedesktop.org Cc: Deucher, Alexander; Koenig, Christian; o...@redhat.com; a...@linux-foundation.org; ebied...@xmission.com Subject: Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang. On

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

:19 AM To: Panariti, David; linux-kernel@vger.kernel.org; amd-...@lists.freedesktop.org Cc: Deucher, Alexander; Koenig, Christian; o...@redhat.com; a...@linux-foundation.org; ebied...@xmission.com Subject: Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang. On

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Andrey Grodzovsky writes: > If the ring is hanging for some reason allow to recover the waiting > by sending fatal signal. > > Originally-by: David Panariti > Signed-off-by: Andrey Grodzovsky > --- >

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Andrey Grodzovsky writes: > If the ring is hanging for some reason allow to recover the waiting > by sending fatal signal. > > Originally-by: David Panariti > Signed-off-by: Andrey Grodzovsky > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 14 ++ > 1 file changed, 10

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/24/2018 11:52 AM, Panariti, David wrote: Hi, It looks like there can be an infinite loop if neither of the if()'s become true. Is that an impossible condition? That intended, we want to wait until either the fence signals or fatal signal received, we don't want to terminate the wait

Re: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

On 04/24/2018 11:52 AM, Panariti, David wrote: Hi, It looks like there can be an infinite loop if neither of the if()'s become true. Is that an impossible condition? That intended, we want to wait until either the fence signals or fatal signal received, we don't want to terminate the wait

RE: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Hi, It looks like there can be an infinite loop if neither of the if()'s become true. Is that an impossible condition? -Original Message- From: Andrey Grodzovsky Sent: Tuesday, April 24, 2018 11:31 AM To: linux-kernel@vger.kernel.org;

RE: [PATCH 3/3] drm/amdgpu: Switch to interrupted wait to recover from ring hang.

Hi, It looks like there can be an infinite loop if neither of the if()'s become true. Is that an impossible condition? -Original Message- From: Andrey Grodzovsky Sent: Tuesday, April 24, 2018 11:31 AM To: linux-kernel@vger.kernel.org; amd-...@lists.freedesktop.org Cc: Deucher,