Fixed documentation error in CUDA Coredump Blog #128
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In the CUDA Coredump blog, an incorrect documentation statement is made for this kernel:
// Kernel with illegal memory access - accesses memory beyond allocated bounds
global void normalKernel(int* data, int size) {
int idx = blockIdx.x * blockDim.x + threadIdx.x;
// This will cause illegal memory access - accessing beyond allocated memory
// We allocate 'size' elements but access up to size * 2
if (idx < size) { // Access twice the allocated size
data[idx] = idx; //
}
}
To my knowledge, this should not generate an illegal memory access, as the size allocated is fine in the source code as well. There is also an incomplete comment for data[idx] = idx; so I'm making the assumption that these comments are a mistake
I also removed the use of the phrase 'normal execution' as it can confuse readers who may think the author is referring to the execution of the kernel called 'normalKernel.' I have replaced the wording to just be 'execution.'