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

Asynchronous memory transfer #123

Open
LWisteria opened this issue Sep 18, 2018 · 5 comments
Open

Asynchronous memory transfer #123

LWisteria opened this issue Sep 18, 2018 · 5 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@LWisteria
Copy link
Member

Currently, all async memory operation are implemented as sync.

They are should be implemented as async.

@LWisteria LWisteria added the enhancement New feature or request label Sep 18, 2018
@LWisteria LWisteria added the good first issue Good for newcomers label Feb 5, 2019
@ybsh ybsh self-assigned this Jul 26, 2019
@ybsh
Copy link
Collaborator

ybsh commented Jul 26, 2019

@LWisteria When the source and destination of copy_from_device_async() are both device memory objects (cl_mem), should it raise an error, or execute EnqueueCopyBuffer just as in copy_from_device()?

@LWisteria
Copy link
Member Author

@ybsh I understand that your question is about clpy itself but not about this issue. If you suggest raising an error on copy_from_device() (and also on _async), please make an issue for it.

@ybsh
Copy link
Collaborator

ybsh commented Jul 26, 2019

@LWisteria
Thank you for your advice, and sorry that was off-topic.
Aside from the error handling issue above, I think it is necessary to migrate CuPy's Event/Stream classes first, as those async family take a CUDA stream as one of their arguments.
Do you agree? If you do, could I open an issue for that?

@ybsh
Copy link
Collaborator

ybsh commented Jul 31, 2019

Regarding the migration of CUDA Stream, @LWisteria told me an issue for that has already been created: #5

@LWisteria
Copy link
Member Author

This issue would be pending because _async functions are not used in cupy tests and even in chainer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants