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

Restores lost states after wakeup from sleep #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

aluveitie
Copy link
Contributor

@trulyspinach Not tested yet - can I ask you for a review before I try to run this?

@aluveitie
Copy link
Contributor Author

@trulyspinach I've tested and verified it. This properly restores the lost Pstate and CPB settings after wakeup.

@aluveitie
Copy link
Contributor Author

aluveitie commented May 26, 2020

@trulyspinach I've updated the pull request. I've been experiencing complete freezes about 30mins after waking up from sleep forcing to power off and restart the machine. With the last commit I could no longer reproduce any freezes after the last couple of sleep cycles.

@trulyspinach
Copy link
Owner

Thanks a lot :). Was the freeze cause by MPERF-APERF being zero? I believe it's very unlikely for that to happen. You can view to panic log for the freeze in system log, they are all recorded.

@aluveitie
Copy link
Contributor Author

I searched for logs, but could not find any.
I'm not completely sure since I could not reproduce it anymore (had a log there if MPERF-APERF being zero). But I guess guarding the division might still be good to have.

@trulyspinach
Copy link
Owner

Yes, of course, It should definitely present as once happen it will crash the whole system. I will look in to this more when I have more time available :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants