All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] drm/amdgpu: invalidate mmhub semphore workaround in amdgpu_virt
@ 2019-11-20  9:14 ` Changfeng.Zhu
  0 siblings, 0 replies; 42+ messages in thread
From: Changfeng.Zhu @ 2019-11-20  9:14 UTC (permalink / raw)
  To: Christian.Koenig-5C7GfCeVMHo, Jack.Xiao-5C7GfCeVMHo,
	Tao.Zhou1-5C7GfCeVMHo, Ray.Huang-5C7GfCeVMHo,
	Xinmei.Huang-5C7GfCeVMHo,
	amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW
  Cc: changzhu

From: changzhu <Changfeng.Zhu@amd.com>

It may lose gpuvm invalidate acknowldege state across power-gating off
cycle. To avoid this issue in virt invalidation, add semaphore acquire
before invalidation and semaphore release after invalidation.

Change-Id: Ie98304e475166b53eed033462d76423b6b0fc25b
Signed-off-by: changzhu <Changfeng.Zhu@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 26 ++++++++++++++++++++++--
 drivers/gpu/drm/amd/amdgpu/amdgpu_virt.h |  3 ++-
 drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c    |  3 ++-
 3 files changed, 28 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
index f04eb1a64271..70ffaf91cd12 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c
@@ -135,7 +135,8 @@ void amdgpu_virt_kiq_wreg(struct amdgpu_device *adev, uint32_t reg, uint32_t v)
 
 void amdgpu_virt_kiq_reg_write_reg_wait(struct amdgpu_device *adev,
 					uint32_t reg0, uint32_t reg1,
-					uint32_t ref, uint32_t mask)
+					uint32_t ref, uint32_t mask,
+					uint32_t sem)
 {
 	struct amdgpu_kiq *kiq = &adev->gfx.kiq;
 	struct amdgpu_ring *ring = &kiq->ring;
@@ -144,9 +145,30 @@ void amdgpu_virt_kiq_reg_write_reg_wait(struct amdgpu_device *adev,
 	uint32_t seq;
 
 	spin_lock_irqsave(&kiq->ring_lock, flags);
-	amdgpu_ring_alloc(ring, 32);
+	amdgpu_ring_alloc(ring, 60);
+
+	/*
+	 * It may lose gpuvm invalidate acknowldege state across power-gating
+	 * off cycle, add semaphore acquire before invalidation and semaphore
+	 * release after invalidation to avoid entering power gated state
+	 * to WA the Issue
+	 */
+
+	/* a read return value of 1 means semaphore acuqire */
+	if (ring->funcs->vmhub == AMDGPU_MMHUB_0 ||
+	    ring->funcs->vmhub == AMDGPU_MMHUB_1)
+	amdgpu_ring_emit_reg_wait(ring, sem, 0x1, 0x1);
+
 	amdgpu_ring_emit_reg_write_reg_wait(ring, reg0, reg1,
 					    ref, mask);
+	/*
+	 * add semaphore release after invalidation,
+	 * write with 0 means semaphore release
+	 */
+	if (ring->funcs->vmhub == AMDGPU_MMHUB_0 ||
+	    ring->funcs->vmhub == AMDGPU_MMHUB_1)
+	amdgpu_ring_emit_wreg(ring, sem, 0);
+
 	amdgpu_fence_emit_polling(ring, &seq);
 	amdgpu_ring_commit(ring);
 	spin_unlock_irqrestore(&kiq->ring_lock, flags);
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.h
index b0b2bdc750df..bda6a2f37dc0 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.h
@@ -295,7 +295,8 @@ uint32_t amdgpu_virt_kiq_rreg(struct amdgpu_device *adev, uint32_t reg);
 void amdgpu_virt_kiq_wreg(struct amdgpu_device *adev, uint32_t reg, uint32_t v);
 void amdgpu_virt_kiq_reg_write_reg_wait(struct amdgpu_device *adev,
 					uint32_t reg0, uint32_t rreg1,
-					uint32_t ref, uint32_t mask);
+					uint32_t ref, uint32_t mask,
+					uint32_t sem);
 int amdgpu_virt_request_full_gpu(struct amdgpu_device *adev, bool init);
 int amdgpu_virt_release_full_gpu(struct amdgpu_device *adev, bool init);
 int amdgpu_virt_reset_gpu(struct amdgpu_device *adev);
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
index f25cd97ba5f2..1ae59af7836a 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -448,9 +448,10 @@ static void gmc_v9_0_flush_gpu_tlb(struct amdgpu_device *adev, uint32_t vmid,
 			!adev->in_gpu_reset) {
 		uint32_t req = hub->vm_inv_eng0_req + eng;
 		uint32_t ack = hub->vm_inv_eng0_ack + eng;
+		uint32_t sem = hub->vm_inv_eng0_sem + eng;
 
 		amdgpu_virt_kiq_reg_write_reg_wait(adev, req, ack, tmp,
-				1 << vmid);
+						   1 << vmid, sem);
 		return;
 	}
 
-- 
2.17.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply related	[flat|nested] 42+ messages in thread

end of thread, other threads:[~2019-11-20 17:17 UTC | newest]

Thread overview: 42+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-20  9:14 [PATCH 1/2] drm/amdgpu: invalidate mmhub semphore workaround in amdgpu_virt Changfeng.Zhu
2019-11-20  9:14 ` Changfeng.Zhu
     [not found] ` <20191120091418.26526-1-changfeng.zhu-5C7GfCeVMHo@public.gmane.org>
2019-11-20 11:23   ` Christian König
2019-11-20 11:23     ` Christian König
     [not found]     ` <657609f7-bbe2-17fe-d3c1-2b16ec8868a1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2019-11-20 13:24       ` 答复: " Liu, Monk
2019-11-20 13:24         ` Liu, Monk
2019-11-20 13:30       ` Liu, Monk
2019-11-20 13:30         ` Liu, Monk
     [not found]         ` <MN2PR12MB39338596AB3B87681585B369844F0-rweVpJHSKTq/67K4VYF1uAdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 13:36           ` Christian König
2019-11-20 13:36             ` Christian König
2019-11-20 13:18   ` Liu, Monk
2019-11-20 13:18     ` Liu, Monk
     [not found]     ` <MN2PR12MB3933863C8DFCE1F68A44EFFA844F0-rweVpJHSKTq/67K4VYF1uAdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 13:20       ` Christian König
2019-11-20 13:20         ` Christian König
     [not found]         ` <1d7cd15c-f529-93da-c15e-a1fde745e6c5-5C7GfCeVMHo@public.gmane.org>
2019-11-20 13:54           ` 答复: " Liu, Monk
2019-11-20 13:54             ` Liu, Monk
     [not found]             ` <MN2PR12MB39330A0D6169250A0B75700D844F0-rweVpJHSKTq/67K4VYF1uAdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 14:00               ` Christian König
2019-11-20 14:00                 ` Christian König
     [not found]                 ` <2d410370-01d0-c845-1229-529610898604-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2019-11-20 14:16                   ` Zhu, Changfeng
2019-11-20 14:16                     ` Zhu, Changfeng
     [not found]                     ` <MN2PR12MB289616C5DA35AF8AE1799114FD4F0-rweVpJHSKToIQ/pRnFqe/QdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 14:30                       ` 答复: " Liu, Monk
2019-11-20 14:30                         ` Liu, Monk
     [not found]                         ` <MN2PR12MB3933F8A02E0C57D0092FD2AC844F0-rweVpJHSKTq/67K4VYF1uAdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 14:38                           ` Christian König
2019-11-20 14:38                             ` Christian König
     [not found]                             ` <89d689b2-19dc-6bdf-5ca5-de0a817dcaee-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2019-11-20 14:50                               ` Zhu, Changfeng
2019-11-20 14:50                                 ` Zhu, Changfeng
     [not found]                                 ` <MN2PR12MB28963F6B1C272969646EA1A0FD4F0-rweVpJHSKToIQ/pRnFqe/QdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 14:54                                   ` Christian König
2019-11-20 14:54                                     ` Christian König
2019-11-20 14:59                               ` 答复: " Liu, Monk
2019-11-20 14:59                                 ` Liu, Monk
     [not found]                                 ` <MN2PR12MB3933FBB4D1F8F3FDCFB39A4E844F0-rweVpJHSKTq/67K4VYF1uAdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 15:06                                   ` Christian König
2019-11-20 15:06                                     ` Christian König
2019-11-20 14:18                   ` Liu, Monk
2019-11-20 14:18                     ` Liu, Monk
2019-11-20 15:04           ` Zeng, Oak
2019-11-20 15:04             ` Zeng, Oak
     [not found]             ` <BL0PR12MB2580D7594496F74418BC57D1804F0-b4cIHhjg/p/XzH18dTCKOgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 15:13               ` Christian König
2019-11-20 15:13                 ` Christian König
     [not found]                 ` <4d57d0bb-c10d-4aa3-b95a-06e4aaa92679-5C7GfCeVMHo@public.gmane.org>
2019-11-20 15:38                   ` Zeng, Oak
2019-11-20 15:38                     ` Zeng, Oak
     [not found]                     ` <BL0PR12MB2580062E72CCFCDCD812484E804F0-b4cIHhjg/p/XzH18dTCKOgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2019-11-20 17:17                       ` Christian König
2019-11-20 17:17                         ` Christian König

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.