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

Managed memory on /src/norm.cu #16

Open
AnderBiguri opened this issue Mar 27, 2020 · 0 comments
Open

Managed memory on /src/norm.cu #16

AnderBiguri opened this issue Mar 27, 2020 · 0 comments

Comments

@AnderBiguri
Copy link

AnderBiguri commented Mar 27, 2020

Hi!
I am just checking around the CUDA code to understand few things, and when looking at the code for the normalization, I found several instances of the following:

https://github.com/NiftyPET/NIPET/blob/master/niftypet/nipet/src/norm.cu#L94-L98

It looks like you are allocating managed memory if its not a windows 32 OS. I have some questions about why this is done:

  1. Managed memory will behave differently in pre and post 6.x cc, so you may get different performance, "in the general case"

  2. This code does not seem to need managed memory (if I am not wrong, please correct me!). You allocate an array on GPU and use it on the kernel, then free it. Managed memory will likely cause just slower execution. Why are you using it? Is it maybe because you are worried of running out of RAM, and it may help in the cc>6.0 archs ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants