[AMD Official Use Only - AMD Internal Distribution Only]

Please correct the code comments before committing the patch.

/* 1 second timeout */  10ms

With that fixed,

Reviewed-by:  Yang Wang <kevinyang.w...@amd.com>

Best Regards,
Kevin

-----Original Message-----
From: Kenneth Feng <kenneth.f...@amd.com>
Sent: Tuesday, March 11, 2025 16:55
To: amd-gfx@lists.freedesktop.org
Cc: Wang, Yang(Kevin) <kevinyang.w...@amd.com>; Deucher, Alexander 
<alexander.deuc...@amd.com>; Feng, Kenneth <kenneth.f...@amd.com>
Subject: [PATCH 1/2] drm/amd/amdgpu: shorten the gfx idle worker timeout

Shorten the gfx idle worker timeout. This is to sync with DAL when there is no 
activity on the screen. Original 1 second can not sync with DAL, so DAL can not 
apply MALL when the workload type is not bootup default.

Signed-off-by: Kenneth Feng <kenneth.f...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
index ddf4533614ba..b7b40edf4a31 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
@@ -58,7 +58,7 @@ enum amdgpu_gfx_pipe_priority {  #define 
AMDGPU_GFX_QUEUE_PRIORITY_MAXIMUM  15

 /* 1 second timeout */
-#define GFX_PROFILE_IDLE_TIMEOUT       msecs_to_jiffies(1000)
+#define GFX_PROFILE_IDLE_TIMEOUT       msecs_to_jiffies(10)

 enum amdgpu_gfx_partition {
        AMDGPU_SPX_PARTITION_MODE = 0,
--
2.34.1

Reply via email to