jaberg/Theano — comment on commit 24de195c9fa848f97b3192d9e0f8ef4f16fd279d
File:  line 0
By jaberg:

Ok, after much cursing and un-necessary detouring I've determined that
there is in fact an illegal read. The kernel iterates a group of rows
at a time, but it doesn't take care not to read past the end of the
image when reading the last group. On older hardware this was
allowed, but newer hardware / software is getting stricter about not
allowing reads of un-allocated memory. The old code was mathematically
correct, because there was logic to prevent *writing* the results
computed from these illegal reads.

I have added a fix to the PR, and also included some code cleanup &
documentation for conv.cu and conv_kernel.cu.

The f'ing unit tests are still running, and I can't wait longer for
them to finish. Let the buildbot take care of it.

On Wed, Mar 7, 2012 at 2:28 PM, nouiz

wrote:
> I tested it on more computer/gpu and I'm still not able to reproduice it. Did you ran the gpumemtest? If you want to look at the code in is in sandbox/cuda/conv_kernel.cu, in the function conv_rows_stack. The problem is probably related to the two call to load_to_shared.
>
> Did you try to clear your theano cache? Just in case.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jaberg/Theano/commit/24de195c9fa848f97b3192d9e0f8ef4f16fd279d#commitcomment-1055154