Your message dated Mon, 24 Nov 2014 19:27:50 +0000 with message-id <1416857270.28376.20.camel@adam-barratt.org.uk> and subject line Re: Bug#770862: unblock: fftw3/3.3.4-2 has caused the Debian Bug report #770862, regarding unblock: fftw3/3.3.4-2 to be marked as done. This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact owner@bugs.debian.org immediately.) -- 770862: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=770862 Debian Bug Tracking System Contact owner@bugs.debian.org with problems
--- Begin Message ---
- To: Debian Bug Tracking System <submit@bugs.debian.org>
- Subject: unblock: fftw3/3.3.4-2
- From: Sébastien Villemot <sebastien@debian.org>
- Date: Mon, 24 Nov 2014 20:23:16 +0100
- Message-id: <[🔎] 20141124192313.GA19454@villemot.name>
Package: release.debian.org Severity: normal User: release.debian.org@packages.debian.org Usertags: unblock Dear Release Team, Please unblock fftw3. Version 3.3.4-2 fixes #767138, which was causing several packages to FTBFS on armhf (only on CPUs without NEON support, which includes many of our armhf buildds). Changelog and debdiff follow. fftw3 (3.3.4-2) unstable; urgency=medium * Team upload. * fix-runtime-neon-detection.patch: new patch, fixes runtime NEON detection on armhf (and also arm64, but NEON is currently unconditionnally disabled there). Thanks to Edmund Grimley Evans for the patch. (Closes: #767138) -- Sébastien Villemot <sebastien@debian.org> Sun, 23 Nov 2014 20:49:59 +0100 unblock fftw3/3.3.4-2 Thanks, -- .''`. Sébastien Villemot : :' : Debian Developer `. `' http://www.dynare.org/sebastien `- GPG Key: 4096R/381A7594diff -Nru fftw3-3.3.4/debian/changelog fftw3-3.3.4/debian/changelog --- fftw3-3.3.4/debian/changelog 2014-10-14 04:13:24.000000000 +0200 +++ fftw3-3.3.4/debian/changelog 2014-11-23 21:04:48.000000000 +0100 @@ -1,3 +1,13 @@ +fftw3 (3.3.4-2) unstable; urgency=medium + + * Team upload. + * fix-runtime-neon-detection.patch: new patch, fixes runtime NEON + detection on armhf (and also arm64, but NEON is currently + unconditionnally disabled there). Thanks to Edmund Grimley Evans for + the patch. (Closes: #767138) + + -- Sébastien Villemot <sebastien@debian.org> Sun, 23 Nov 2014 20:49:59 +0100 + fftw3 (3.3.4-1.1) unstable; urgency=low * Add mips64(el), ppc64el and arm64 to the list with long double support. diff -Nru fftw3-3.3.4/debian/patches/fix-runtime-neon-detection.patch fftw3-3.3.4/debian/patches/fix-runtime-neon-detection.patch --- fftw3-3.3.4/debian/patches/fix-runtime-neon-detection.patch 1970-01-01 01:00:00.000000000 +0100 +++ fftw3-3.3.4/debian/patches/fix-runtime-neon-detection.patch 2014-11-23 21:04:48.000000000 +0100 @@ -0,0 +1,103 @@ +Description: Fix runtime detection of NEON support on armhf and arm64 +Author: Edmund Grimley Evans <edmund.grimley.evans@gmail.com> +Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=767138 +Forwarded: no +Reviewed-by: Sébastien Villemot <sebastien@debian.org> +Last-Update: 2014-11-23 +--- +This patch header follows DEP-3: http://dep.debian.net/deps/dep3/ +--- a/simd-support/neon.c ++++ b/simd-support/neon.c +@@ -23,56 +23,45 @@ + + #if HAVE_NEON + +-/* check for an environment where signals are known to work */ +-#if defined(unix) || defined(linux) +- # include <signal.h> +- # include <setjmp.h> +- +- static jmp_buf jb; +- +- static void sighandler(int x) +- { +- UNUSED(x); +- longjmp(jb, 1); +- } +- +- static int really_have_neon(void) +- { +- void (*oldsig)(int); +- oldsig = signal(SIGILL, sighandler); +- if (setjmp(jb)) { +- signal(SIGILL, oldsig); +- return 0; +- } else { +- /* paranoia: encode the instruction in binary because the +- assembler may not recognize it without -mfpu=neon */ +- /*asm volatile ("vand q0, q0, q0");*/ +- asm volatile (".long 0xf2000150"); +- signal(SIGILL, oldsig); +- return 1; +- } +- } +- +- extern void X(check_alignment_of_sse2_pm)(void); +- +- int X(have_simd_neon)(void) +- { +- static int init = 0, res; +- +- if (!init) { +- res = really_have_neon(); +- init = 1; +- } +- return res; +- } ++#ifdef __linux__ + ++#ifdef __aarch64__ ++ ++/* HWCAP_ASIMD is defined in <asm/hwcap.h> but not included by <sys/auxv.h>. ++ Since all current AArch64 implementations have NEON/ASIMD it is probably ++ better to return 1 than include a header file which is not intended for ++ use by user programs. */ ++ ++int X(have_simd_neon)(void) ++{ ++ return 1; ++} + + #else +-/* don't know how to autodetect NEON; assume it is present */ +- int X(have_simd_neon)(void) +- { +- return 1; +- } ++ ++#include <sys/auxv.h> ++ ++int X(have_simd_neon)(void) ++{ ++ static int cached = 2; ++ int ret; ++ ++ /* This should be thread-safe in all reasonable circumstances. */ ++ ret = cached; ++ if (ret == 2) ++ { ++ ret = !!(getauxval(AT_HWCAP) & HWCAP_ARM_NEON); ++ cached = ret; ++ } ++ return ret; ++} ++ ++#endif ++ ++#else ++ ++#error Please implement a run-time test for NEON/ASIMD for your platform. ++ + #endif + + #endif diff -Nru fftw3-3.3.4/debian/patches/series fftw3-3.3.4/debian/patches/series --- fftw3-3.3.4/debian/patches/series 1970-01-01 01:00:00.000000000 +0100 +++ fftw3-3.3.4/debian/patches/series 2014-11-23 21:04:48.000000000 +0100 @@ -0,0 +1 @@ +fix-runtime-neon-detection.patchAttachment: signature.asc
Description: Digital signature
--- End Message ---
--- Begin Message ---
- To: Sébastien Villemot <sebastien@debian.org>, 770862-done@bugs.debian.org
- Subject: Re: Bug#770862: unblock: fftw3/3.3.4-2
- From: "Adam D. Barratt" <adam@adam-barratt.org.uk>
- Date: Mon, 24 Nov 2014 19:27:50 +0000
- Message-id: <1416857270.28376.20.camel@adam-barratt.org.uk>
- In-reply-to: <[🔎] 20141124192313.GA19454@villemot.name>
- References: <[🔎] 20141124192313.GA19454@villemot.name>
On Mon, 2014-11-24 at 20:23 +0100, Sébastien Villemot wrote: > fftw3 (3.3.4-2) unstable; urgency=medium > > * Team upload. > * fix-runtime-neon-detection.patch: new patch, fixes runtime NEON > detection on armhf (and also arm64, but NEON is currently > unconditionnally disabled there). Thanks to Edmund Grimley Evans for > the patch. (Closes: #767138) Unblocked. Regards, Adam
--- End Message ---