Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 213780 - Wrong CPU instruction set USE flags and ewarns in mplayer ebuilds
Summary: Wrong CPU instruction set USE flags and ewarns in mplayer ebuilds
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Gentoo Media-video project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-18 06:20 UTC by nm (RETIRED)
Modified: 2008-03-18 14:56 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description nm (RETIRED) gentoo-dev 2008-03-18 06:20:07 UTC
The mplayer ebuilds have an incorrect warning message about ssse3.

"pni" (Prescott New Instructions) is *NOT* the same thing as "ssse3". PNI is the same as SSE3, a common feature of all AMD64 and Intel Core/Core2 chips.

SSSE3 is a completely different instruction set, one that AMD64 chips do *not* support. It's found on newer Intel chips, and was an interim instruction set between SSE3 and SSE4/SSE4.1/SSE4.2.

Please fix the ebuilds by changing the USE flags to SSE3 (or if it really is for SSSE3, then forget about it working on AMD64 chips; it's Intel-only), and by altering the warn to say "pni/sse3".

For reference:
http://en.wikipedia.org/wiki/SSSE3
http://en.wikipedia.org/wiki/SSE3
http://en.wikipedia.org/wiki/SSE4
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2008-03-18 07:32:43 UTC
I really don't understand what's your trouble with the ebuild??? (And yeah it really is SSSE3, not SSE3).

$ euse -i ssse3
[-    ] ssse3 (media-video/mplayer):
faster floating point optimization for SSSE3 capable chips (Intel Core 2 and later chips)

Comment 2 Ben de Groot (RETIRED) gentoo-dev 2008-03-18 10:25:18 UTC
The way the warning is written now suggests that pni is the same as ssse3, which it isn't, or that there would be a useflag for pni, which there isn't. We should just drop the "pni/" part in the warning.
Comment 3 Steve Dibb (RETIRED) gentoo-dev 2008-03-18 14:34:12 UTC
Josh,

Just following upstream's naming scheme:

$ ./configure --help | grep -i ssse3
  --enable-ssse3            enable SSSE3 [autodetect]

Also, the ewarn was already removed in latest mplayer ( mplayer-1.0_rc2_p26258 )
Comment 4 Steve Dibb (RETIRED) gentoo-dev 2008-03-18 14:56:51 UTC
Okay well, I thought it was removed, so fixed on both, now.