Bug 1369169

Summary: [pioneer] Disable build on Power and s390
Product: [Fedora] Fedora Reporter: Rafael Fonseca <rdossant>
Component: pioneerAssignee: Antonio T. (sagitter) <anto.trande>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 25CC: anto.trande, rosser.bjr
Target Milestone: ---   
Target Release: ---   
Hardware: ppc64   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-10-26 13:55:10 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:
Bug Depends On:    
Bug Blocks: 467765, 1071880, 1051573    
Attachments:
Description Flags
Make pioneer ExcludeArch for Power and s390 none

Description Rafael Fonseca 2016-08-22 15:05:42 UTC 
Created attachment 1192955 [details]
Make pioneer ExcludeArch for Power and s390

Description of problem: pioneer uses some x86-only asm instruction which do not work on Power or s390.

../../contrib/profiler/Profiler.h: In constructor 'BVHTree::BVHTree(int, const objPtr_t*, const Aabb*)':
../../contrib/profiler/Profiler.h:158:51: error: impossible register constraint in 'asm'
     asm volatile("rdtsc" : "=a" (__a), "=d" (__d));


Version-Release number of selected component (if applicable): all versions are affected


How reproducible: always on ppc-koji and s390-koji


Actual results: http://ppc.koji.fedoraproject.org/koji/taskinfo?taskID=3630383


Additional info: patch attached
Comment 1 Ben Rosser 2016-08-22 16:54:46 UTC 
It looks like this might be fixable without disabling building on those architectures; on ARM, the code sets a flag in the profiler [1] that disables the use of those instructions [2], and the spec has already been modified to set it on aarch64 too (in addition to other stuff) [3]. Possibly the same modifications will enable building on these architectures?

[1] https://github.com/pioneerspacesim/pioneer/blob/master/contrib/profiler/Profiler.h#L13

[2] https://github.com/pioneerspacesim/pioneer/blob/master/contrib/profiler/Profiler.h#L154

[3] http://pkgs.fedoraproject.org/cgit/rpms/pioneer.git/tree/pioneer.spec#n131
Comment 2 Rafael Fonseca 2016-08-22 17:25:26 UTC 
I'll try it out and report back.
Comment 3 Rafael Fonseca 2016-10-26 13:55:10 UTC 
pioneer is now ExclusiveArch in Fedora because of [1]. So I'm closing this bug.

[1] https://github.com/pioneerspacesim/pioneer/issues/3846