Skip to content
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

fix: futex requeue error and implement futex key #49

Merged
merged 1 commit into from
May 11, 2024
Merged

fix: futex requeue error and implement futex key #49

merged 1 commit into from
May 11, 2024

Conversation

YXalix
Copy link

@YXalix YXalix commented May 9, 2024

在futex requeue中使用的move_num使用的是timeout本身的值, 在修复后就不需要check_dead_wait了.

同时使用futexkey来代替vaddr, 目前只实现了private futex

@YXalix YXalix force-pushed the dev-futex branch 2 times, most recently from 8c21ab9 to b096bcb Compare May 10, 2024 13:24
Azure-stars
Azure-stars approved these changes May 11, 2024
View reviewed changes
Copy link

@Azure-stars Azure-stars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

该 futex 的实现参考了 linux 的 futex 实现，不采用 Starry 本身的睡眠任务队列机制，而是通过循环 yield 的形式使得任务可以提前醒来。

@Azure-stars Azure-stars merged commit fb87c24 into Arceos-monolithic:main May 11, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Azure-stars Azure-stars Azure-stars approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@YXalix @Azure-stars