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

Memory leak in artiq_gui #379

Closed
cjbe opened this issue Apr 6, 2016 · 12 comments
Closed

Memory leak in artiq_gui #379

cjbe opened this issue Apr 6, 2016 · 12 comments
Assignees
Milestone

Comments

@cjbe
Copy link
Contributor

cjbe commented Apr 6, 2016

Using Artiq 1.0rc2 on Ubuntu 15.10, with the examples/master device_db, and the kc705-clock gateware, artiq_gui leaks memory.

Over 1000 minutes, the RSS linearly increased from 87 MB to 1500 MB.

I don't think this is a regression - I believe I have seen this issue in earlier versions of artiq_gui.

@sbourdeauducq
Copy link
Member

Were you running something in particular or is the memory consumption increasing while nothing is happening?

@cjbe
Copy link
Contributor Author

cjbe commented Apr 6, 2016

I was not running anything. I just started the master, ctrlmgr, and gui, and monitored the memory usage

@sbourdeauducq
Copy link
Member

Were there any logging messages emitted by e.g. ctlmgr?

@cjbe
Copy link
Contributor Author

cjbe commented Apr 6, 2016

No logging messages were emitted

@jordens
Copy link
Member

jordens commented Apr 6, 2016

Reproduced. Comes in chunks of about 6MB and valgrind doesn't see it. It's probably not strictly a "leak" then.

@jordens jordens added this to the 1.0 milestone Apr 6, 2016
@sbourdeauducq
Copy link
Member

Python has some functions to print the state of the GC (number of live objects, etc.). What do they say if you call them regularly with a timer?

@jordens
Copy link
Member

jordens commented Apr 6, 2016

@jordens jordens self-assigned this Apr 6, 2016
@jordens
Copy link
Member

jordens commented Apr 6, 2016

10 Qtimers, 10 functions, 10 tuples, and 30 cells per second.

@jordens
Copy link
Member

jordens commented Apr 6, 2016

probably harvimt/quamash#62

@jordens
Copy link
Member

jordens commented Apr 6, 2016

was quamash.
@sbourdeauducq or @whitequark could you please copy the package over?

@jordens jordens closed this as completed Apr 6, 2016
@sbourdeauducq
Copy link
Member

Already there.

@jordens
Copy link
Member

jordens commented Apr 6, 2016

There is a bunch of Exception ignored in: Exception ignored in: pairs with the new quamash one for each applet. https://bugs.python.org/issue22836

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

3 participants