Skip to content
This repository has been archived by the owner on May 27, 2021. It is now read-only.

expose UVM API #93

Merged
merged 1 commit into from
Mar 30, 2018
Merged

expose UVM API #93

merged 1 commit into from
Mar 30, 2018

Conversation

vchuravy
Copy link
Member

@vchuravy vchuravy commented Feb 2, 2018

I mocked this up to test how UVM would impact an application.
This is low priority for me so I won't be spending much time on it.

@maleadt
Copy link
Member

maleadt commented Feb 2, 2018

I think we need a Buffer type in Base (eg. JuliaLang/julia#12447) before tackling this.

@vchuravy
Copy link
Member Author

vchuravy commented Feb 3, 2018

Yeah it is much less useful if you only use it for GPU memory

@maleadt
Copy link
Member

maleadt commented Mar 27, 2018

Maybe we should make this part of the type, so that CuArray can have an optimized convert to Array that just does unsafe_wrap.
EDIT: or just make CuArray use this all the time.

@maleadt
Copy link
Member

maleadt commented Mar 30, 2018

Rebased.

@maleadt
Copy link
Member

maleadt commented Mar 30, 2018

Yeah it is much less useful if you only use it for GPU memory

So that turned out to be false, and unified memory is very useful for handling multi-GPU setups.
Let's merge if CI agrees.

@maleadt maleadt merged commit 577a42d into master Mar 30, 2018
@maleadt maleadt deleted the vc/uvm branch March 30, 2018 04:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants