* [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5"
@ 2017-04-09 13:00 Bernd Kuhls
2017-04-09 13:00 ` [Buildroot] [PATCH v2 2/2] package/mplayer: disable inline asm on i386 Bernd Kuhls
2017-04-09 13:46 ` [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5" Thomas Petazzoni
0 siblings, 2 replies; 3+ messages in thread
From: Bernd Kuhls @ 2017-04-09 13:00 UTC (permalink / raw)
To: buildroot
This reverts commit bfb8df2ad9b164b421d25294c6882c8b61dc59a5.
With current toolchains it was not possible to trigger the bug anymore.
Tested with these settings:
BR2_i386=y
BR2_x86_i586=y
BR2_GCC_VERSION_4_8_X=y
BR2_i386=y
BR2_x86_i586=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_i586=y
BR2_GCC_VERSION_6_X=y
Other x86 variants also work:
BR2_i386=y
BR2_x86_i486=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_i686=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_pentium_mmx=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_corei7=y
BR2_GCC_VERSION_5_X=y
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: initial revision
package/ffmpeg/ffmpeg.mk | 5 -----
1 file changed, 5 deletions(-)
diff --git a/package/ffmpeg/ffmpeg.mk b/package/ffmpeg/ffmpeg.mk
index 348bc8212..223ed84cf 100644
--- a/package/ffmpeg/ffmpeg.mk
+++ b/package/ffmpeg/ffmpeg.mk
@@ -364,11 +364,6 @@ ifeq ($(BR2_X86_CPU_HAS_MMX),y)
FFMPEG_CONF_OPTS += --enable-yasm
FFMPEG_DEPENDENCIES += host-yasm
else
-ifeq ($(BR2_x86_i586),y)
-# Needed to work around a bug with gcc 5.x:
-# error: 'asm' operand has impossible constraints
-FFMPEG_CONF_OPTS += --disable-inline-asm
-endif
FFMPEG_CONF_OPTS += --disable-yasm
FFMPEG_CONF_OPTS += --disable-mmx
endif
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH v2 2/2] package/mplayer: disable inline asm on i386
2017-04-09 13:00 [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5" Bernd Kuhls
@ 2017-04-09 13:00 ` Bernd Kuhls
2017-04-09 13:46 ` [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5" Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Bernd Kuhls @ 2017-04-09 13:00 UTC (permalink / raw)
To: buildroot
A similar fix was committed to ffmpeg in the past:
https://git.buildroot.net/buildroot/commit/package/ffmpeg?id=bfb8df2ad9b164b421d25294c6882c8b61dc59a5
which will be reverted with another patch.
Mplayer contains ffmpeg 3.0, with current ffmpeg 3.2.4 the bug is gone,
the difference between both revisions is too big in order to find a
real fix for the problem.
Special cflags, added by
https://git.buildroot.net/buildroot/commit/package/multimedia/mplayer?id=fd38100e1189d19cad87a64c52df2c773eb47e40
are no longer needed now so remove them as well.
Testing these defconfigs did not produce compile errors with current
ffmpeg but with mplayer:
BR2_i386=y
BR2_x86_i586=y
BR2_GCC_VERSION_4_8_X=y
BR2_i386=y
BR2_x86_i586=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_i586=y
BR2_GCC_VERSION_6_X=y
BR2_i386=y
BR2_x86_i486=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_i686=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_pentium_mmx=y
BR2_GCC_VERSION_5_X=y
BR2_i386=y
BR2_x86_corei7=y
BR2_GCC_VERSION_5_X=y
Fixes
http://autobuild.buildroot.net/results/030/03066dd8937ef4c75d62f237fd195df92b247ee2//
and many others.
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: Clarified commit message (Thomas)
package/mplayer/mplayer.mk | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/package/mplayer/mplayer.mk b/package/mplayer/mplayer.mk
index 6b3009abc..14ab8acee 100644
--- a/package/mplayer/mplayer.mk
+++ b/package/mplayer/mplayer.mk
@@ -285,10 +285,13 @@ MPLAYER_CFLAGS += -mfpu=neon
endif
endif
+define MPLAYER_DISABLE_INLINE_ASM
+ $(SED) 's,#define HAVE_INLINE_ASM 1,#define HAVE_INLINE_ASM 0,g' \
+ $(@D)/config.h
+endef
+
ifeq ($(BR2_i386),y)
-# inline asm breaks with "can't find a register in class 'GENERAL_REGS'"
-# inless we free up ebp
-MPLAYER_CFLAGS += -fomit-frame-pointer
+MPLAYER_POST_CONFIGURE_HOOKS += MPLAYER_DISABLE_INLINE_ASM
endif
ifeq ($(BR2_X86_CPU_HAS_MMX),y)
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5"
2017-04-09 13:00 [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5" Bernd Kuhls
2017-04-09 13:00 ` [Buildroot] [PATCH v2 2/2] package/mplayer: disable inline asm on i386 Bernd Kuhls
@ 2017-04-09 13:46 ` Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2017-04-09 13:46 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 9 Apr 2017 15:00:02 +0200, Bernd Kuhls wrote:
> This reverts commit bfb8df2ad9b164b421d25294c6882c8b61dc59a5.
>
> With current toolchains it was not possible to trigger the bug anymore.
>
> Tested with these settings:
>
> BR2_i386=y
> BR2_x86_i586=y
> BR2_GCC_VERSION_4_8_X=y
>
> BR2_i386=y
> BR2_x86_i586=y
> BR2_GCC_VERSION_5_X=y
>
> BR2_i386=y
> BR2_x86_i586=y
> BR2_GCC_VERSION_6_X=y
>
> Other x86 variants also work:
>
> BR2_i386=y
> BR2_x86_i486=y
> BR2_GCC_VERSION_5_X=y
>
> BR2_i386=y
> BR2_x86_i686=y
> BR2_GCC_VERSION_5_X=y
>
> BR2_i386=y
> BR2_x86_pentium_mmx=y
> BR2_GCC_VERSION_5_X=y
>
> BR2_i386=y
> BR2_x86_corei7=y
> BR2_GCC_VERSION_5_X=y
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: initial revision
Thanks a lot for the extended explanation on the second patch, and
taking the time to look at ffmpeg as well. Both patches applied, thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-04-09 13:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-09 13:00 [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5" Bernd Kuhls
2017-04-09 13:00 ` [Buildroot] [PATCH v2 2/2] package/mplayer: disable inline asm on i386 Bernd Kuhls
2017-04-09 13:46 ` [Buildroot] [PATCH v2 1/2] Revert "package/ffmpeg: Fix i586 compilation with gcc5" Thomas Petazzoni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.