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

CPU usage is too high and fans go crazy on 2.27RC2 #13641

Closed
jrainville opened this issue Feb 20, 2024 · 9 comments · Fixed by #13658
Closed

CPU usage is too high and fans go crazy on 2.27RC2 #13641

jrainville opened this issue Feb 20, 2024 · 9 comments · Fixed by #13658
Assignees
Labels
bug Something isn't working

Comments

@jrainville
Copy link
Member

Bug Report

Description

The CPU stays at around 10% for me when the app is opened and not doing anything (that's a lot of me as my laptop is very powerful).
Also, the fans go crazy because of it.
Even with a new account it does it

Steps to reproduce

  1. Open the app
  2. Wait a little
    VROOOOOOOOOM

Expected behavior

Almost no CPU usage when the app is idle

Actual behavior

CPU goes high for no reason

Additional Information

  • Status desktop version: 2.27 RC2
  • Operating System: Ubuntu 20 (happens on Mac too)
@jrainville jrainville added bug Something isn't working messenger-team labels Feb 20, 2024
@jrainville jrainville added this to the 2.27.0 Alpha RC milestone Feb 20, 2024
@caybro
Copy link
Member

caybro commented Feb 20, 2024

Yup, I quickly ran hotspot and here's the culprit :) The CPU here is idling about 20% all the time

image

Something for @richard-ramos ? Seems to be doing lots of nothing inside waku/go-discover nodes

@jrainville jrainville self-assigned this Feb 20, 2024
@jrainville jrainville moved this to In Progress in Status Desktop/Mobile Board Feb 20, 2024
@richard-ramos
Copy link
Member

Let's take a look!

@richard-ramos
Copy link
Member

@jrainville can you try with status-desktop branch px-delay? that branch has a status-go version with changes:

@jrainville
Copy link
Member Author

Sadly it didn't fix it.

Video for proof:

20240220_152753.mp4

@jrainville
Copy link
Member Author

Latest commit did fix it!

@richard-ramos
Copy link
Member

Cool!
Let's wait for waku-org/go-waku#1038 to be approved! :)

@richard-ramos
Copy link
Member

Ah it's already approved!

@jrainville jrainville moved this from In Progress to Code review in Status Desktop/Mobile Board Feb 20, 2024
@jrainville
Copy link
Member Author

Ah it's already approved!

I'll approve everything!!!

@richard-ramos
Copy link
Member

We need @chaitanyaprem 's approval too because he's a code owner!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants