Bug 781874

Summary: DirectFB should be built with MMX on ix86
Product: [Fedora] Fedora Reporter: Ville Skyttä <ville.skytta>
Component: directfbAssignee: Matthias Saou <matthias>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: kwizart, matthias
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-07-22 12:37:28 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Ville Skyttä 2012-01-15 21:16:50 UTC
DirectFB is built with --disable-mmx on ix86.  I believe --disable-mmx should be removed because this is just something that controls whether MMX support is compiled in -- whether to actually use MMX is detected by DirectFB at runtime.

Not sure about --disable-sse, it "smells" the same as --disable-mmx but I couldn't find anything that would be actually affected by --disable-sse on a quick look at the sources.

Comment 1 Nicolas Chauvet (kwizart) 2012-01-19 16:28:09 UTC
Thx for the report Ville.
Actually I lack time to double check this for the current release. I just remember some options like that was needed for PPC previously. Now it would be best to test for the coming next release unless you want to have it in current releases.

So it would be best for you to request the ACL for directfb actually, as I don't want to be a locker in this case.

Comment 2 Ville Skyttä 2012-01-19 18:03:01 UTC
Being a provenpackager, I don't think I need an ACL in order to be able to push and build it.  If you want, I can push a build that removes --disable-mmx and --disable-sse for Rawhide, but please be aware that I don't have a single ix86 box to test this with (x86_64 only here, I just noticed the build flags while looking for something else).

Comment 3 Ville Skyttä 2012-02-07 19:30:24 UTC
(In reply to comment #2)
> If you want, I can push a build that removes --disable-mmx and
> --disable-sse for Rawhide

Thoughts?

Comment 4 Nicolas Chauvet (kwizart) 2012-04-25 20:42:53 UTC
Looking at that again, and I think the disablement was intended.
We cannot suppose mmx or sse on i686 whereas sse2 is a default on x86_64 ?

Comment 5 Ville Skyttä 2012-04-26 04:10:28 UTC
Please see the initial comment...

Comment 6 Nicolas Chauvet (kwizart) 2012-04-26 20:14:31 UTC
OK, I've finally got it. Actually I haven't seen any occurrence of USE_SSE in the code.

Comment 7 Fedora Update System 2012-04-26 20:40:44 UTC
directfb-1.5.3-5.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/directfb-1.5.3-5.fc17

Comment 8 Fedora Update System 2012-05-04 23:06:37 UTC
directfb-1.5.3-5.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.