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.

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

fix:修复epoll_pwait部分与linux不一致的行为 #1090

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sparkzky
Copy link
Contributor

No description provided.

@dragonosbot
Copy link

@sparkzky: no appropriate reviewer found, use r? to override

@dragonosbot dragonosbot added the S-等待审查 Status: 等待assignee以及相关方的审查。 label Feb 19, 2025
@github-actions github-actions bot added the ambiguous The title of PR/issue doesn't match the format label Feb 19, 2025
@sparkzky
Copy link
Contributor Author

r? @fslongjin

@sparkzky sparkzky changed the title 修复epoll_pwait部分与linux不一致的行为 fix:修复epoll_pwait部分与linux不一致的行为 Feb 20, 2025
@github-actions github-actions bot added Bug fix A bug is fixed in this pull request and removed ambiguous The title of PR/issue doesn't match the format labels Feb 20, 2025
let oset = *guard.sig_blocked();

let flags = pcb.flags();
flags.set(ProcessFlags::RESTORE_SIG_MASK, true);
Copy link
Member

Choose a reason for hiding this comment

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

这个有问题啊,有设置的地方肯定要有clear的地方。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image
我看他这里的逻辑好像就是这样的,当遇到非中断的错误时就会恢复,也就是在restore_saved_sigmask里面恢复信号并且更改标志位

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug fix A bug is fixed in this pull request S-等待审查 Status: 等待assignee以及相关方的审查。
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants