Re: [PATCH] drm/amdgpu: Remove fence_process in count_emitted

2022-09-26 Thread Christian König

Am 23.09.22 um 15:27 schrieb jiadong@amd.com:

From: "Jiadong.Zhu" 

The function amdgpu_fence_count_emitted used in work_hander should not call
amdgpu_fence_process which must be used in irq handler.

Signed-off-by: Jiadong.Zhu 


Reviewed-by: Christian König 


---
  drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
index d16c8c1f72db..790f7bfdc654 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
@@ -394,7 +394,6 @@ unsigned amdgpu_fence_count_emitted(struct amdgpu_ring 
*ring)
/* We are not protected by ring lock when reading the last sequence
 * but it's ok to report slightly wrong fence count here.
 */
-   amdgpu_fence_process(ring);
emitted = 0x1ull;
emitted -= atomic_read(>fence_drv.last_seq);
emitted += READ_ONCE(ring->fence_drv.sync_seq);




[PATCH] drm/amdgpu: Remove fence_process in count_emitted

2022-09-23 Thread jiadong.zhu
From: "Jiadong.Zhu" 

The function amdgpu_fence_count_emitted used in work_hander should not call
amdgpu_fence_process which must be used in irq handler.

Signed-off-by: Jiadong.Zhu 
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
index d16c8c1f72db..790f7bfdc654 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
@@ -394,7 +394,6 @@ unsigned amdgpu_fence_count_emitted(struct amdgpu_ring 
*ring)
/* We are not protected by ring lock when reading the last sequence
 * but it's ok to report slightly wrong fence count here.
 */
-   amdgpu_fence_process(ring);
emitted = 0x1ull;
emitted -= atomic_read(>fence_drv.last_seq);
emitted += READ_ONCE(ring->fence_drv.sync_seq);
-- 
2.25.1