-
Notifications
You must be signed in to change notification settings - Fork 346
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
Out of memory in case of very frequent hooked function #4829
Comments
Not sure if I would call it a bug, but it also doesn't surprise me. AFAIK we never optimized for such a high load of calls. With "Orbit app" I guess you mean the client, right? So far the limiting factor has always been the rendering. We calculate some histograms but I don't know from the top of my head when this happens. I know some parts are done in the postprocessing step, but probably not all of it. Maybe @florian-kuebler knows more. There are 2 ways to find out more details:
I hope that helps. 😬 |
Thank you for these hints! I'm not sure if it helps with memory profiling, but I'll give it a try in any case. |
Sorry for being late to this. So I am not surprised if we do waste (or even leak) memory at some places. We unfortunately did not had time to look into memory. Also I agree that profiling Orbit with Orbit (or using introspection), will work, but also that Orbit is not particularly good in memory profiling (it has some memory tracking though). What I know for sure is, that we keep every record of a function call (what we call a timer) in memory. That would 1.000.000* 10sec * ~4x8 Bytes (uint64) is around 0.3 Gb. That does not explain that completely. But we do have some of these places. So I would not be surprised if this is already the reason. Unfortunately, I won't have any time soon to look into this more closely.... |
When the hooked function is called, let's say, 1 million times per second, the Orbit app consumes almost 5 Gb of RAM after approximately 10 seconds. If the frequency is lower (100.000 times per second), then the time extends proportionally. I suspect that it might happen because of the data collected for a histogram, but I'm not sure yet.
I'm wondering if it's an expected behavior or a bug?
The text was updated successfully, but these errors were encountered: