Bug 1515404

Summary: linking cp2k with libxsmm makes cp2k crash with SIGILL
Product: [Fedora] Fedora Reporter: Dominik 'Rathann' Mierzejewski <dominik>
Component: libxsmmAssignee: Dave Love <dave.love>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 28CC: dave.love, dominik
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-03-22 10:12:26 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Dominik 'Rathann' Mierzejewski 2017-11-20 18:29:10 UTC
Description of problem:
Per https://groups.google.com/forum/#!searchin/cp2k/hf.pabst@gmail.com%7Csort:date/cp2k/OS6M94u63ts/Dk4HqAitAgAJ I am building cp2k with:
-D__LIBXSMM in DFLAGS and -lxsmmf -lxsmm -ldl in LIBS. However, the resulting binary crashes with SIGILL every time.

Version-Release number of selected component (if applicable):
libxsmm-1.8.1-2.fc28
cp2k-5.1-1.fc28

How reproducible:
Always.

Steps to Reproduce:
1. Build cp2k with libxsmm enabled.
2. Run testsuite.

Actual results:
Every test crashes with SIGILL.

Expected results:
Testsuite passes.

Additional info:
This is a placeholder only so as not to forget this. cp2k-5.1 update is still not committed to rawhide.

Comment 1 Dave Love 2017-11-21 11:23:22 UTC
Oops.  That looks like something I fixed locally and didn't push.  Try again now.
(I didn't notice on the first got that it needs an extra FLAG compared with previously.)

Comment 2 Dominik 'Rathann' Mierzejewski 2017-11-21 14:31:32 UTC
Thanks, will retry with the new build sometime this week (hopefully).

Comment 3 Dominik 'Rathann' Mierzejewski 2017-11-21 17:10:41 UTC
By the way, I think you should pass %makeflags to make test, too, otherwise the library gets built again with different flags and a different version is used for test than for standard build.

Comment 4 Dominik 'Rathann' Mierzejewski 2017-12-18 18:59:19 UTC
Just to confirm, the new build (1.8.1-3.fc28) doesn't crash cp2k anymore, but you should still address comment #3.

Comment 5 Dave Love 2017-12-19 15:22:19 UTC
(In reply to Dominik 'Rathann' Mierzejewski from comment #4)
> Just to confirm, the new build (1.8.1-3.fc28) doesn't crash cp2k anymore,
> but you should still address comment #3.

Yes, I should have investigated further and asked Pabst about it.  Thanks for catching that.
I've disabled the test for now and rebuilt it.

Comment 6 Fedora End Of Life 2018-02-20 15:37:09 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 28 development cycle.
Changing version to '28'.