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

Pitched pointers #36

Open
samuelpowell opened this issue Jan 2, 2018 · 2 comments
Open

Pitched pointers #36

samuelpowell opened this issue Jan 2, 2018 · 2 comments
Labels
cuda libraries Stuff about CUDA library wrappers.

Comments

@samuelpowell
Copy link

Further to discussion about porting CUFFT.jl to use CUDAdrv (JuliaAttic/CUFFT.jl#12), it would be useful to provide support for CUDA pitched pointers, in addition to standard memory types.

How might such an implementation be best approached, given longer term plans for this package (e.g., is there still a plan to move to the CuArray implementation?).

@maleadt
Copy link
Member

maleadt commented Jan 2, 2018

cuMemAllocPitched is not currently wrapped indeed, and would probably need to be implemented as a method of alloc in the following: https://github.com/JuliaGPU/CUDAdrv.jl/blob/a8b2e3ca93a3a2be5b525f3bdf09425363b823c9/src/memory.jl#L145-L163

is there still a plan to move to the CuArray implementation?

Yes, I'm gradually moving towards deprecating CUDAdrv.CuArray, but am waiting for 0.7/1.0 to be supported by GPUArrays/CuArrays. In that world, using these new alloc methods would be a responsibility of an array type like CuArrays.jl, taking into account the overhead trade offs and effective pitch to index correctly.

@samuelpowell
Copy link
Author

Thanks Tim, that makes sense. I'll keep my eye on changes and attempt a PR when time permits.

@maleadt maleadt transferred this issue from JuliaGPU/CUDAdrv.jl May 27, 2020
@maleadt maleadt added the cuda libraries Stuff about CUDA library wrappers. label May 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda libraries Stuff about CUDA library wrappers.
Projects
None yet
Development

No branches or pull requests

2 participants