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

Fix: TargetBackend object leak in onnc #140

Merged
merged 1 commit into from Mar 15, 2019

Conversation

Projects
None yet
2 participants
@poyenc
Copy link
Contributor

poyenc commented Mar 14, 2019

[Problem]
Memory leak occurs when running onnc to compile models.

[Root Cause]

  1. The program onnc never destroys the created TargetBackend object.
  2. NvDlaBackend does not release all acquired memory.

[Solution]
Add correspond resource release logics.

@poyenc poyenc requested a review from a127a127 Mar 14, 2019

@poyenc poyenc changed the title Fix `TargetBackend` object leak in onnc Fix TargetBackend object leak in onnc Mar 14, 2019

@poyenc poyenc changed the title Fix TargetBackend object leak in onnc Fix: TargetBackend object leak in onnc Mar 14, 2019

@a127a127 a127a127 merged commit fce697b into master Mar 15, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@a127a127 a127a127 deleted the feature/fix-target-backend-object-leak branch Mar 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.