RE: [PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened

2020-12-10 Thread Liu, Cheng Zhe
@lists.freedesktop.org Cc: Xiao, Jack ; Xu, Feifei ; Wang, Kevin(Yang) ; Tuikov, Luben ; Deucher, Alexander ; Yuan, Xiaojie ; Zhang, Hawking Subject: Re: [PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened Hi Cheng, well with whom you discussed that? Such discussions should happen

Re: [PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened

2020-12-10 Thread Christian König
, 2020 6:06 PM To: Liu, Cheng Zhe ; amd-gfx@lists.freedesktop.org Cc: Tuikov, Luben ; Deucher, Alexander ; Xiao, Jack ; Zhang, Hawking ; Xu, Feifei ; Wang, Kevin(Yang) ; Yuan, Xiaojie Subject: Re: [PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened Am 09.12.20 um 10:46 schrieb Liu

RE: [PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened

2020-12-09 Thread Liu, Cheng Zhe
-gfx@lists.freedesktop.org Cc: Tuikov, Luben ; Deucher, Alexander ; Xiao, Jack ; Zhang, Hawking ; Xu, Feifei ; Wang, Kevin(Yang) ; Yuan, Xiaojie Subject: Re: [PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened Am 09.12.20 um 10:46 schrieb Liu ChengZhe: > If CS init return -ECANCE

Re: [PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened

2020-12-09 Thread Christian König
Am 09.12.20 um 10:46 schrieb Liu ChengZhe: If CS init return -ECANCELED, UMD will free and create new context. Job in this new context could conitnue exexcuting. In the case of BACO or mode 1, we can't allow this happpen. Because VRAM has lost after whole gpu reset, the job can't guarantee to

[PATCH] drm/amdgpu: Return -EINVAL when whole gpu reset happened

2020-12-09 Thread Liu ChengZhe
If CS init return -ECANCELED, UMD will free and create new context. Job in this new context could conitnue exexcuting. In the case of BACO or mode 1, we can't allow this happpen. Because VRAM has lost after whole gpu reset, the job can't guarantee to succeed. Signed-off-by: Liu ChengZhe ---