Re: 答复: [V2 04/11] drm/amdgpu/virt: use kiq to access registers

2017-01-11 Thread Christian König
't check the code on detail previously, thanks ! BR Monk *发件人:* amd-gfx 代表 Liu, Monk *发送时间:* 2017年1月11日 11:06:34 *收件人:* Christian König; Yu, Xiangliang; amd-gfx@lists.freedesktop.org *主题:* 答复: [V2 04/11] drm/amdgpu/v

答复: [V2 04/11] drm/amdgpu/virt: use kiq to access registers

2017-01-11 Thread Liu, Monk
l previously, thanks ! BR Monk 发件人: amd-gfx 代表 Liu, Monk 发送时间: 2017年1月11日 11:06:34 收件人: Christian König; Yu, Xiangliang; amd-gfx@lists.freedesktop.org 主题: 答复: [V2 04/11] drm/amdgpu/virt: use kiq to access registers > + fence_get(f); > + amdgpu_ring_co

答复: [V2 04/11] drm/amdgpu/virt: use kiq to access registers

2017-01-10 Thread Liu, Monk
> + mutex_lock(&adev->virt.lock); > + amdgpu_ring_alloc(ring, 32); > + amdgpu_ring_emit_hdp_flush(ring); > + amdgpu_ring_emit_rreg(ring, reg); > + amdgpu_ring_emit_hdp_invalidate(ring); > + amdgpu_fence_emit(ring, &f); > + fence_get(f); > + amdgpu_ring_commit(ring);

答复: [V2 04/11] drm/amdgpu/virt: use kiq to access registers

2017-01-10 Thread Liu, Monk
> + fence_get(f); > + amdgpu_ring_commit(ring); > + mutex_unlock(&adev->virt.lock); > + > + r = fence_wait(f, false); > + fence_put(f); Why do you grab and release an extra fence reference here? Christian. [ML] e.g. without those grab/release pare, if fence is signaled right