Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Issues installing on Linux with nvidia #55

Open
tiagoantao opened this issue Mar 16, 2017 · 42 comments
Open

Issues installing on Linux with nvidia #55

tiagoantao opened this issue Mar 16, 2017 · 42 comments

Comments

@tiagoantao
Copy link

When I installed node-opencl and run the examples, I got an immediate segmentation fault.
My configuration is as follows:

  • Ubuntu 16.04
  • Nvidia Tesla K40
  • NVidia OpenCL bindings that are v 1.2 (not 2)

It so happens that the opencl headers that are made available with ubuntu (package opencl-headers) are 2.0

This means that, for example, commandqueue.cpp #ifndef CL_VERSION_2_0 will add the wrong function on compilation.

My hack was to edit the headers and undefine CL_VERSION_2_0

I do not have the experience to make a recommendation here, maybe this is just a problem with the way things are set up on ubuntu, but I think one of these might work

  • document this so that people are aware OR
  • detect the openCL version of the device in a different, more dynamic way
@robertleeplummerjr
Copy link

Did you get any further here?

@mikeseven
Copy link
Owner

mikeseven commented Nov 17, 2017 via email

@robertleeplummerjr
Copy link

It looks like this library will be used by http://gpu.rocks, fyi. You think there will be sufficient support?

@mikeseven
Copy link
Owner

mikeseven commented Nov 17, 2017 via email

@robertleeplummerjr
Copy link

exactly. the architectural layer would give us a means of finally having a full on GPU + js api. Design in simple js, execute... everywhere.

@robertleeplummerjr
Copy link

This is my exact issue on Ubuntu:

>  require('node-opencl/examples/saxpy');
SAXPY with vector size: 524288 elements
using device: GeForce GTX 1080 Ti
node: relocation error: /home/usergpu/node_modules/node-opencl/build/Release/opencl.node: symbol clCreateCommandQueueWithProperties, version OPENCL_2.0 not defined in file libOpenCL.so.1 with link time reference

@robertleeplummerjr
Copy link

robertleeplummerjr commented Nov 17, 2017 •
edited
Loading

I can provide ssh if you like.

@mikeseven
Copy link
Owner

mikeseven commented Nov 17, 2017 via email

@robertleeplummerjr
Copy link

Do you know what turnaround time will be? No rush.

@robertleeplummerjr
Copy link

Also: this is so exciting!

@mikeseven
Copy link
Owner

mikeseven commented Nov 17, 2017 via email

@TheLarkInn
Copy link

Bumping for status. I'm very very very interested in this endeavor especially for NodeJS core, and our library webpack for parallel processing.

@robertleeplummerjr
Copy link

@mikeseven did you get around to it yet?

@mikeseven
Copy link
Owner

I haven't tested extensively but it has been in feature branch for a while.

@robertleeplummerjr
Copy link

Nice: https://github.com/mikeseven/node-opencl/tree/feature/upgrade_to_NAN_2.8.0
So we just need to test, give feedback here, and based off that, improve and release or just release?

@mikeseven
Copy link
Owner

mikeseven commented Feb 16, 2018 via email

@TheLarkInn
Copy link

TheLarkInn commented Feb 16, 2018 via email

@mikeseven
Copy link
Owner

mikeseven commented Feb 16, 2018 via email

@robertleeplummerjr
Copy link

the benefit of gpu.js, with node-opencl (planned, in prototype stage) is that you won't have to learn any of it, just plain old good fashion javascript.

😎

@mikeseven
Copy link
Owner

mikeseven commented Feb 19, 2018 via email

@robertleeplummerjr
Copy link

As a developer of I can say: GPU.js uses webgl as one of its rendering targets, along with javascript, but it will use opencl soon as another: gpujs/gpu.js#52

It'd be easy to add any number of targets that don't support graphics backends.

@mikeseven
Copy link
Owner

mikeseven commented Feb 19, 2018 via email

@robertleeplummerjr
Copy link

To use opencl as backend, it would need to be outside the browser

Already done. GPU.js originally only used webgl, but now compiles a kernel just fine on node. So GPU.js has both node support and browser and handles typed arrays like Float32Array, just fine. Because we built it to work in a specific kernel environment, type detection is actually super easy.

Note: yes GPU.js actually compiles a javascript kernel, in javascript. This is done for performance, and... well, just because we can.

The javascript is compiled directly to whatever environment we need. It doesn't currently support an opencl kernel, which is why I started getting involved in this thread in the first place, all we need is a means of computing the numbers, that is easily installable. After that, the GPU.js library takes care of the rest. An example would be compiling a kernel similar to how it is done here in your unit tests: https://github.com/mikeseven/node-opencl/blob/master/test/test.kernel.js#L65

In short the javascript is compiled one to one with the subset of c++ environment (webgl or planned opencl, in the future: arrayfire, cuda, or perhaps webcl if it ever materializes).

@mikeseven
Copy link
Owner

mikeseven commented Feb 19, 2018 via email

@robertleeplummerjr
Copy link

Tried to install, this is what I get:

M86510-4:gpu.js robert.plummer$ npm i git://github.com/mikeseven/node-opencl.git#7d2de1fbddbd914e1baa41e3ed80287c717c583c
npm WARN deprecated [email protected]: pngjs2 has now taken over the original pngjs package on npm. Please change to use pngjs dependency, version 2+.

> [email protected] install /Users/robert.plummer/20% Time/gpu.js/node_modules/node-opencl
> node-gyp rebuild --msvs_version=auto

gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
  CXX(target) Release/obj.target/opencl/src/addon.o
  CXX(target) Release/obj.target/opencl/src/types.o
  CXX(target) Release/obj.target/opencl/src/common.o
  CXX(target) Release/obj.target/opencl/src/commandqueue.o
  CXX(target) Release/obj.target/opencl/src/context.o
  CXX(target) Release/obj.target/opencl/src/device.o
  CXX(target) Release/obj.target/opencl/src/event.o
  CXX(target) Release/obj.target/opencl/src/kernel.o
  CXX(target) Release/obj.target/opencl/src/memobj.o
  CXX(target) Release/obj.target/opencl/src/pipe.o
  CXX(target) Release/obj.target/opencl/src/platform.o
  CXX(target) Release/obj.target/opencl/src/program.o
  CXX(target) Release/obj.target/opencl/src/sampler.o
  CXX(target) Release/obj.target/opencl/src/svm.o
  SOLINK_MODULE(target) Release/opencl.node
[email protected] /Users/robert.plummer/20% Time/gpu.js
+-- [email protected]  extraneous
+-- [email protected]  invalid (git://github.com/mikeseven/node-opencl.git#7d2de1fbddbd914e1baa41e3ed80287c717c583c)
| +-- [email protected]
| | +-- [email protected] 
| | `-- [email protected] 
| +-- [email protected]
| | `-- [email protected] 
| `-- [email protected] 
`-- [email protected] 

M86510-4:gpu.js robert.plummer$

@robertleeplummerjr
Copy link

robertleeplummerjr commented Feb 20, 2018 •
edited
Loading

I ran it once more, using save, and this time no warnings:

M86510-4:gpu.js robert.plummer$ npm i --save git://github.com/mikeseven/node-opencl.git#7d2de1fbddbd914e1baa41e3ed80287c717c583c
npm WARN deprecated [email protected]: pngjs2 has now taken over the original pngjs package on npm. Please change to use pngjs dependency, version 2+.

> [email protected] install /Users/robert.plummer/20% Time/gpu.js/node_modules/node-opencl
> node-gyp rebuild --msvs_version=auto

gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
  CXX(target) Release/obj.target/opencl/src/addon.o
  CXX(target) Release/obj.target/opencl/src/types.o
  CXX(target) Release/obj.target/opencl/src/common.o
  CXX(target) Release/obj.target/opencl/src/commandqueue.o
  CXX(target) Release/obj.target/opencl/src/context.o
  CXX(target) Release/obj.target/opencl/src/device.o
  CXX(target) Release/obj.target/opencl/src/event.o
  CXX(target) Release/obj.target/opencl/src/kernel.o
  CXX(target) Release/obj.target/opencl/src/memobj.o
  CXX(target) Release/obj.target/opencl/src/pipe.o
  CXX(target) Release/obj.target/opencl/src/platform.o
  CXX(target) Release/obj.target/opencl/src/program.o
  CXX(target) Release/obj.target/opencl/src/sampler.o
  CXX(target) Release/obj.target/opencl/src/svm.o
  SOLINK_MODULE(target) Release/opencl.node
[email protected] /Users/robert.plummer/20% Time/gpu.js
+-- [email protected]  extraneous
+-- [email protected]  (git://github.com/mikeseven/node-opencl.git#7d2de1fbddbd914e1baa41e3ed80287c717c583c)
`-- [email protected]

@robertleeplummerjr
Copy link

I believe that means, we are release ready?

@robertleeplummerjr robertleeplummerjr mentioned this issue Feb 20, 2018
Closed
@mikeseven
Copy link
Owner

mikeseven commented Feb 20, 2018 via email

@robertleeplummerjr
Copy link

When I try one of the examples, I get cl.createContextFromType is not a function, I'm assuming I'm not doing something right. I did try just building a kernel, without running it, and it ran perfectly.

@mikeseven
Copy link
Owner

mikeseven commented Feb 20, 2018 via email

@robertleeplummerjr
Copy link

It is, sadly: https://support.apple.com/en-us/HT202823

Is there v1 example you could point me to?

@mikeseven
Copy link
Owner

mikeseven commented Feb 21, 2018 via email

@robertleeplummerjr
Copy link

fantastic!

@robertleeplummerjr robertleeplummerjr mentioned this issue Feb 23, 2018
Closed
@robertleeplummerjr
Copy link

any updates? Things are moving fast, and want to be able to use soon.

@mikeseven
Copy link
Owner

mikeseven commented Feb 25, 2018 via email

@robertleeplummerjr
Copy link

nice

@mikeseven
Copy link
Owner

mikeseven commented Feb 25, 2018 via email

@robertleeplummerjr
Copy link

It builds, but when I run unit tests (which are fantastically written) I get 72 failures, and then then error Command failed with exit code 72.. But a lot of them pass. Is that normal? If it is normal, can we get labels perhaps on the tests that identify them as to fail under what conditions?

@robertleeplummerjr
Copy link

robertleeplummerjr commented Mar 18, 2018 •
edited
Loading

fyi: all examples work 100%
I'd say at the very least, you have a release candidate.

@mikeseven
Copy link
Owner

mikeseven commented Mar 18, 2018 via email

@robertleeplummerjr
Copy link

What do you think timeline will be on the next release that allows for opencl 1.2 on Apple?

@mikeseven
Copy link
Owner

mikeseven commented Mar 18, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@tiagoantao @robertleeplummerjr @mikeseven @TheLarkInn