Skip to content

mutex: Avoid taking platform::mutex if possible #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
avdgrinten merged 2 commits into from
Jan 17, 2026
Merged

mutex: Avoid taking platform::mutex if possible #33

avdgrinten merged 2 commits into from
Jan 17, 2026

Conversation

@avdgrinten
Copy link
Member

@avdgrinten avdgrinten commented Jan 15, 2026

This improves the performance of async::mutex uses in Thor significantly (mostly because disabling + re-enabling IRQs within the platform::mutex (= IrqSpinlock) is expensive).

@avdgrinten avdgrinten merged commit e6ebcb5 into managarm:master Jan 17, 2026
2 checks passed
@avdgrinten avdgrinten deleted the optimize-mutex branch January 17, 2026 22:14
@avdgrinten avdgrinten mentioned this pull request Jan 17, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@no92 no92 no92 approved these changes

@ArsenArsen ArsenArsen ArsenArsen approved these changes

@qookei qookei qookei approved these changes

@Geertiebear Geertiebear Awaiting requested review from Geertiebear

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@avdgrinten @no92 @ArsenArsen @qookei