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

Create assert memory tracking to shut down unused references #6

Open
PeteHorn opened this issue Dec 18, 2018 · 4 comments
Open

Create assert memory tracking to shut down unused references #6

PeteHorn opened this issue Dec 18, 2018 · 4 comments
Assignees

Comments

@PeteHorn
Copy link
Owner

Feedback from GDevCon that this is worth doing - no idea how though

@PeteHorn PeteHorn self-assigned this Dec 18, 2018
@PeteHorn
Copy link
Owner Author

PeteHorn commented Jan 8, 2019

Had a quick think about how to do this, looking through forums to see if there is a way of getting an event when a vi drops out of memory - nothing....

image

Could poll through all the calling vi's at regular intervals to monitor their execution state and shut down if they are idle or bad. This seems inefficient but I think the only possible way of doing it. Maybe only poll every 30 secs or something, or take a 100ms break every X callers to avoid it sucking CPU power too much.

image

@PeteHorn
Copy link
Owner Author

PeteHorn commented Jan 8, 2019

Started to think about this - added some new flags in to the config file with default values to manage how garbage/reference collection should work.

@PeteHorn
Copy link
Owner Author

PeteHorn commented Jan 8, 2019

image

@PeteHorn
Copy link
Owner Author

PeteHorn commented Mar 1, 2019

Created a process monitor actor which will have to be called from each assert vi. These should all test to see if it is open, and if not open it. If it is then get a reference to the one thats open - only ever want one instance of this open.

Then determine how to identify dead references (potentially through polling) and how to determine when the process monitor itself should be shut down....

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

1 participant