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

endless high CPU spike #15611

Closed
bitdisaster opened this issue Nov 6, 2018 · 8 comments
Closed

endless high CPU spike #15611

bitdisaster opened this issue Nov 6, 2018 · 8 comments
Labels
3-0-x blocked/need-info ❌ Cannot proceed without more information bug/regression ↩️ A new version of Electron broke something platform/macOS platform/windows

Comments

@bitdisaster
Copy link
Contributor

bitdisaster commented Nov 6, 2018

  • Output of node_modules/.bin/electron --version: 3.0.8
  • Operating System (Platform and Version): x64 Win7, 8.1, Server 2012 R2, Win 10 (Win 10 almost not reproducible)
  • Output of node_modules/.bin/electron --version on last known working Electron version (if applicable): 2.0.x

Expected Behavior
CPU load should not be significant higher than Electron 2.0.x. After initial CPU load the process should return to idle.

Actual behavior
CPU load is higher than it used to be on Electron 2.0.x. The process enters a state of "endless" CPU consumption.

To Reproduce

> git clone https://github.com/bitdisaster/electron-quick-start.git -b high_cpu_load_bug
> npm install
> npm run start 

The likelihood of this bug depends on the initial available CPU resources and the Windows version. I had the best luck with Windows 8.1 and 1 CPU. However, I was able to repro this on other Windows versions and with more CPUs.

  1. Launch App
  2. Observe CPU (if not at 100% continue with step 3)
  3. Search for something
  4. Scroll result page

Screenshots
cpuload

Additional Information
Trace:
--trace-startup --trace-startup-file=trace.json --trace-startup-duration=45
trace.zip

@welcome
Copy link

welcome bot commented Nov 6, 2018

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

To help make it easier for us to investigate your issue, please follow the contributing guidelines.

@bitdisaster bitdisaster changed the title Endless High CPU s Endless High CPU spike Nov 7, 2018
@bitdisaster bitdisaster changed the title Endless High CPU spike endless high CPU spike Nov 7, 2018
@spiral2k
Copy link

spiral2k commented Nov 8, 2018

i'm suffering from this too, checked with two different Macbooks (MacOs High Sierra)

@sofianguy sofianguy added this to Needs Review in 3.0.x / 3.1.x Nov 13, 2018
@nornagon
Copy link
Member

This seems related to #14319, and might be fixed by #15975? @bitdisaster would you try on v3.1.0-beta.3 or higher?

@bitdisaster
Copy link
Contributor Author

Will, do when I'm back home on my main dev machine. 🤞

@bitdisaster
Copy link
Contributor Author

I can confirm that its fixed for my repro. However, we still see the same symptoms on a multicore system with Slack Desktop App. There are multiple renderer processes that do nothing. But 2 processes keep running at full blast. Is there a possibility that this fix could extend to multi-core machines?

@chevonc
Copy link

chevonc commented Jul 8, 2019

Hey @bitdisaster :). Did you get this resolved by any chance?

@sofianguy
Copy link
Contributor

Thank you for taking the time to report this issue and helping to make Electron better.

The version of Electron you reported this on has been superseded by newer releases.

If you're still experiencing this issue in Electron v4.2.x or later, please add a comment specifying the version you're testing with and any other new information that a maintainer trying to reproduce the issue should know.

I'm setting the blocked/need-info label for the above reasons. This issue will be closed 7 days from now if there is no response.

Thanks in advance! Your help is appreciated.

@sofianguy sofianguy added the blocked/need-info ❌ Cannot proceed without more information label Aug 9, 2019
@sofianguy
Copy link
Contributor

Thank you for your issue!

We haven't gotten a response to our questions in our comment above. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3-0-x blocked/need-info ❌ Cannot proceed without more information bug/regression ↩️ A new version of Electron broke something platform/macOS platform/windows
Projects
No open projects
3.0.x / 3.1.x
Unsorted Issues
Development

No branches or pull requests

6 participants