<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body bgcolor="white" lang="ZH-CN" link="blue" vlink="purple"> <div class="WordSection1"> Any updates for the rest patches ?<o:p></o:p> <o:p> </o:p> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Koenig, Christian Sent: 2017年10月4日 18:56 To: Liu, Monk <Monk.Liu@amd.com>; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 12/12] drm/amdgpu/sriov:no shadow buffer recovery<o:p></o:p> </div> </div> <o:p> </o:p> <div> Ah! Sorry, my fault. I've missed the "no" and thought you wanted to abandon all processing because VRAM is always lost. Going to review the remaining patches today. Christian. Am 04.10.2017 um 11:41 schrieb Liu, Monk:<o:p></o:p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <div> <div> Why ? the page tables are resided in VRAM, no need to recovery if no VRAM lost<o:p></o:p> <o:p></o:p> Sent from <a href="https://go.microsoft.com/fwlink/?LinkId=550986"> Mail</a> for Windows 10<o:p></o:p> <o:p></o:p> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: <a href="mailto:ckoenig.leichtzumerken@gmail.com">Christian König</a> Sent: 2017年10月1日 17:36 To: <a href="mailto:Monk.Liu@amd.com">Liu, Monk</a>; <a href="mailto:amd-gfx@lists.freedesktop.org"> amd-gfx@lists.freedesktop.org</a> Subject: Re: [PATCH 12/12] drm/amdgpu/sriov:no shadow buffer recovery<o:p></o:p> </div> <o:p></o:p> </div> </div> <div> Am 30.09.2017 um 08:03 schrieb Monk Liu: > 1, we have deadlock unresloved between shadow bo recovery > and ctx_do_release, > > 2, for loose mode gpu reset we always assume VRAM not lost > so there is no need to do that from begining > > Change-Id: I5259f9d943239bd1fa2e45eb446ef053299fbfb1 > Signed-off-by: Monk Liu <a href="mailto:Monk.Liu@amd.com"><Monk.Liu@amd.com></a> NAK, even when VRAM ist lost we must restore the page tables or otherwise no process would be able to proceed. Regards, Christian. > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 29 ----------------------------- > 1 file changed, 29 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > index c3f10b5..8ae7a2c 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > @@ -2840,9 +2840,7 @@ int amdgpu_sriov_gpu_reset(struct amdgpu_device *adev, struct amdgpu_job *job) > { > int i, j, r = 0; > int resched; > - struct amdgpu_bo *bo, *tmp; > struct amdgpu_ring *ring; > - struct dma_fence *fence = NULL, *next = NULL; > > /* other thread is already into the gpu reset so just quit and come later */ > if (!atomic_add_unless(&adev->in_sriov_reset, 1, 1)) > @@ -2909,33 +2907,6 @@ int amdgpu_sriov_gpu_reset(struct amdgpu_device *adev, struct amdgpu_job *job) > /* release full control of GPU after ib test */ > amdgpu_virt_release_full_gpu(adev, true); > > - DRM_INFO("recover vram bo from shadow\n"); > - > - ring = adev->mman.buffer_funcs_ring; > - mutex_lock(&adev->shadow_list_lock); > - list_for_each_entry_safe(bo, tmp, &adev->shadow_list, shadow_list) { > - next = NULL; > - amdgpu_recover_vram_from_shadow(adev, ring, bo, &next); > - if (fence) { > - r = dma_fence_wait(fence, false); > - if (r) { > - WARN(r, "recovery from shadow isn't completed\n"); > - break; > - } > - } > - > - dma_fence_put(fence); > - fence = next; > - } > - mutex_unlock(&adev->shadow_list_lock); > - > - if (fence) { > - r = dma_fence_wait(fence, false); > - if (r) > - WARN(r, "recovery from shadow isn't completed\n"); > - } > - dma_fence_put(fence); > - > for (i = j; i < j + AMDGPU_MAX_RINGS; ++i) { > ring = adev->rings[i % AMDGPU_MAX_RINGS]; > if (!ring || !ring->sched.thread) <o:p></o:p> </div> </blockquote> <o:p> </o:p> </div> </body> </html>