ARM unwind table linker processing broke OO's uno2cpp

Bug #436617 reported by Matthias Klose
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
binutils
Invalid
Medium
binutils (Ubuntu)
Invalid
High
Unassigned
Karmic
Won't Fix
High
Unassigned

Bug Description

Binary package hint: binutils

See the upstream report

Tags: armel
Revision history for this message
In , Matthias Klose (doko) wrote :

libgcc3_uno.so is built as part of OpenOffice.org; when built with 2.19.91 an
exception is thrown; when built with 2.19.1, the application starts fine.
Tracked down to the patches for the ARM unwind table linker processing.

 - attaching revert.diff, when applied to the 2.20 branch, builds a working
   library.
 - http://people.canonical.com/~doko/tmp/test.tar.gz has the testcase,
   together with two prebuilt libraries.

I don't know if this is a bug in binutils, or in the uno2cpp bridge for ARM.

Revision history for this message
In , Matthias Klose (doko) wrote :

Created attachment 4230
reversion of the unwinder changes

Matthias Klose (doko)
Changed in binutils (Ubuntu):
importance: Undecided → High
status: New → Triaged
Changed in binutils:
status: Unknown → Confirmed
Loïc Minier (lool)
tags: added: armel
Revision history for this message
In , Matthias Klose (doko) wrote :

filed as well http://www.openoffice.org/issues/show_bug.cgi?id=105359

Comment from Paul Brook on irc:

  I guess you get to debug it then. Figure out what's wrong with
  the unwind tables, and where that bogosity comes from. My WAG
  would be that it works by chance before because it's picking up
  an unwind table from some other funciton. After the patch we're
  inserting cantunwind markers for code that can't be unwound,
  so the latent failure is exposed.

Revision history for this message
Loïc Minier (lool) wrote :

Shall we close this bug?

Revision history for this message
In , Matthias Klose (doko) wrote :

this is fixed in 2.20 and trunk by Jakub's cfi_sections patch, and the fix for
GCC PR 40521

Revision history for this message
Matthias Klose (doko) wrote :

fix with current binutils in karmic

Changed in binutils (Ubuntu Karmic):
status: Triaged → Fix Released
Changed in binutils:
status: Confirmed → Fix Released
Revision history for this message
Matthias Klose (doko) wrote :

not fixed, made a mistake testing ...

Changed in binutils (Ubuntu Karmic):
status: Fix Released → Confirmed
Revision history for this message
Loïc Minier (lool) wrote :

oo.o is still borken for another unknown reason, but cfi_sections are now passed between gcc and as, so it's as good as it can be on that front for binutils (see upstream bug).

Changed in binutils (Ubuntu Karmic):
status: Confirmed → Fix Released
Revision history for this message
In , Matthias Klose (doko) wrote :

I made a mistake with testing, 40521 didn't fix this.

Revision history for this message
In , Matthias Klose (doko) wrote :

I see now the same exception with the unwinder changes reverted. So start again ...

Revision history for this message
Matthias Klose (doko) wrote :

reopening, although the assumption about the ARM unwind table linker processing seems to be wrong

Changed in binutils (Ubuntu Karmic):
status: Fix Released → Confirmed
Changed in binutils:
status: Fix Released → Confirmed
Revision history for this message
Loïc Minier (lool) wrote :

Dropped karmic task; happy to have it as a SRU but not release critical

Changed in binutils (Ubuntu Karmic):
status: Confirmed → Won't Fix
Revision history for this message
In , Matthias Klose (doko) wrote :

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40860 shows regressions in the
libjava tests, which were tracked down to this change.

Revision history for this message
In , Matthew Gretton-Dann (matthew-gretton-dann) wrote :

Following discussion and investigation this has been shown to be a problem in
OpenOffice's codebase and not a problem with ld.

See the following pages for mote information:
http://www.openoffice.org/issues/show_bug.cgi?id=105359 and
https://bugs.launchpad.net/ubuntu/+source/openoffice.org/+bug/417009

Changed in binutils:
status: Confirmed → Invalid
Revision history for this message
Matthias Klose (doko) wrote :

OOo bug, fixed for lucid

Changed in binutils (Ubuntu):
status: Confirmed → Invalid
Changed in binutils:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.