Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 888073 - media-video/mplayer-1.4-r1: ebuild patch proposal for finer x86 properties tuning
Summary: media-video/mplayer-1.4-r1: ebuild patch proposal for finer x86 properties tu...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal enhancement
Assignee: Gentoo Media-video project
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2022-12-23 12:41 UTC by Jocelyn Mayer
Modified: 2022-12-24 10:22 UTC (History)
3 users (show)

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


Attachments
Ebuild patch proposal for finer x86 CPU features tuning (media-video_mplayer_mplayer-1.4-r1.ebuild.patch,1.74 KB, patch)
2022-12-23 12:43 UTC, Jocelyn Mayer
Details | Diff
Output from emerge --info =media-video/mplayer-1.4-r1 (emerge-infos-mplayer-1.4-r1,24.24 KB, text/plain)
2022-12-23 12:43 UTC, Jocelyn Mayer
Details
Emerge build log with patch applied (mplayer-1.4-r1:20221223-120247.log.gz,42.74 KB, application/gzip)
2022-12-23 12:45 UTC, Jocelyn Mayer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jocelyn Mayer 2022-12-23 12:41:08 UTC
In the current ebuild, not all CPU flags checked by the mplayer build system are set from the CPU_FLAGS_X86.
The proposed patch is to add more cpu_flags_x86_xxx in IUSE and thus in myconf in order to get optimizations more consistent with the whole system configuration.
The only drawback is that, for sse4_1 and sse4_2, there is no direct mapping between CPU_FLAGS_X86 and configure flags then those 2 cases have to be checked by specific directive.

Reproducible: Always

Steps to Reproduce:
1. Emerge mplayer
2. Check the build log

Actual Results:  
avx avx2 fma3 fma4 sse3 sse4 sse42 and xop flags are not set by the ebuild but detected by the configure script.


Expected Results:  
After adding more cpu_flags_x86 to IUSE and build configuration, the only CPU flag I saw in the build not set by the ebuild is cmov.
Comment 1 Jocelyn Mayer 2022-12-23 12:43:06 UTC
Created attachment 844799 [details, diff]
Ebuild patch proposal for finer x86 CPU features tuning
Comment 2 Jocelyn Mayer 2022-12-23 12:43:44 UTC
Created attachment 844801 [details]
Output from emerge --info =media-video/mplayer-1.4-r1
Comment 3 Jocelyn Mayer 2022-12-23 12:45:06 UTC
Created attachment 844803 [details]
Emerge build log with patch applied

Note that the patch submitted for this issue:
https://bugs.gentoo.org/861140
has also been applied to get a successful merge
Comment 4 Larry the Git Cow gentoo-dev 2022-12-24 10:21:32 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8cdb49a98ff943afe53695735fb3eed4b77c3b28

commit 8cdb49a98ff943afe53695735fb3eed4b77c3b28
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-12-24 10:16:46 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-12-24 10:19:57 +0000

    media-video/mplayer: add 1.5
    
    Unclear if anything happened with security bug #858107 though.
    
    Bug: https://bugs.gentoo.org/858107
    Bug: https://bugs.gentoo.org/870406
    Closes: https://bugs.gentoo.org/728640
    Closes: https://bugs.gentoo.org/836428
    Closes: https://bugs.gentoo.org/870649
    Closes: https://bugs.gentoo.org/888073
    Signed-off-by: Sam James <sam@gentoo.org>

 media-video/mplayer/Manifest            |   1 +
 media-video/mplayer/mplayer-1.5.ebuild  | 627 ++++++++++++++++++++++++++++++++
 media-video/mplayer/mplayer-9999.ebuild | 378 ++++++++++---------
 3 files changed, 829 insertions(+), 177 deletions(-)