On Donnerstag 15 Januar 2009, you wrote:
> Andreas,
>
> After digging around a little bit, I found a few potential 'buggy' spots
> regarding Memcpy3D:
>
> in file:
> src/wrapper/wrap_cudadrv.cpp:557:      .def("__call__", &cl::execute,
> py::arg("aligned"))
> should it be
> src/wrapper/wrap_cudadrv.cpp:557:      .def("__call__", &cl::execute)
> ?

Applied.

> in the file src/cpp/cuda.hpp, should we replace reserved0 and reserverd1 by
> srcLOD and dstLOD respectively?

No, reserved{0,1} should stay around. srcLOD and dstLOD need to be initialized 
to 0 as well, though.

> What do you think?

Yay for untested code. :) Based on your initial test, I've added a unit test 
for 3D arrays and fixed the code so it passes. It should work ok in current 
git.

Andreas

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
PyCuda mailing list
[email protected]
http://tiker.net/mailman/listinfo/pycuda_tiker.net

Reply via email to