Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Christian König
Hi Amarnath, not a bad idea, but that won't work either because you really need to return to make sure that a potential next reset can run. What could be done is to have a work item which does that, but then I think it would just be easier to teach the udev function a GFP mask. Regards, Chr

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Somalapuram, Amaranath
Hi Christian, if sending udev event during reset is going to create problem, we can move this code from reset sequence to re-int  (after GPU reset succeeded). Regards, S.Amarnath On 1/17/2022 5:27 PM, Christian König wrote: Am 17.01.22 um 12:43 schrieb Sharma, Shashank: Hello Christian, O

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Christian König
Am 17.01.22 um 12:43 schrieb Sharma, Shashank: Hello Christian, On 1/17/2022 11:37 AM, Christian König wrote: Am 17.01.22 um 11:34 schrieb Somalapuram, Amaranath: [SNIP] Any suggestion how we can notify user space during this situation. Well trace points should work. They use a ring buffer

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Lazar, Lijo
On 1/17/2022 12:03 PM, Somalapuram Amaranath wrote: AMDGPURESET uevent added to notify userspace, collect dump_stack and trace Signed-off-by: Somalapuram Amaranath --- drivers/gpu/drm/amd/amdgpu/nv.c | 45 + 1 file changed, 45 insertions(+) diff --git a/dr

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Sharma, Shashank
Hello Christian, On 1/17/2022 11:37 AM, Christian König wrote: Am 17.01.22 um 11:34 schrieb Somalapuram, Amaranath: [SNIP] Any suggestion how we can notify user space during this situation. Well trace points should work. They use a ring buffer and are specially made to work in such situatio

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Christian König
Am 17.01.22 um 11:34 schrieb Somalapuram, Amaranath: [SNIP] Any suggestion how we can notify user space during this situation. Well trace points should work. They use a ring buffer and are specially made to work in such situations. Alternative would be to audit the udev code and allow givin

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Somalapuram, Amaranath
, Alexander ; Sharma, Shashank Subject: Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset Am 17.01.22 um 11:01 schrieb Somalapuram, Amaranath: [AMD Official Use Only] -Original Message- From: Koenig, Christian Sent: Monday, January 17, 2022 12:57 PM To: Somalapuram

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Christian König
, Alexander ; Sharma, Shashank Subject: Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset Am 17.01.22 um 07:33 schrieb Somalapuram Amaranath: AMDGPURESET uevent added to notify userspace, collect dump_stack and trace Signed-off-by: Somalapuram Amaranath --- drivers/gpu/drm

RE: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Somalapuram, Amaranath
, Alexander ; Sharma, Shashank > > Subject: Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU > reset > > Am 17.01.22 um 07:33 schrieb Somalapuram Amaranath: >> AMDGPURESET uevent added to notify userspace, collect dump_stack and >> trace >> >> S

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Christian König
2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset Am 17.01.22 um 07:33 schrieb Somalapuram Amaranath: AMDGPURESET uevent added to notify userspace, collect dump_stack and trace Signed-off-by: Somalapuram Amaranath --- drivers/gpu/drm/amd/amdgpu/nv.c | 45

RE: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-17 Thread Somalapuram, Amaranath
[AMD Official Use Only] -Original Message- From: Koenig, Christian Sent: Monday, January 17, 2022 12:57 PM To: Somalapuram, Amaranath ; amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander ; Sharma, Shashank Subject: Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU

Re: [PATCH 2/2] drm/amdgpu: add AMDGPURESET uevent on AMD GPU reset

2022-01-16 Thread Christian König
Am 17.01.22 um 07:33 schrieb Somalapuram Amaranath: AMDGPURESET uevent added to notify userspace, collect dump_stack and trace Signed-off-by: Somalapuram Amaranath --- drivers/gpu/drm/amd/amdgpu/nv.c | 45 + 1 file changed, 45 insertions(+) diff --git a/drive