Bananymous
/
banan-os
1
1
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

WindowServer: Make select timeout more precise to hit target FPS 

Previously window server was only running at around 40 FPS while
targetting 60. This makes the select timeout properly sleep so that
target FPS is reached!
This commit is contained in:
Bananymous 2024-11-17 03:14:59 +02:00
parent e1b82e4e43
commit 411f32c766
1 changed files with 16 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
userspace/programs/WindowServer/main.cpp
View File

@ -198,19 +198,23 @@ int main()
if (auto ret = window_server.set_background_image(BAN::move(config.background_image)); ret.is_error()) if (auto ret = window_server.set_background_image(BAN::move(config.background_image)); ret.is_error())
dwarnln("Could not set background image: {}", ret.error()); dwarnln("Could not set background image: {}", ret.error());
const auto get_current_us =
[]() -> uint64_t
{
timespec current_ts;
clock_gettime(CLOCK_MONOTONIC, &current_ts);
return (current_ts.tv_sec * 1'000'000) + (current_ts.tv_nsec / 1'000);
};
constexpr uint64_t sync_interval_us = 1'000'000 / 60; constexpr uint64_t sync_interval_us = 1'000'000 / 60;
timespec last_sync { .tv_sec = 0, .tv_nsec = 0 }; uint64_t last_sync_us = 0;
while (!window_server.is_stopped()) while (!window_server.is_stopped())
{ {
timespec current_ts; const auto current_us = get_current_us();
clock_gettime(CLOCK_MONOTONIC, &current_ts); if (current_us - last_sync_us > sync_interval_us)
uint64_t us_since_last_sync = (current_ts.tv_sec - last_sync.tv_sec) * 1'000'000 + (current_ts.tv_nsec - last_sync.tv_nsec) / 1000;
if (us_since_last_sync > sync_interval_us)
{ {
window_server.sync(); window_server.sync();
us_since_last_sync = 0; last_sync_us += sync_interval_us;
last_sync = current_ts;
} }
int max_fd = server_fd; int max_fd = server_fd;
@ -230,10 +234,10 @@ int main()
} }
max_fd = BAN::Math::max(max_fd, window_server.get_client_fds(fds)); max_fd = BAN::Math::max(max_fd, window_server.get_client_fds(fds));
timeval select_timeout { timeval select_timeout {};
.tv_sec = 0, if (auto current_us = get_current_us(); current_us - last_sync_us < sync_interval_us)
.tv_usec = static_cast<long>(sync_interval_us - us_since_last_sync) select_timeout.tv_usec = sync_interval_us - (current_us - last_sync_us);
};
int nselect = select(max_fd + 1, &fds, nullptr, nullptr, &select_timeout); int nselect = select(max_fd + 1, &fds, nullptr, nullptr, &select_timeout);
if (nselect == 0) if (nselect == 0)
continue; continue;