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

Investigate Poolable.maxSize to test quickly for memory leaks #55

Open
zepumph opened this issue Mar 7, 2019 · 2 comments
Open

Investigate Poolable.maxSize to test quickly for memory leaks #55

zepumph opened this issue Mar 7, 2019 · 2 comments
Assignees

Comments

@zepumph
Copy link
Member

zepumph commented Mar 7, 2019

From phetsims/sun#432, we think that seeting the maxSize to zero could help us find memory leaks faster.

An idea of another test, manually set the Poolable option maxSize to 0, and see that there would be no slow "tail" increasing the memory as the pools get to their max size.

@jonathanolson said that increasing the number of pooled object allocations in the pool shouldn't be a problem.

This test will be performed by @jbphet to see if there is a memory leak.

@jonathanolson said that phet-core doesn't support query parameters, so we can't support this as a "memory leak testing" query param.

@samreid is interested in this to see if it is a good way to easily find memory leaks quickly, since it involved bypassing the pool.

@jbphet will investigate, report, and bring it back to developer meeting.

@jbphet
Copy link
Contributor

jbphet commented Mar 18, 2019

I tested this by setting the pool sizes to zero, which just forces all pool allocations to call the constructor. The result was that the sims use less memory overall, but leaks at the same rate. I think this probably means that it's not a problem with the pools.

There still seems to be a slow memory leak, or maybe it just takes a lot of fuzzing for all of the pools to get full. At any rate, the leak rate as it is now isn't bad enough that it will impact our users, and it's not easy to figure out where it's coming from. I don't feel that it's worth any more of my time to investigate this, and I'll put it up for developer meeting to see if we want to close or have someone else pursue it.

@jbphet jbphet removed their assignment Mar 18, 2019
@jonathanolson
Copy link
Contributor

This seems like something that would be beneficial for things I do, so I'm happy to volunteer.

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

No branches or pull requests

4 participants