-
Notifications
You must be signed in to change notification settings - Fork 14
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
Some tests hang at 100% CPU #3
Comments
Thanks for this. I have noticed some similar issues when I try to run the library on the Github Actions MacOS machines, but unfortunately, I do not have my own MacOS machine to debug them on, and the issues do not seem to appear on Ubuntu. Could you do me a small favour and try the Version 2 WIP of the library (here: https://github.com/DanielLiamAnderson/Packaide/tree/v2) and tell me if you get the same error? I will try to get access to a MacOS machine to test on at some point. |
I will try to remember to take a look when I have a bit of free time. Would you like me to evaluate this branch for issue #2 as well? |
Sure, the Version 2 code has much better instructions and a correct |
Oh, and the good news is that I can probably borrow a Mac in the near future, so I can hopefully start investigating this issue. |
Tests 0 and 4 and possibly later tests put my laptop in space-heater mode, using 100% CPU without making apparent progress. (I'm on macOS 12.5 Monterey.) Sampling the process with Activity Monitor shows that it is spending almost all its time in
os_stat
. This suggests to me that there is code somewhere that waits for the status of a file to change within a spin loop. Of course, on a POSIX systemos_stat
could be used on something other than a file, like a pipe or a socket for example. I'm not savvy enough to tell if this is the case just by looking at the data given by Activity Monitor.The text was updated successfully, but these errors were encountered: