You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/05/05 16:30:54 UTC
[GitHub] [incubator-tvm] tqchen opened a new pull request #5512: [RPC] Call sync in remote cpu to gpu copies
tqchen opened a new pull request #5512:
URL: https://github.com/apache/incubator-tvm/pull/5512
cc @tmoreau89 @areusch @ZihengJiang
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tqchen edited a comment on pull request #5512: [RPC] Call sync in CopyFromRemote and CopyToRemote
Posted by GitBox <gi...@apache.org>.
tqchen edited a comment on pull request #5512:
URL: https://github.com/apache/incubator-tvm/pull/5512#issuecomment-624189332
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] tqchen commented on pull request #5512: [RPC] Call sync in remote cpu to gpu copies
Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #5512:
URL: https://github.com/apache/incubator-tvm/pull/5512#issuecomment-624189332
@areusch We only do it for CopyFromRemote and CopyToRemote. In both cases, we(user or rpc server) allocate temp meomory from the arena, which will become invalid after the response period. The user do not guarantee these byte arrays stays alive after the call ends. So that is why explicit sync is needed here.
For DeviceAPI::CopyDataFromTo, we do not run the sync, since the resources are NDArrays managed by the remote user, and the remote user can choose to call sync after the copy
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-tvm] areusch commented on pull request #5512: [RPC] Call sync in remote cpu to gpu copies
Posted by GitBox <gi...@apache.org>.
areusch commented on pull request #5512:
URL: https://github.com/apache/incubator-tvm/pull/5512#issuecomment-624186951
what's the justification for doing this just over RPC sessions?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org