Your message dated Sat, 14 May 2011 09:17:22 +0000 with message-id <e1qlayq-0007kx...@franck.debian.org> and subject line Bug#623162: fixed in ntl 5.4.2-4.1 has caused the Debian Bug report #623162, regarding ntl: FTBFS on mips/mipsel, use old assembly constraints 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 ow...@bugs.debian.org immediately.) -- 623162: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=623162 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
--- Begin Message ---Package: ntl Version: 5.4.2-4 Severity: serious Since GCC 4.4 it's not possible anymore to use the 'h' constraints for MIPS inline assembly code when doing a multiplication. That's why sprng fails to build from source on mips and mipsel. That said GCC supports 32x32 => 64 multiplication on 32-bit architecture for a lot of time, so there is no need to use assembly code for that. The patch below fixes the problem by using standard multiplication instead of assembly code. I have also included the code for MIPS64 using 128-bit hints for reference (the second hunk), though it is not used in Debian. --- ntl-5.4.2.orig/include/NTL/SPMM_ASM.h +++ ntl-5.4.2/include/NTL/SPMM_ASM.h @@ -147,8 +147,8 @@ static inline unsigned long MulHiUL(unsigned long a, unsigned long b) { - unsigned long hi, lo; - __asm__ ("multu %2,%3" : "=l" (lo), "=h" (hi) : "d" (a), "d" (b)); + unsigned long hi; + hi = ((unsigned long long) a * b) >> 32; return hi; } @@ -159,8 +159,9 @@ static inline unsigned long MulHiUL(unsigned long a, unsigned long b) { - unsigned long hi, lo; - __asm__ ("dmultu %2,%3" : "=l" (lo), "=h" (hi) : "d" (a), "d" (b)); + typedef unsigned int uint128_t __attribute__((mode(TI))); + unsigned long hi; + hi = ((uint128_t) a * b) >> 64; return hi; } -- System Information: Debian Release: wheezy/sid APT prefers unstable APT policy: (500, 'unstable'), (1, 'experimental') Architecture: mips (mips64) Kernel: Linux 2.6.32-5-5kc-malta Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/bash
--- End Message ---
--- Begin Message ---Source: ntl Source-Version: 5.4.2-4.1 We believe that the bug you reported is fixed in the latest version of ntl, which is due to be installed in the Debian FTP archive: libntl-5.4.2_5.4.2-4.1_mipsel.deb to main/n/ntl/libntl-5.4.2_5.4.2-4.1_mipsel.deb libntl-dev_5.4.2-4.1_mipsel.deb to main/n/ntl/libntl-dev_5.4.2-4.1_mipsel.deb ntl_5.4.2-4.1.diff.gz to main/n/ntl/ntl_5.4.2-4.1.diff.gz ntl_5.4.2-4.1.dsc to main/n/ntl/ntl_5.4.2-4.1.dsc A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to 623...@bugs.debian.org, and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Aurelien Jarno <aure...@debian.org> (supplier of updated ntl package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing ftpmas...@debian.org) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Format: 1.8 Date: Mon, 09 May 2011 20:41:04 +0200 Source: ntl Binary: libntl-5.4.2 libntl-dev Architecture: source mipsel Version: 5.4.2-4.1 Distribution: unstable Urgency: low Maintainer: Aurelien Jarno <aure...@debian.org> Changed-By: Aurelien Jarno <aure...@debian.org> Description: libntl-5.4.2 - Number Theory Library, shared library libntl-dev - Number Theory Library, development files Closes: 623162 Changes: ntl (5.4.2-4.1) unstable; urgency=low . * Non-maintainer upload. * Fix gcc-4.5 FTBFS on mips, due to old assembly constraints usage (Closes: #623162). Checksums-Sha1: e40500d77a29ae603318ec9fb9fee94ef4c81a7d 1081 ntl_5.4.2-4.1.dsc 5dd7d78dfdb6fcd2385009e02025855aaa48e46a 4406 ntl_5.4.2-4.1.diff.gz 25bbe430dd1950c738f8fe35160d2a413b62a77e 707940 libntl-5.4.2_5.4.2-4.1_mipsel.deb ff4447b787955742fa9e55c351224858ef95296a 1326266 libntl-dev_5.4.2-4.1_mipsel.deb Checksums-Sha256: caa0f436ad8bb5d891fa40849ccb5a49c91b63bfe325d7b97b412beffa42329e 1081 ntl_5.4.2-4.1.dsc 425db1235cbbf16a3f5500a5870b804c3cf0e81ef5d08761cfe5b5a1775a19b5 4406 ntl_5.4.2-4.1.diff.gz 49824996aea3b6088d369b27d3b58c2e6d1a336c2dc009ebf3a67c5bed401691 707940 libntl-5.4.2_5.4.2-4.1_mipsel.deb 692a8d53ff30257694f2214bc17f1d1c9785f3cd471ceb167907fed9589e7fa2 1326266 libntl-dev_5.4.2-4.1_mipsel.deb Files: 8269e3bd6543de72893ab56a18e16d65 1081 math optional ntl_5.4.2-4.1.dsc 9f659e93710da552e94d21ccc4862272 4406 math optional ntl_5.4.2-4.1.diff.gz 4e7cc9d373b92947c0b3d0227fed769e 707940 libs optional libntl-5.4.2_5.4.2-4.1_mipsel.deb 95b03fc64f5d94c3811ed6b539ed453f 1326266 libdevel extra libntl-dev_5.4.2-4.1_mipsel.deb -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iD8DBQFNyGDmw3ao2vG823MRAnEgAJ9b/s9kf0qlazW+SXkMRhDOlVYkPQCfepR2 ogjweO/p83rPRO7F/dWAKP8= =ZNNB -----END PGP SIGNATURE-----
--- End Message ---
