-
Notifications
You must be signed in to change notification settings - Fork 133
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
fsmonitor: fix hangs by delayed fs event listening #1804
base: master
Are you sure you want to change the base?
fsmonitor: fix hangs by delayed fs event listening #1804
Conversation
The thread serving the client (ipc-thread) calls with_lock__wait_for_cookie() in which a cookie file is created. with_lock__wait_for_cookie() then waits for the event caused by the cookie file from the thread for fs events (fsevent-thread). However, in high load situations, the fsevent-thread may start actual fs event listening (triggered by FSEventStreamStart() for Darwin, for example) *after* the cookie file is created. In this case, the fsevent-thread cannot detect the cookie file and with_lock__wait_for_cookie() waits forever, so that the whole daemon hangs [1]. Extend listen_error_code to express that actual fs event listening starts. listen_error_code is accessed in a thread-safe manner by utilizing a dedicated mutex. [1]: https://lore.kernel.org/git/[email protected]/ Suggested-by: Jeff King <[email protected]> Signed-off-by: Koji Nakamaru <[email protected]>
/submit |
Submitted as [email protected] To fetch this version into
To fetch this version to local tag
|
This patch series was integrated into seen via git@e6e38f1. |
This branch is now known as |
This patch series was integrated into seen via git@1859bb5. |
This patch series was integrated into seen via git@d60c7c7. |
This patch series was integrated into seen via git@b64ab23. |
This patch series was integrated into seen via git@bca997f. |
This patch series was integrated into seen via git@514a9ae. |
There was a status update in the "Cooking" section about the branch Under high load, fsmonitor process can hang at start-up, which has been corrected. Comments? source: <[email protected]> |
cc: Jeff King [email protected]