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

JS heap usage is leaking alarmingly and causing long-lived tabs to crawl #2647

Closed
ara4n opened this issue Nov 25, 2016 · 7 comments
Closed

Comments

@ara4n
Copy link
Member

ara4n commented Nov 25, 2016

I think this may be an artefact of the increasing number of E2E rooms, but on Chrome I got 6 minute's worth of "application unresponsive" warnings on unsleeping this morning interspersed with snatches of responsivity. It finally recovered; here's the JS console:

wedged-during-catchup.txt

It's possible that the JS heap had grown so huge that Chrome was swapping, given that it's been trying to generate a JS heap snapshot for the last 4 minutes now and has only got 32% of the way there.....

@ara4n
Copy link
Member Author

ara4n commented Nov 25, 2016

(except the chrome process in question is 'only' 2.4GB RSZ, and the box isn't swapping :/)

@ara4n
Copy link
Member Author

ara4n commented Nov 25, 2016

rather splendidly, the Inspector crashes about 5 minutes into trying to generate a heapdump, so i have no clue what state Chrome's in here :( Restarting...

@ara4n
Copy link
Member Author

ara4n commented Nov 25, 2016

i think this is basically a memory leak - tabs which are older than a few days seem to be using loads of heap.

@ara4n ara4n changed the title 0.9.0 just took 6 minutes to recover after being paused overnight. JS heap usage is leaking alarmingly and causing long-lived tabs to crawl Nov 25, 2016
@ara4n
Copy link
Member Author

ara4n commented Nov 25, 2016

i imagine this will be very problematic for electron too. may be related to #2109.

@richvdh
Copy link
Member

richvdh commented Dec 1, 2016

let's not assume this is e2e until proven otherwise

@richvdh richvdh removed the A-E2EE label Dec 1, 2016
@ara4n
Copy link
Member Author

ara4n commented Apr 15, 2017

this is indeed just running out of heap, which is capped at 1.5GB on Chrome. It's been improved by various bugfixes by kegan but it's still a problem because we simply never reap old events from memory.

@ara4n
Copy link
Member Author

ara4n commented Apr 24, 2017

closing as a dup of #2951

@ara4n ara4n closed this as completed Apr 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants