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 on standby/resume cycle #82

Closed
fabsenet opened this issue May 7, 2019 · 4 comments
Closed

Memory leak on standby/resume cycle #82

fabsenet opened this issue May 7, 2019 · 4 comments
Labels
Milestone

Comments

@fabsenet
Copy link
Owner

fabsenet commented May 7, 2019

Adrilight increases its memory usage by roughly 50mb for each standby/resume cycle. (#80 does not solve it)

@fabsenet fabsenet added the bug label May 7, 2019
@HopefullyICanChangeThatNameLaterOn

@fabsenet kann ich bei mir so nicht nachstellen. Beim ersten Öffnen ca 35mb RAM. Nach dem ersten Mal "Energie Sparen" dann 85mb - die 85mb blieben dann aber auch nach mehrmaligen "Energie Sparen".

Ich würde empfehlen den Debugger währenddessen laufen zu lassen und Heap Dumps zu erstellen und diese dann zu vergleichen.

Ansonsten als Quick and Dirty Lösung die Anwendung neustarten beim jeweiligen Event (SystemEvents.PowerModeChanged).

@fabsenet
Copy link
Owner Author

fabsenet commented May 9, 2019

ich hatte den ramverbrauch aus dem taskmanager als grundlage genommen, du auch?

Ich glaube ja, dass es sich um unmanaged memory handelt, da ich zumindest in "meinem" code nichts finden konnte, was die größe/menge rechtfertigt.

PowerModeChanged ist schon abonniert, aber bisher nur für logging. ich glaube, dieses event ist nicht super vertrauenswürdig und ich will da lieber keine issues bekommen, dass adrilight manchmal nicht mehr funktioniert nach einem resume.

@HopefullyICanChangeThatNameLaterOn

Ja, Taskmanager Spalte "Arbeitsspeicher (aktiver privater Arbeitssatz)". Ich habe es gerade auch nochmal mit dem aktuellen dev-branch getestet. Dort habe ich zwar kurzfristig etwas Zuwachs gesehen (200mb Verbrauch bei 3x Standby), aber nach einigen Minuten war der RAM Verbrauch wieder normal (~66mb).
Ich bekomme es also nicht wirklich nachgestellt.
Hängt bestimmt mit irgendeinem Garbage Collector zusammen... vielleicht kannst du den irgendwie mal erzwingen?

@psbeau
Copy link

psbeau commented May 9, 2019

I wasn't really paying much attention to this thread until I saw Adrilight taking 3GB of memory :-/
I'm running the dev 84CCB854 version.

fabsenet added a commit that referenced this issue May 29, 2019
fabsenet added a commit that referenced this issue May 29, 2019
@fabsenet fabsenet added this to the 2.0.9 milestone May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants