drm/amd/display: Change pstate expected timeout warning to 180us on linux
authorVictor Lu <victorchengchi.lu@amd.com>
Tue, 24 Nov 2020 15:06:04 +0000 (10:06 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 15 Dec 2020 16:33:22 +0000 (11:33 -0500)
[Why]
There is a warning that triggers when pstate takes too long.
Pstate can take up to ~200us on Linux without hanging but
it is currently set to 40us.

[How]
Change the timeout for the warning to be 180us on Linux.

Signed-off-by: Victor Lu <victorchengchi.lu@amd.com>
Reviewed-by: Roman Li <Roman.Li@amd.com>
Acked-by: Qingqing Zhuo <qingqing.zhuo@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c

index 75637c2..6f42d10 100644 (file)
@@ -124,11 +124,11 @@ bool hubbub1_verify_allow_pstate_change_high(
         * still not asserted, we are probably stuck and going to hang
         *
         * TODO: Figure out why it takes ~100us on linux
-        * pstate takes around ~100us on linux. Unknown currently as to
-        * why it takes that long on linux
+        * pstate takes around ~100us (up to 200us) on linux. Unknown currently
+        * as to why it takes that long on linux
         */
        const unsigned int pstate_wait_timeout_us = 200;
-       const unsigned int pstate_wait_expected_timeout_us = 40;
+       const unsigned int pstate_wait_expected_timeout_us = 180;
        static unsigned int max_sampled_pstate_wait_us; /* data collection */
        static bool forced_pstate_allow; /* help with revert wa */