Skip to content

fix mic offload for mkl 11.3#915

Merged
boegel merged 3 commits intoeasybuilders:developfrom
stachon:master
May 12, 2016
Merged

fix mic offload for mkl 11.3#915
boegel merged 3 commits intoeasybuilders:developfrom
stachon:master

Conversation

@stachon
Copy link
Copy Markdown
Contributor

@stachon stachon commented May 11, 2016

A directory has been renamed for MIC_LD_LIBRARY_PATH in imkl 11.3. I am not sure if this applies also to imkl 11.2

@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@boegel boegel added this to the v2.8.0 milestone May 11, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented May 11, 2016

Jenkins: ok to test

if LooseVersion(self.version) >= LooseVersion('11.0'):
if LooseVersion(self.version) >= LooseVersion('11.1'):
if LooseVersion(self.version) >= LooseVersion('11.3'):
retdict['MIC_LD_LIBRARY_PATH'] = ['lib/intel64_lin_mic', 'mkl/lib/mic'];
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mkl/lib/mic is a symlink to mkl/lib/intel64_lin_mic in Intel MKL v11.3.x, so use the latter?

$ ls -ld imkl/11.3.*/mkl/lib/*mic
drwxr-xr-x 3 vsc40023 vsc40023 4096 Mar 17 11:46 imkl/11.3.0.109-iimpi-2016.00-GCC-4.9.3-2.25/mkl/lib/intel64_lin_mic
lrwxrwxrwx 1 vsc40023 vsc40023   17 Mar 17 11:45 imkl/11.3.0.109-iimpi-2016.00-GCC-4.9.3-2.25/mkl/lib/mic -> ./intel64_lin_mic
drwxr-xr-x 3 vsc40023 vsc40023 4096 Mar 17 11:52 imkl/11.3.1.150-iimpi-2016.01-GCC-4.9.3-2.25/mkl/lib/intel64_lin_mic
lrwxrwxrwx 1 vsc40023 vsc40023   17 Mar 17 11:52 imkl/11.3.1.150-iimpi-2016.01-GCC-4.9.3-2.25/mkl/lib/mic -> ./intel64_lin_mic
drwxr-xr-x 3 vsc40023 vsc40023 4096 Mar 17 11:55 imkl/11.3.2.181-iimpi-2016.02-GCC-4.9.3-2.25/mkl/lib/intel64_lin_mic
lrwxrwxrwx 1 vsc40023 vsc40023   17 Mar 17 11:54 imkl/11.3.2.181-iimpi-2016.02-GCC-4.9.3-2.25/mkl/lib/mic -> ./intel64_lin_mic
drwxr-xr-x 3 vsc40023 vsc40023 4096 May  8 07:58 imkl/11.3.3.210-iimpi-2016.03-GCC-5.3.0-2.26/mkl/lib/intel64_lin_mic
lrwxrwxrwx 1 vsc40023 vsc40023   17 May  8 07:57 imkl/11.3.3.210-iimpi-2016.03-GCC-5.3.0-2.26/mkl/lib/mic -> ./intel64_lin_mic

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it does not matter...the easyblock already references intel64 in several places which is actually a symlink to intel64_lin

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, yeah, ok, doesn't really matter, just figured it would be better to include the actual directory rather than the symlink (for massive speedups ;-))

@boegel
Copy link
Copy Markdown
Member

boegel commented May 11, 2016

@stachon in imkl 11.2.x, it's still lib/mic and mkl/lib/mic, I checked, so your version check is OK

@hpcugentbot
Copy link
Copy Markdown

Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1980/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 12, 2016

Verified, going in, thanks @stachon!

@boegel boegel merged commit d41241e into easybuilders:develop May 12, 2016
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.

3 participants