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

Transient Pool #110

Merged
53 commits merged into from
Jul 11, 2022
Merged

Transient Pool #110

53 commits merged into from
Jul 11, 2022

Conversation

ryanolson
Copy link
Contributor

/**
 * @brief ReusablePool of memory::buffers that are used as reusable reference-counted monotonic memory resources
 *
 * TransientPool is a ReusablePool of memory::buffers from which smaller buffers are allocated similar to a monotonic
 * memory resource, i.e. pointer pusing stack; however the TransientBuffer or Transient<T> object pulled from the pool
 * hold a SharedReusable<memory::buffer> which keeps the entire monotonic stack from returning to the ReusablePool
 * until all transient objects created on a given stack are deallocated.
 *
 * Allocation of Transient object should be incredibly fast; even faster than the Reusable/SharedResuable on which they
 * are based, since a single Reusable<memory::buffer> might back 10s-1000s of allocations depending on size.
 *
 * It is critical that all Transient object allocated from a pool have similar life cycles
 */

@ryanolson ryanolson requested review from a team as code owners July 2, 2022 03:45
@ryanolson ryanolson self-assigned this Jul 2, 2022
@ryanolson ryanolson added non-breaking Non-breaking change feature request New feature or request labels Jul 2, 2022
@ryanolson ryanolson added this to the Multi-Node Support milestone Jul 2, 2022
@ryanolson
Copy link
Contributor Author

This was already approved as #107 but was closed by the CI bot.

@mdemoret-nv
Copy link
Contributor

@gpucibot merge

@ghost ghost merged commit e634fb7 into nv-morpheus:branch-22.08 Jul 11, 2022
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Non-breaking change
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants