Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

armeabi libatomic.a uses ifuncs #31

Closed
DoDoENT opened this issue Mar 21, 2016 · 11 comments
Closed

armeabi libatomic.a uses ifuncs #31

DoDoENT opened this issue Mar 21, 2016 · 11 comments
Assignees
Milestone

Comments

@DoDoENT
Copy link

DoDoENT commented Mar 21, 2016

When compiling our app with libc++ runtime and clang, as soon as first native method is called, we get a SIGSEGV with invalid stack trace (stack trace points to file and line number which can be reached only from completely different JNI method).

When using libc++ + GCC, app crashes when attempting to throw c++ exception.

GNU STL + clang and GNU STL + GCC combinations work correctly. We've tested on Samsung Galaxy Ace: armeabi ABI + Android 2.3 (API level 10).

@DanAlbert
Copy link
Member

Most likely this is a dup of http://b.android.com/181517. I assume this was libc++_shared?

@DoDoENT
Copy link
Author

DoDoENT commented Mar 21, 2016

@DanAlbert I cannot open your link. It was libc++_static.

Edit: http version of link works for me.

@DanAlbert
Copy link
Member

Oops. Link fixed, but I guess not then...

@DanAlbert DanAlbert self-assigned this Mar 21, 2016
@DoDoENT
Copy link
Author

DoDoENT commented Mar 21, 2016

Well, the libc++ + clang version crashed with SIGSEGV immediately when calling native method and stack trace was completely wrong.

Hopefully I will have time to investigate this further next week and probably prepare some sample.

@DanAlbert
Copy link
Member

E/AndroidRuntime( 5460): FATAL EXCEPTION: main
E/AndroidRuntime( 5460): java.lang.ExceptionInInitializerError
E/AndroidRuntime( 5460): at java.lang.Class.newInstanceImpl(Native Method)
E/AndroidRuntime( 5460): at java.lang.Class.newInstance(Class.java:1409)
E/AndroidRuntime( 5460): at android.app.Instrumentation.newActivity(Instrumentation.java:1021)
E/AndroidRuntime( 5460): at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1561)
E/AndroidRuntime( 5460): at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:1663)
E/AndroidRuntime( 5460): at android.app.ActivityThread.access$1500(ActivityThread.java:117)
E/AndroidRuntime( 5460): at android.app.ActivityThread$H.handleMessage(ActivityThread.java:931)
E/AndroidRuntime( 5460): at android.os.Handler.dispatchMessage(Handler.java:99)
E/AndroidRuntime( 5460): at android.os.Looper.loop(Looper.java:130)
E/AndroidRuntime( 5460): at android.app.ActivityThread.main(ActivityThread.java:3683)
E/AndroidRuntime( 5460): at java.lang.reflect.Method.invokeNative(Native Method)
E/AndroidRuntime( 5460): at java.lang.reflect.Method.invoke(Method.java:507)
E/AndroidRuntime( 5460): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:839)
E/AndroidRuntime( 5460): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:597)
E/AndroidRuntime( 5460): at dalvik.system.NativeStart.main(Native Method)
E/AndroidRuntime( 5460): Caused by: java.lang.UnsatisfiedLinkError: Cannot load library: reloc_library[1479]: 67 unknown reloc type 160 @ ( 379)
E/AndroidRuntime( 5460): at java.lang.Runtime.loadLibrary(Runtime.java:434)
E/AndroidRuntime( 5460): at java.lang.System.loadLibrary(System.java:554)
E/AndroidRuntime( 5460): at com.example.hellojni.HelloJni.(HelloJni.java:64)
E/AndroidRuntime( 5460): ... 15 more
W/ActivityManager( 106): Force finishing activity com.example.hellojni/.HelloJni

Relocation 160 is an ifunc. After some readelf and some objdump, tracked that down to libatomic. Android doesn't support ifuncs until M, so this is going to require a fix to GCC...

@DanAlbert
Copy link
Member

A little more testing shows that this is specific to GCC, which is why the libc++ test suite didn't catch it (we don't run the tests with GCC yet because our version of libc++ is far too old for the test suite to be very GCC compatible).

@cjhopman
Copy link

cjhopman commented Apr 4, 2016

Maybe the title should be updated, this is much worse than the title implies:

The failure isn't limited to libc++ runtime, and it affects android up until 5.0 (not just the rather old 2.3).

Would it be possible to recompile libatomic with HAVE_IFUNC=false? After a cursory look it appears to support that.

@DanAlbert DanAlbert changed the title LibC++ runtime does not work on Android 2.3 armeabi libatomic.a uses ifuncs Apr 4, 2016
@DanAlbert
Copy link
Member

Yeah, the fix is nearly done. The changes to our GCC were https://android-review.googlesource.com/#/c/210660/ and https://android-review.googlesource.com/#/c/212250/. I just need to actually pull the new prebuilts into the NDK.

Updated the title to reflect the issue more clearly.

@DanAlbert
Copy link
Member

Just as an FYI, not clear if this will be in the r12 beta that I'll be kicking out in a couple weeks, but it will definitely be in the r12 release in June.

@DoDoENT
Copy link
Author

DoDoENT commented Apr 5, 2016

@DanAlbert, we didn't experience that crash on Android 4.0, 4.1 and 4.4 - only on 2.3. Are you sure this affects all android versions up until 5.0?

EDIT: sorry, I didn't notice that this is solely an armeabi issue (i.e. not present in armeabi-v7a and x86, which we used for testing on 4.0, 4.1 and 4.4).

@DanAlbert
Copy link
Member

Fix is in for r12.

DanAlbert added a commit that referenced this issue Apr 16, 2016
The version of libatomic we have from GCC uses ifuncs for some
functions, which are not supported on anything but the latest
versions of Android. GCC ends up using these functions when using
libc++, so GCC+libc++ libraries will not be able to load on most
devices.

Bug: http://b/22699878
Bug: #31
Change-Id: I90ddc472685d016db76690773ab47da44ddc1037
DanAlbert added a commit that referenced this issue Apr 16, 2016
Bug: http://b/26085687
Bug: http://b/22699878
Bug: #31
Change-Id: Iac37f5b8657c12c082838caedacb068b5b2dd7cf
tmpsantos added a commit to mapbox/mapbox-gl-native that referenced this issue Jul 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants