benoitsteiner / tensorflow-opencl

OpenCL support for TensorFlow
Apache License 2.0
472 stars 86 forks source link

More than one SYCL device supported? #68

Open xangma opened 7 years ago

xangma commented 7 years ago

Hi all,

Looking to see if more than one SYCL is possible at the moment. I can see that the sycl device factory adds them incrementally upon building tensorflow-opencl from source, but the physical_device_desc: "device: 0, name SYCL, pci bus id: 0" is hardcoded in, which makes me think only one SYCL device is currently supported.

All my gpus show up in computecpp_info fine, and clinfo.

Does anyone know?

Thanks!

Environment info

Operating System: Ubuntu 14.04 Computecpp version: 0.1.4

rodburns commented 7 years ago

The most up to date branch to use is here https://github.com/lukeiwanski/tensorflow-opencl.git with changes being pushed to the main TensorFlow branch. I believe what you are seeing is fixed in that branch. The latest Pull Request for getting the latest to the main branch is here https://github.com/tensorflow/tensorflow/pull/9117

xangma commented 7 years ago

Thanks for your response. Doesn't seem to be fixed yet. I compiled the pull request changes and SYCL:1 is still not showing up sadly.

rodburns commented 7 years ago

We've updated the documentation now and there is a new branch that we believe will work for you. See here https://www.codeplay.com/products/computesuite/computecpp/guides/how-to-setup-tensorflow-with-computecpp We are propagating the changes to the main TensorFlow repository at the moment.