Bug 1435545

Summary: llvm produces illegal instructions for SKL i7 6600U with pocl
Product: [Fedora] Fedora Reporter: Abrahm Scully <abrahm.scully>
Component: llvmAssignee: Tom Stellard <tstellar>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 25CC: ajax, bos, davejohansen, dmalcolm, jistone, nalimilan, orion, petersen, scottt.tw, tstellar
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: llvm-3.9.1-5.fc25 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-08-31 20:51:03 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:
Attachments:
Description Flags
clpeak under gdb with disassembly
none
fix illegal instruction generation on SKL
none
identify kabylake cpus
none
clpeak output after patches none

Description Abrahm Scully 2017-03-24 07:48:38 UTC
Created attachment 1265982 [details]
clpeak under gdb with disassembly

Description of problem:

Running clpeak with pocl and llvm 3.9 crashes with illegal instruction on an intel i7-6600U.

Version-Release number of selected component (if applicable):
llvm 3.9.1-2

How reproducible:
100%

Steps to Reproduce:
1. Install pocl.
2. Install clpeak.
3. Run clpeak.

Actual results:
clpeak crashes on illegal instruction.

Expected results:
clpeak completes without crashing.

Additional info:

Comment 1 Abrahm Scully 2017-03-24 07:52:54 UTC
Created attachment 1265983 [details]
fix illegal instruction generation on SKL

This issue was fixed with llvm svn commit r291084.

Comment 2 Abrahm Scully 2017-03-24 07:57:44 UTC
Created attachment 1265984 [details]
identify kabylake cpus

I happen to have a skylake XPS 13 9350 and a kabylake 9360 for debugging... Only the skylake crashed with the illegal instruction, which is why I starting debugging this since they laptops are so similar.

llvm changeset r291086 adds the CPU ids for my 9360.

Comment 3 Abrahm Scully 2017-03-24 08:01:05 UTC
Created attachment 1265985 [details]
clpeak output after patches

Added the two patches to the llvm spec file and rebuild with mock. After installing the patched llvm-libs-3.9.1, clpeak runs fine on both laptops.

Comment 4 Milan Bouchet-Valat 2017-03-24 13:41:41 UTC
The report is about the main (unversioned) llvm package, though we may want to apply the patches to llvm3.9 too.

Comment 5 Fedora Admin XMLRPC Client 2017-05-04 13:57:37 UTC
This package has changed ownership in the Fedora Package Database.  Reassigning to the new owner of this component.

Comment 6 Fedora Update System 2017-08-25 21:40:41 UTC
llvm-3.9.1-5.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-3ff7bcde09

Comment 7 Fedora Update System 2017-08-26 22:37:55 UTC
llvm-3.9.1-5.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-3ff7bcde09

Comment 8 Fedora Update System 2017-08-31 20:51:03 UTC
llvm-3.9.1-5.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.