Andrew/redot-engine
1
0
Fork 0
mirror of https://github.com/Redot-Engine/redot-engine.git synced 2025-12-06 07:17:42 -05:00
Code Issues Packages Projects Releases Wiki Activity

Use more efficient sleep approach on Windows when low-processor mode is enabled

Browse Source 
This aims to fix the idle CPU utilization regression from 4.3 to 4.4
by reverting to the previous approach, but only when low-processor mode
is enabled.
This commit is contained in:
Hugo Locurcio
2025-03-07 18:06:41 +01:00
parent f2cc3f1275
commit 03b4e0dd58
1 changed files with 19 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
platform/windows/os_windows.cpp
View File

@@ -2340,18 +2340,26 @@ void OS_Windows::add_frame_delay(bool p_can_draw) {
target_ticks += dynamic_delay;
uint64_t current_ticks = get_ticks_usec();
if (target_ticks > current_ticks + delay_resolution) {
uint64_t delay_time = target_ticks - current_ticks - delay_resolution;
// Make sure we always sleep for a multiple of delay_resolution to avoid overshooting.
// Refer to: https://learn.microsoft.com/en-us/windows/win32/api/synchapi/nf-synchapi-sleep#remarks
delay_time = (delay_time / delay_resolution) * delay_resolution;
if (delay_time > 0) {
delay_usec(delay_time);
if (!is_in_low_processor_usage_mode()) {
if (target_ticks > current_ticks + delay_resolution) {
uint64_t delay_time = target_ticks - current_ticks - delay_resolution;
// Make sure we always sleep for a multiple of delay_resolution to avoid overshooting.
// Refer to: https://learn.microsoft.com/en-us/windows/win32/api/synchapi/nf-synchapi-sleep#remarks
delay_time = (delay_time / delay_resolution) * delay_resolution;
if (delay_time > 0) {
delay_usec(delay_time);
}
}
// Busy wait for the remainder of time.
while (get_ticks_usec() < target_ticks) {
YieldProcessor();
}
} else {
// Use a more relaxed approach for low processor usage mode.
// This has worse frame pacing but is more power efficient.
if (current_ticks < target_ticks) {
delay_usec(target_ticks - current_ticks);
}
}
// Busy wait for the remainder of time.
while (get_ticks_usec() < target_ticks) {
YieldProcessor();
}
current_ticks = get_ticks_usec();