libmad ftbfs on armel in lucid

Bug #534287 reported by Alexander Sack
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libmad (Ubuntu)
High
Unassigned
Lucid
High
Unassigned

Bug Description

Alexander Sack (asac)
Changed in libmad (Ubuntu):
status: New → Triaged
importance: Undecided → Critical
Changed in libmad (Ubuntu Lucid):
milestone: none → ubuntu-10.04-beta-1
Alexander Sack (asac)
tags: added: armel
Revision history for this message
Dave Martin (dave-martin-arm) wrote :

Try this patch, which provides an alternative implementation of the 64-bit integer negation operation MAD_F_MLN which works around the lack of the RSC instruction in Thumb-2.

A missing earlyclobber constraing flag in the existing implementation is also fixed by the patch.

This fixes the build failure and I believe the code works... can anyone suggest how best to test it? (There seems to be no make check or make test).

Revision history for this message
Dave Martin (dave-martin-arm) wrote :

Oops, it looks like mad.h is a generated file... if so, the second part of the above patch should not be applied.

If mad.h is distrubuted in the source package, it should be rebuilt in the appropriate way from the other headers before uploading.

tags: added: kernel-series-unknown
tags: removed: kernel-series-unknown
tags: added: patch
Paul Larson (pwlars)
tags: added: patch-needswork
removed: patch
Revision history for this message
Alexander Sack (asac) wrote :

Dave, why this hunk below?

-# define FPM_INTEL
+# define FPM_ARM

this define isn't referenced anywhere else in the patch. Is this fixing a second issue?

Steve Langasek (vorlon)
Changed in libmad (Ubuntu Lucid):
importance: Critical → High
Revision history for this message
Dave Martin (dave-martin-arm) wrote :

mad.h is a generated file and has # define FPM_<arch> depending on what the package was built for before the source was distributed... so it changed from INTEL to ARM after I did a build.

Since mad.h is generated from fixed.h during build, please throw away the second half of the patch (which modifies mad.h).

Steve Langasek (vorlon)
Changed in libmad (Ubuntu Lucid):
milestone: ubuntu-10.04-beta-1 → ubuntu-10.04-beta-2
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libmad - 0.15.1b-4ubuntu1

---------------
libmad (0.15.1b-4ubuntu1) lucid; urgency=low

  * libmad.thumb.diff: use "adr" instead of "add" to make code ready for
    thumb2 (LP: #513734)
  * Provide-Thumb-2-alternative-code-for-MAD_F_MLN.diff: fix another
    ftbfs with thumb2 as "rsc" doesnt exist anymore - thanks to Dave
    Martin for this patch (LP: #534287).
 -- Alexander Sack <email address hidden> Fri, 19 Mar 2010 13:09:26 +0100

Changed in libmad (Ubuntu Lucid):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers