[Buildroot] [PATCH] package/ffmpeg: fix build on mips_32
Arnout Vandecappelle
arnout at mind.be
Wed Sep 22 20:34:57 UTC 2021
On 13/09/2021 22:45, Giulio Benetti wrote:
> mips_32 is not supported by ffmpeg and it tries to build with loongson3
> SIMD support that leads to build failure due to:
> /tmp/ccFO2LRa.s: Assembler messages:
> /tmp/ccFO2LRa.s:15314: Error: opcode not supported on this processor: mips32 (mips32) `dmult $2,$6'
> /tmp/ccFO2LRa.s:15316: Error: opcode not supported on this processor: mips32 (mips32) `dsrl $2,$2,32'
I don't think *any* of the archs we define in Config.in.mips supports these
instructions... It's just that these are the only ones we test in the autobuilders.
>
> So let's --disable-asm to prevent using those unsupported opcodes.
>
> Fixes:
> http://autobuild.buildroot.net/results/f01/f01d9cedec8e1b371308d0f7af561a75883fa27c/
>
> Signed-off-by: Giulio Benetti <giulio.benetti at benettiengineering.com>
> ---
> package/ffmpeg/ffmpeg.mk | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/ffmpeg/ffmpeg.mk b/package/ffmpeg/ffmpeg.mk
> index 8ff8ea8c6c..5e405d0f60 100644
> --- a/package/ffmpeg/ffmpeg.mk
> +++ b/package/ffmpeg/ffmpeg.mk
> @@ -508,7 +508,7 @@ FFMPEG_CONF_OPTS += --enable-mipsfpu
> endif
>
> # Fix build failure on "addi opcode not supported"
> -ifeq ($(BR2_mips_32r6)$(BR2_mips_64r6),y)
> +ifeq ($(BR2_mips_32)$(BR2_mips_32r6)$(BR2_mips_64r6),y)
> FFMPEG_CONF_OPTS += --disable-asm
Given that mips seems so problematic, I think it's best to disable asm on mips
in general, not even for specific subarches.
If someone needs some specific subarch to be optimized, they can give us a
(tested!) patch that enables asm again for that specific subarch.
Regards,
Arnout
> endif
> endif # MIPS
>
More information about the buildroot
mailing list