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

Optimizing animation CPU usage in AMD Power Gadget #116

Open
aluveitie opened this issue Nov 24, 2020 · 4 comments
Open

Optimizing animation CPU usage in AMD Power Gadget #116

aluveitie opened this issue Nov 24, 2020 · 4 comments
Labels
attention required enhancement New feature or request help wanted Extra attention is needed

Comments

@aluveitie
Copy link
Contributor

This is the normal CPU usage of the App based on version 0.6.2 (with kext version check removed):
Screenshot 2020-11-24 at 10 58 39

This is the CPU usage of the App after 0.6.3
Screenshot 2020-11-24 at 11 00 32

The new fluid drawing routine is responsible for up to 30% to 50% increased power usage in idle.

@trulyspinach trulyspinach added the enhancement New feature or request label Nov 24, 2020
@trulyspinach
Copy link
Owner

Thanks for pointing this out. I have noticed this since adding the animation. However, I never kept it open for too long and around 17% CPU usage for an active animation GUI app is not a too big deal and totally acceptable to me. Considering if some would like to keep the gadget open all day this might become a big headache.

It is always good to keep optimizing tho. I'll see if it is possible to reduce CPU usage while keeping the same animation. Otherwise, there can be a small button at the interface to disable animation if wanted.

@trulyspinach trulyspinach changed the title Power Gadget causing substantial CPU load Optimizing animation CPU usage in AMD Power Gadget Nov 24, 2020
@trulyspinach trulyspinach added the help wanted Extra attention is needed label Nov 24, 2020
@trulyspinach
Copy link
Owner

Screen Shot 2020-11-24 at 6 33 19 PM

@aluveitie
Copy link
Contributor Author

The App also pushes WindowServer CPU utilization considerably too.

I like to have it open since it saved me a couple of times when the power connection of pump was a little loose and suddenly stopped working. I just noticed this in time due to the CPU temp rising suddenly without any load on the system.

@trulyspinach trulyspinach pinned this issue Dec 7, 2020
cgrazy added a commit to cgrazy/SMCAMDProcessor that referenced this issue May 12, 2021
trulyspinach#116 , double for loop reduzed to one
minimize will not draw the graphs
Repository owner deleted a comment from nanohelperDev Nov 17, 2023
@trulyspinach
Copy link
Owner

@nanohelperDev Please do not post links to unreliable source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
attention required enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants