2009-11-25 18:13:12 |
Dave Martin |
bug |
|
|
added bug |
2009-11-25 18:13:29 |
Dave Martin |
summary |
NS_InvokeByIndex in xptcinvoke_arm.cpp is not Thumb-2 safe |
NS_InvokeByIndex in xptcinvoke_arm.cpp is not Thumb-2 safe for Lucid |
|
2009-11-27 16:57:16 |
Dave Martin |
description |
Binary package hint: xulrunner-1.9.1
Either this file should be built individually as ARM code using -marm, or the code needs to be reviewed and modified to be Thumb-2 safe.
Specifically, calling a function using:
mov lr, pc
mov pc, ip
...will not interwork properly between ARM and Thumb state, depending on the function called.
Instead, something like the following is needed:
#ifdef __thumb__
adr lr, 0f + 1 /* store return address in lr, setting the Thumb bit [0] */
bx ip
0:
#else
mov lr, pc
mov pc, ip
#endif
I'll try and propose a suitable patch.
I'm currently scanning to see whether there is other problematic code in this package. |
Binary package hint: xulrunner-1.9.1
Either this file should be built individually as ARM code using -marm, or the code needs to be reviewed and modified to be Thumb-2 safe.
Specifically, calling a function using:
mov lr, pc
mov pc, ip
...will not interwork properly between ARM and Thumb state, depending on the function called.
Instead, something like the following is needed:
#ifdef __thumb__
blx ip
0:
#else
mov lr, pc
mov pc, ip
#endif
I'll try and propose a suitable patch.
I'm currently scanning to see whether there is other problematic code in this package.
|
|
2009-11-27 18:23:46 |
Dave Martin |
attachment added |
|
gdb transcript demonstrating the workings of the Thumb-2 patch http://launchpadlibrarian.net/36156819/typescript.txt |
|
2009-11-30 15:30:29 |
Dave Martin |
xulrunner-1.9.1 (Ubuntu): assignee |
|
Alexander Sack (asac) |
|
2009-12-01 14:28:06 |
Dave Martin |
description |
Binary package hint: xulrunner-1.9.1
Either this file should be built individually as ARM code using -marm, or the code needs to be reviewed and modified to be Thumb-2 safe.
Specifically, calling a function using:
mov lr, pc
mov pc, ip
...will not interwork properly between ARM and Thumb state, depending on the function called.
Instead, something like the following is needed:
#ifdef __thumb__
blx ip
0:
#else
mov lr, pc
mov pc, ip
#endif
I'll try and propose a suitable patch.
I'm currently scanning to see whether there is other problematic code in this package.
|
Binary package hint: xulrunner-1.9.1
Two issues:
1) Either this file should be built individually as ARM code using -marm, or the code needs to be reviewed and modified to be Thumb-2 safe.
Specifically, calling a function using:
mov lr, pc
mov pc, ip
...will not interwork properly between ARM and Thumb state, depending on the function called.
Instead, something like the following is needed:
#ifdef __thumb__
blx ip
0:
#else
mov lr, pc
mov pc, ip
#endif
I'll try and propose a suitable patch.
I'm currently scanning to see whether there is other problematic code in this package.
Patch posted below.
2) The kernel user helpers are not currently used for implementing atomic operations. (This is the same backend used by the GCC __sync_*() atomic intrinsics.)
There is a configure option --with-arm-kuser to enable this, but it's not currently bassed by the build rules. It's been supported since ff 3.5 / linux 2.6.17 (if I remember correctly) |
|
2009-12-01 22:39:44 |
Reed Loden |
bug watch added |
|
https://bugzilla.mozilla.org/show_bug.cgi?id=532198 |
|
2009-12-01 22:39:44 |
Reed Loden |
bug task added |
|
firefox |
|
2009-12-01 22:41:01 |
Bug Watch Updater |
firefox: status |
Unknown |
Confirmed |
|
2009-12-01 22:41:01 |
Bug Watch Updater |
bug watch added |
|
http://linux.onarm.com/bugzilla/show_bug.cgi?id=36 |
|
2009-12-04 14:18:18 |
Alexander Sack |
xulrunner-1.9.1 (Ubuntu): status |
New |
Triaged |
|
2009-12-04 14:18:23 |
Alexander Sack |
xulrunner-1.9.1 (Ubuntu): importance |
Undecided |
High |
|
2009-12-04 14:18:33 |
Alexander Sack |
nominated for series |
|
Ubuntu Lucid |
|
2009-12-04 14:18:33 |
Alexander Sack |
bug task added |
|
xulrunner-1.9.1 (Ubuntu Lucid) |
|
2009-12-04 14:18:46 |
Alexander Sack |
xulrunner-1.9.1 (Ubuntu Lucid): milestone |
|
lucid-alpha-2 |
|
2010-01-04 23:58:10 |
Launchpad Janitor |
branch linked |
|
lp:~mozillateam/xulrunner/xulrunner-1.9.1.head |
|
2010-01-07 23:52:57 |
Launchpad Janitor |
xulrunner-1.9.1 (Ubuntu Lucid): status |
Triaged |
Fix Released |
|
2010-01-08 00:19:18 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/xulrunner-1.9.1 |
|
2010-01-12 16:06:13 |
Launchpad Janitor |
branch linked |
|
lp:~mozillateam/xulrunner/xulrunner-1.9.2.head |
|
2010-04-20 12:26:47 |
Bug Watch Updater |
firefox: status |
Confirmed |
Fix Released |
|
2010-04-21 04:38:36 |
Micah Gersten |
firefox: milestone |
|
3.7 |
|
2010-09-17 23:17:26 |
Bug Watch Updater |
firefox: importance |
Unknown |
Medium |
|