starpu-runtime / starpu

This is a mirror of https://gitlab.inria.fr/starpu/starpu where our development happens, but contributions are welcome here too!
https://starpu.gitlabpages.inria.fr/
GNU Lesser General Public License v2.1
60 stars 12 forks source link

There is no 'struct starpu_data_interface_ops' registered for interface 10 #1

Closed Muxas closed 2 years ago

Muxas commented 2 years ago

Hello!

I tried to use newly-added starpu_ndim_interface by registering data with help of starpu_ndim_data_register() and got the following error:

There is no 'struct starpu_data_interface_ops' registered for interface 10

I looked here https://github.com/starpu-runtime/starpu/blob/17431690948027ed4cea192e3214f3672f0b4222/src/datawizard/interfaces/data_interface.c#L157

and found that data interfaces for COO, TENSOR and NDIM are NOT presented.

Adding corresponding item above the "default" keyword solved the problem for me.

sthibaul commented 2 years ago

Ah, indeed, that's a place we don't seem to remember to update. I guess this happened while calling starpu_data_cpy?

sthibaul commented 2 years ago

Fixed by https://gitlab.inria.fr/starpu/starpu/-/commit/52c8b65791c2a26fb8cbb9300408b354376beab7