Bug 681739

Summary: atlas.spec gives wrong compiler flags for native build
Product: [Fedora] Fedora Reporter: William Mclean <mcleanbugzilla>
Component: atlasAssignee: Deji Akingunola <dakingun>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 14CC: dakingun
Target Milestone: ---   
Target Release: ---   
Hardware: i686   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-16 12:52:10 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Results of 'make time' none

Description William Mclean 2011-03-03 05:47:15 UTC 
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.13) Gecko/20110103 Fedora/3.6.13-1.fc14 Firefox/3.6.13

I rebuilt atlas setting enable_native_atlas equal to 1.  The configure script correctly detected support for SSE3 on my CPU (Intel Atom N550), but in the
%build step (lines 220-222) sed is used to remove the SSE compiler flags in Make.inc.  I think the spec file should leave these flags alone when building a native atlas.

Reproducible: Always

Steps to Reproduce:
1. Install source rpm on Fedora 14 i686.
2. From ~/rpmbuild/SPECS do rpmbuild -ba atlas.spec
3. Check ARCH (line 13) and ARCHDEFS (line 780) in 
   ~/rpmbuild/BUILD/ATLAS/i386_base/Make.inc
Actual Results:  
ARCH = PPRO32
ARCHDEFS = -DATL_OS_Linux -DATL_ARCH_UNKNOWNx86 -DATL_CPUMHZ=1000 -DATL_GAS_x8632



Expected Results:  
ARCH = UNKNOWNx8632SSE3
ARCHDEFS = -DATL_OS_Linux -DATL_ARCH_UNKNOWNx86 -DATL_CPUMHZ=1000 -DATL_SSE3 -DATL_SSE2 -DATL_SSE1 -DATL_GAS_x8632

Doing 'make time' confirms that the SSE compiler flags improves the performance
of the atlas kernels, by as much as a factor of 4; see attachment.
Comment 1 William Mclean 2011-03-03 05:50:01 UTC 
Created attachment 481993 [details]
Results of 'make time'

This shows timing results using the original atlas.spec and using a modified
spec file that keeps SSE compiler flags.
Comment 2 Fedora End Of Life 2012-08-16 12:52:12 UTC 
This message is a notice that Fedora 14 is now at end of life. Fedora 
has stopped maintaining and issuing updates for Fedora 14. It is 
Fedora's policy to close all bug reports from releases that are no 
longer maintained.  At this time, all open bugs with a Fedora 'version'
of '14' have been closed as WONTFIX.

(Please note: Our normal process is to give advanced warning of this 
occurring, but we forgot to do that. A thousand apologies.)

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, feel free to reopen 
this bug and simply change the 'version' to a later Fedora version.

Bug Reporter: Thank you for reporting this issue and we are sorry that 
we were unable to fix it before Fedora 14 reached end of life. If you 
would still like to see this bug fixed and are able to reproduce it 
against a later version of Fedora, you are encouraged to click on 
"Clone This Bug" (top right of this page) and open it against that 
version of Fedora.

Although we aim to fix as many bugs as possible during every release's 
lifetime, sometimes those efforts are overtaken by events.  Often a 
more recent Fedora release includes newer upstream software that fixes 
bugs or makes them obsolete.

The process we are following is described here: 
http://fedoraproject.org/wiki/BugZappers/HouseKeeping