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

runtime: use std::function for callback logic #5

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mormj
Copy link
Contributor

@mormj mormj commented Aug 24, 2021

Signed-off-by: Josh Morman [email protected]

David Sorber and others added 3 commits August 11, 2021 07:42
Custom Buffer/Accelerator Device Support - Milestone 1 changes:

    * Refactored existing single mapped buffer code and created single
      mapped buffer abstraction; wrapping within single mapped buffers
      is handled explicitly by input blocked and output blocked
      callbacks that are called from block_executor
    * Added simple custom buffer allocation interface (NOTE: this
      interface will change for milestone 2)
    * Accelerated blocks are still responsible for data transfer but the
      custom buffer interface eliminates the double copy problem

Signed-off-by: David Sorber <[email protected]>
Completion of custom buffer/accelerator device support changes:

    * Improved custom buffer interface by removing awkward memory
      allocation functions from the block class
    * Increased flexibility for creating custom buffers by allowing
      creation of buffer_single_mapped subclasses
    * Fully incorporated data movement abstraction into the custom
      buffer interface and the runtime itself; accelerated blocks are no
      longer directly responsible for their own data movement
    * Zero copy back-to-back accelerated blocks are now supported (data
      no longer needs to be moved back to the host between each block)

Signed-off-by: David Sorber <[email protected]>
Signed-off-by: Mike Mason <[email protected]>
@dsorber dsorber force-pushed the master branch 5 times, most recently from 089cc19 to ad391b0 Compare October 18, 2021 15:46
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

Successfully merging this pull request may close these issues.

1 participant