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.

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

Make use of the new travis docker infrastructure #104

Merged
merged 2 commits into from
Jan 18, 2016
Merged

Conversation

vchuravy
Copy link
Member

So the aim of this is to enable the docker infrastructure for travis testing. The problem is that one has to install a working OpenCL implementation manually.

I got the AMD APP SDK working, which is quite nice because we now can test OpenCL 2.0 on travis. The only show stopper right now is a seqfault in waitforevents

https://travis-ci.org/JuliaGPU/OpenCL.jl/jobs/102318248

@dfdx If I can fix the seqfault this should be quite nice, additionally we could think about installing pocl and testing our code against two implementations. I moved back to AMD because one can't install the Intel one without root rights.

@vchuravy vchuravy force-pushed the vc/travis_docker branch 2 times, most recently from 9521a3f to 1f85ab1 Compare January 14, 2016 11:08
@vchuravy
Copy link
Member Author

This is slightly frustrating. On my local computer I was able to reproduce the issue, but sometimes it is also just stalling (like the last travis build).

Also the tests are stalling on my local computer with the Intel SDK so it might be a mire general problem.

@dfdx
Copy link
Contributor

dfdx commented Jan 17, 2016

I'll check if I can reproduce stalling on tests with Intel SDK in my machine.

@dfdx
Copy link
Contributor

dfdx commented Jan 17, 2016

Hmm, I can't find libOpenCL.so after installing Intel SDK. How do you switch between different implementations?

With AMD and NVidia your branch works fine on my laptop.

@vchuravy
Copy link
Member Author

are you using AMD CPU or GPU? Handling the Intel SDK is a bit tricky, I will have to check once I am at work how I have setup things. Also sometimes the error occurs only during GC runs, so it might require several runs too hit it.

@vchuravy vchuravy mentioned this pull request Jan 18, 2016
@vchuravy
Copy link
Member Author

If you are using the Intel sdk you will also have to install the runtime, which apparently contains the .so

@vchuravy
Copy link
Member Author

I should tested it on my home computer and we pass all tests on 2014_R2. The library itself should be in /opt/intel/opencl-1.2-4.4.0.117/lib64/libOpenCL.so.

I will need to double check that I wasn't running the tests at work against beignet.

@vchuravy
Copy link
Member Author

using an older AMD SDK 2.9.1 seems to fix the test problems.

vchuravy added a commit that referenced this pull request Jan 18, 2016
Make use of the new travis docker infrastructure
@vchuravy vchuravy merged commit e3f14f2 into master Jan 18, 2016
@vchuravy vchuravy deleted the vc/travis_docker branch January 18, 2016 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants