Skip to content

Use usize milliseconds instead of float seconds for sleep - #916

Merged
vinc merged 8 commits into
trunkfrom
fix/i686-sleep
Sep 6, 2026
Merged

Use usize milliseconds instead of float seconds for sleep#916
vinc merged 8 commits into
trunkfrom
fix/i686-sleep

Conversation

@vinc

@vinc vinc commented Sep 5, 2026

Copy link
Copy Markdown
Owner

We can't pass a f64 directly to a syscall on i686 and I've never been satified with using a float for the sleeping duration at the low level so this PR changes that. The sleep program and lisp still use a number of seconds in float at the high level.

@vinc

vinc commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

TODO: benchmark httpd with siege to measure the impact of clamping the sleep duration, and see if we can reduce CPU usage when idle.

@vinc

vinc commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

DOOM needed to be updated too (https://github.com/vinc/moros-doom)

@vinc
vinc marked this pull request as ready for review September 6, 2026 06:46
@vinc
vinc merged commit c58d2e5 into trunk Sep 6, 2026
2 checks passed
@vinc
vinc deleted the fix/i686-sleep branch September 6, 2026 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant