Bug 781874 - DirectFB should be built with MMX on ix86
Summary: DirectFB should be built with MMX on ix86
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: directfb
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Matthias Saou
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-15 21:16 UTC by Ville Skyttä
Modified: 2012-07-22 12:37 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-07-22 12:37:28 UTC
Type: ---


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.