RE: [PATCH 1/2] drm/amd/amdgpu: shorten the gfx idle worker timeout

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[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.wang@xxxxxxx>

Best Regards,
Kevin

-----Original Message-----
From: Kenneth Feng <kenneth.feng@xxxxxxx>
Sent: Tuesday, March 11, 2025 16:55
To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Cc: Wang, Yang(Kevin) <KevinYang.Wang@xxxxxxx>; Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Feng, Kenneth <Kenneth.Feng@xxxxxxx>
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.feng@xxxxxxx>
---
 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





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux