Bug#693040: [libav-devel] Strange build failure on several archs in Debian.

2012-11-13 Thread Måns Rullgård
Loïc Minier l...@dooz.org writes:

 I guess the libav build you're seeing with armv4 + softfp is the VFP
 pass for the Debian armel port.  It's using the toolchain defaults
 (armv4 + float-abi=soft) for the main build and turns on VFP (-mfpu=vfp
 --float-abi=softfp) for the VFP pass.

 If there's absolutely NO ARMv4 hardware with VFP ever going to run
 Debian armel and if enabling ARMv5 would greatly benefit the builds,
 then we could special case this and force -march=armv5 for the vfp pass
 if the toolchain defaults to armv4.

 However, if there's some ARMv4 hardware with VFP around, the vfp flavor
 of libav might get picked up by hwcaps and that will cause SIGILL at
 runtime if we enable ARMv5 instructions.

There has never been, nor will there ever be, any ARMv4 implementation
with VFP.  There exists a VFP design intended for use with ARM9E cores
(v5TE), but actual silicon with this combination is extremely rare.  In
practice, VFP is only found with ARM11 (v6) and later cores.

Allowing ARMv5TE instructions is definitely beneficial since this enables
the half-word multiply instructions which are at least twice as fast as
the full-size multiplies in v4.  GCC does use these instructions unaided
in many cases, and we have hand-written asm as well.

 I'm afraid I have no knowledge of what ARMv4 hardware we still care for
 in Debian, nor whether there's a hwcap for ARMv5 that we could use here;

The 'edsp' hwcap indicates an ARMv5TE or better core.

-- 
Måns Rullgård
m...@mansr.com

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers


Bug#642810: [x264-devel] configure fails to detect alphaev67 as ALPHA

2011-10-06 Thread Måns Rullgård
Fabian Greffrath fab...@greffrath.com writes:

 Hi all,

 Debian's alpha buildds identify themselves as alphaev67, so the check
 for $ARCH = ALPHA in configure around line 607 fails. As a
 consequence the shared library is built without PIC and the linker
 fails. This is documented in the Debian BTS #642810:
 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=642810

 The attached patch fixes this issue by considering alpha* as ALPHA in
 the configure script.

 Best Regards,
 Fabian Greffrath

 Author: Fabian Greffrath fabian+deb...@greffrath.com
 Description: Debian's alpha buildds identify themselves as alphaev67,
  so consider alpha* as ALPHA (Closes: #642810).
 Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=642810

 --- x264.orig/configure
 +++ x264/configure
 @@ -580,6 +580,9 @@ case $host_cpu in
  ia64)
  ARCH=IA64
  ;;
 +alpha*)
 +ARCH=ALPHA
 +;;
  *)
  ARCH=$(echo $host_cpu | tr a-z A-Z)
  ;;

Looks reasonable.

-- 
Måns Rullgård
m...@mansr.com



___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers