-
Notifications
You must be signed in to change notification settings - Fork 11
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
Windows 10 and 11 version "forgets" CPU WU after system restart #290
Comments
Does the forgetting not happen if you quit the client via sys tray before restarting the system? |
Quitting and then restarting resumes from where it stopped. |
My thought, which has been disputed in the past, is that Windows unceremoniously kills the client when there is a reboot, leaving database locked with unsaved changes. The client is also running a native Win event loop. |
I have a suspicion, too, that windows shutdown procedures are killing FAH processes, however, these systems had no issues with v7, as well as v7 would survive crashing system most of the time. I think v8 needs a bit more robust progress saving system. I'm not sure how v8 reacts to crahsing systems, since mine have been stable for quite some time now. |
Also, what is weird, that V8 is downloading new WU, instead of resetting currently running one to zero. |
Could this be the same permissions issue that is causing the client not to restart. If the old WUs file permissions were changed such that it was inaccessible then it would not run after reboot. In this case, I would expect the old WU files to still be on the disk. |
The permissions problem could be caused by either the client trying to start in the wrong directory, the file permissions on |
I will be able to test this in an hour or so. The test PC has ProgramData/FahClient security settings tweaked to allow writing by any Users, which solved the restart issue. Do you have some sort of longer WU for me to download during the test that I do not lose real work? 2 minute long WU would be sufficient. If you don't mind creating something, ping me details in the Slack |
Just restarted the PC and it downloaded new WU ignoring the old one. Here is the log. Old one had 2% done. Windows Event Viewer does not report any FAH related errors or any other app related errors. |
It does look like the client was hard killed during shutdown. The log just ends abruptly. Then when it starts again it says: |
Not sure if that makes any difference, on my Windows 10 PC, FAHClient process is running under muziqaz2 user name.
|
Just for giggles, I gave full control to muziqaz2 for the WU folder, rebooted, it still downloaded new WU. So doubt this is permission issue |
Does this build work any better?: https://master.foldingathome.org/builds/fah-client/windows-10-64bit/release/v8.4.6-b135/fah-client_8.4.6_AMD64.exe I added detection of the Windows shutdown message. |
I don't believe I am breeding killer OSs :D |
New version ignored old WU and downloaded new WU after installation even without the system restart :( Is it possible windows process termination method is broken on my systems, which can't be, since it would be hell of the coincidence to have 2 different Windows versions sledgehammering processes instead of closing them |
Restarting still downloaded new WU Also, installation did not change |
Is the Windows logout message also handled by the new code? |
I added code to shut the client down if it receives a |
Today I had reason to pause folding on W11 and lost all progress using 8.4.5 |
I thought it would be better to create a separate issue regarding the WU stuff.
So every time I restart my Windows PCs, both systems FAH client downloads new WU instead of continuing with the old one, which was downloaded and being folded before the restart. On one system I can see 5 WU folders in
Work
directory, while I am only folding one at the moment.Log is not showing anything suspicious, except some subprocess error
14:20:09:E :Subprocess deallocated while process is still running
Here is the log.
log.txt
There has been no alterations to both systems in regards to boot/restart sequence. Everything is default as father Microsoft intended.
Linux system seems to work as intended
I mentioned this issue before in this comment and thread: #286 (comment)
The text was updated successfully, but these errors were encountered: