From 242481b812c5e8a18adedc0e4b523cbe2ff2e0e2 Mon Sep 17 00:00:00 2001 From: Jonathan Peppers Date: Wed, 3 Aug 2022 13:56:45 -0500 Subject: [PATCH] [Mono.Android] fix crash on startup with EnableLLVM (#7188) `dotnet new android` apps would crash on startup when built with `-c Release -p:EnableLLVM=true`: 07-20 08:55:44.642 2983 2983 F monodroid-assembly: Internal p/invoke symbol 'java-interop @ java_interop_jvm_list' (hash: 0x58c48fc8b89cb484) not found in compile-time map. ... 07-20 08:55:44.834 3004 3004 F DEBUG : signal 6 (SIGABRT), code -1 (SI_QUEUE), fault addr -------- 07-20 08:55:44.834 3004 3004 F DEBUG : Abort message: 'Internal p/invoke symbol 'java-interop @ java_interop_jvm_list' (hash: 0x58c48fc8b89cb484) not found in compile-time map.' `java_interop_jvm_list` should *never* be called on Android, it is the result of `new AndroidRuntime()` having not been called yet? The problem being that the static `Android.App.Application.cctor` was somehow running *before* `JNIEnv.Initialize()` was complete? And this only happens with LLVM? Reviewing the code: [UnmanagedCallersOnly] internal static unsafe void Initialize (JnienvInitializeArgs* args) { //... SynchronizationContext.SetSynchronizationContext (Android.App.Application.SynchronizationContext); } We do indeed access `Android.App.Application`... To fix this, we can move this to a new method decorated with `MethodImplOptions.NoInlining`. Apps built with LLVM now launch for me. However, we can't really enable the LLVM `Mono.Android-Tests` quite yet. Read on! ~~ Known Issues ~~ * https://github.com/dotnet/runtime/issues/73003 Using Profiled AOT and LLVM at the same time can hit: [ERROR] FATAL UNHANDLED EXCEPTION: System.PlatformNotSupportedException: Arg_PlatformNotSupported at System.Text.ASCIIUtility.GetIndexOfFirstNonAsciiByte_Intrinsified(Byte* , UIntPtr ) at System.Text.Unicode.Utf8Utility.GetPointerToFirstInvalidByte(Byte* , Int32 , Int32& , Int32& ) at System.Text.UTF8Encoding.GetCharCount(Byte* , Int32 ) at System.String.CreateStringFromEncoding(Byte* , Int32 , Encoding ) at System.Text.Encoding.GetString(Byte* , Int32 ) at System.String.CreateStringForSByteConstructor(Byte* , Int32 ) at System.String.Ctor(SByte* value) at Java.Interop.TypeManager.GetClassName(IntPtr ) at Android.Runtime.JNIEnv.RegisterJniNatives(IntPtr , Int32 , IntPtr , IntPtr , Int32 ) For now we can set `AndroidEnableProfiledAot=false` for the LLVM tests. * https://github.com/dotnet/runtime/issues/73304 Some Java.Interop unit tests fail with: System.InvalidOperationException : InvalidOperation_EnumEnded at System.ArrayEnumerator.get_Current() at Cadenza.Collections.Tests.CollectionContract`1[[System.Int16, System.Private.CoreLib, Version=7.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].DisposeCollection(IEnumerable c) at Cadenza.Collections.Tests.CollectionContract`1[[System.Int16, System.Private.CoreLib, Version=7.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].CopyTo() at System.Reflection.MethodInvoker.InterpretedInvoke(Object , Span`1 , BindingFlags ) * https://github.com/xamarin/java.interop/blob/a3de91efdaec04b3927b755a07018e8d26cfdc71/tests/Java.Interop-Tests/Cadenza.Collections/CollectionContract.cs#L205-L221 * https://github.com/xamarin/java.interop/blob/a3de91efdaec04b3927b755a07018e8d26cfdc71/tests/Java.Interop-Tests/Cadenza.Collections/CollectionContract.cs#L85-L86 The `InetAccess` category causes the `Mono.Android-Tests` suite to crash with: 08-02 21:26:57.581 5154 5173 I NUnit : System.NetTests.ProxyTest : 646.269 ms 08-02 21:26:57.593 5154 5173 I droid.NET_Test: Explicit concurrent copying GC freed 20(95KB) AllocSpace objects, 0(0B) LOS objects, 59% free, 1054KB/2590KB, paused 1.469ms total 7.180ms 08-02 21:26:57.594 5154 5173 I NUnit : SslTest 08-02 21:26:57.594 5154 5173 I NUnit : HttpsShouldWork 08-02 21:26:57.858 5154 5181 D NetworkSecurityConfig: No Network Security Config specified, using platform default 08-02 21:26:58.585 5154 5164 I droid.NET_Test: Background young concurrent copying GC freed 39658(2920KB) AllocSpace objects, 0(0B) LOS objects, 22% free, 1999KB/2590KB, paused 69.607ms total 194.902ms 08-02 21:26:58.922 5154 5164 I droid.NET_Test: Background concurrent copying GC freed 12412(836KB) AllocSpace objects, 0(0B) LOS objects, 46% free, 6987KB/12MB, paused 2.127ms total 327.448ms 08-02 21:26:58.953 5154 5181 I monodroid-timing: [1/0] LZ4 decompression time for ; elapsed: 0:0::75000 08-02 21:26:58.954 5154 5181 W linker : Warning: "/data/app/Mono.Android.NET_Tests-QcXozuJGB3EpfK6bvDvvMQ==/split_config.x86_64.apk!/lib/x86_64/libaot-System.Formats.Asn1.dll.so" has no DT_SONAME (will use libaot-System.Formats.Asn1.dll.so instead) and will not work when the app moves to API level 23 or later (https://android.googlesource.com/platform/bionic/+/master/missing-soname-enforced-for-api-level-23) (allowing for now because this app's target API level is still 21) 08-02 21:26:59.037 5154 5181 I monodroid-timing: [1/0] LZ4 decompression time for ; elapsed: 0:0::129000 08-02 21:26:59.038 5154 5181 W linker : Warning: "/data/app/Mono.Android.NET_Tests-QcXozuJGB3EpfK6bvDvvMQ==/split_config.x86_64.apk!/lib/x86_64/libaot-System.Runtime.Numerics.dll.so" has no DT_SONAME (will use libaot-System.Runtime.Numerics.dll.so instead) and will not work when the app moves to API level 23 or later (https://android.googlesource.com/platform/bionic/+/master/missing-soname-enforced-for-api-level-23) (allowing for now because this app's target API level is still 21) 08-02 21:26:56.680 5154 5154 W Instrumentation: type=1400 audit(0.0:432): avc: denied { search } for name="run" dev="vdc" ino=32774 scontext=u:r:untrusted_app_25:s0:c512,c768 tcontext=u:object_r:varrun_file:s0 tclass=dir permissive=0 app=Mono.Android.NET_Tests 08-02 21:26:56.770 5154 5154 W Instrumentation: type=1400 audit(0.0:433): avc: denied { ioctl } for path="socket:[49468]" dev="sockfs" ino=49468 ioctlcmd=0x8946 scontext=u:r:untrusted_app_25:s0:c512,c768 tcontext=u:r:untrusted_app_25:s0:c512,c768 tclass=udp_socket permissive=0 app=Mono.Android.NET_Tests 08-02 21:26:56.780 5154 5154 W Instrumentation: type=1400 audit(0.0:434): avc: denied { ioctl } for path="socket:[49468]" dev="sockfs" ino=49468 ioctlcmd=0x8946 scontext=u:r:untrusted_app_25:s0:c512,c768 tcontext=u:r:untrusted_app_25:s0:c512,c768 tclass=udp_socket permissive=0 app=Mono.Android.NET_Tests 08-02 21:27:00.029 1540 1540 I Zygote : Process 5154 exited due to signal 11 (Segmentation fault) Not really any useful info here... Disabled this category for now. We can file an issue for this one, assuming it is different than the other issues. --- build-tools/automation/azure-pipelines.yaml | 8 +++++--- src/Mono.Android/Android.Runtime/JNIEnv.cs | 9 ++++++++- .../Mono.Android.NET-Tests.csproj | 4 ++-- 3 files changed, 15 insertions(+), 6 deletions(-) diff --git a/build-tools/automation/azure-pipelines.yaml b/build-tools/automation/azure-pipelines.yaml index 979f6e1b52e..9c772956a3b 100644 --- a/build-tools/automation/azure-pipelines.yaml +++ b/build-tools/automation/azure-pipelines.yaml @@ -379,7 +379,7 @@ stages: testName: Mono.Android_Tests-Aot project: tests/Mono.Android-Tests/Mono.Android-Tests.csproj testResultsFiles: TestResult-Mono.Android_Tests-$(ApkTestConfiguration)-Aot.xml - extraBuildArgs: /p:AotAssemblies=True /p:EnableLlvm=True + extraBuildArgs: /p:AotAssemblies=True /p:EnableLLVM=True artifactSource: bin/Test$(ApkTestConfiguration)/Mono.Android_Tests-Signed.apk artifactFolder: AotLlvm @@ -691,13 +691,15 @@ stages: - template: yaml-templates/apk-instrumentation.yaml parameters: - # TODO: disable LLVM test, see: https://github.com/dotnet/runtime/issues/68914 + # TODO: disable LLVM test, see: + # https://github.com/dotnet/runtime/issues/68914 + # https://github.com/dotnet/runtime/issues/73304 condition: false configuration: $(XA.Build.Configuration) testName: Mono.Android.NET_Tests-AotLlvm project: tests/Mono.Android-Tests/Runtime-Microsoft.Android.Sdk/Mono.Android.NET-Tests.csproj testResultsFiles: TestResult-Mono.Android.NET_Tests-$(XA.Build.Configuration)AotLlvm.xml - extraBuildArgs: -p:TestsFlavor=AotLlvm -p:EnableLlvm=true + extraBuildArgs: -p:TestsFlavor=AotLlvm -p:EnableLLVM=true -p:AndroidEnableProfiledAot=false artifactSource: bin/Test$(XA.Build.Configuration)/$(DotNetTargetFramework)-android/Mono.Android.NET_Tests-Signed.aab artifactFolder: $(DotNetTargetFramework)-AotLlvm useDotNet: true diff --git a/src/Mono.Android/Android.Runtime/JNIEnv.cs b/src/Mono.Android/Android.Runtime/JNIEnv.cs index 36c2e476340..3d1a5cde5e1 100644 --- a/src/Mono.Android/Android.Runtime/JNIEnv.cs +++ b/src/Mono.Android/Android.Runtime/JNIEnv.cs @@ -198,10 +198,17 @@ internal static unsafe void Initialize (JnienvInitializeArgs* args) } #if !MONOANDROID1_0 - SynchronizationContext.SetSynchronizationContext (Android.App.Application.SynchronizationContext); + SetSynchronizationContext (); #endif } +#if !MONOANDROID1_0 + // NOTE: prevents Android.App.Application static ctor from running + [MethodImpl (MethodImplOptions.NoInlining)] + static void SetSynchronizationContext () => + SynchronizationContext.SetSynchronizationContext (Android.App.Application.SynchronizationContext); +#endif + internal static void Exit () { /* Manually dispose surfaced objects and close the current JniEnvironment to diff --git a/tests/Mono.Android-Tests/Runtime-Microsoft.Android.Sdk/Mono.Android.NET-Tests.csproj b/tests/Mono.Android-Tests/Runtime-Microsoft.Android.Sdk/Mono.Android.NET-Tests.csproj index 22d9dce4925..2460510dca5 100644 --- a/tests/Mono.Android-Tests/Runtime-Microsoft.Android.Sdk/Mono.Android.NET-Tests.csproj +++ b/tests/Mono.Android-Tests/Runtime-Microsoft.Android.Sdk/Mono.Android.NET-Tests.csproj @@ -21,10 +21,10 @@ IL2037 DotNetIgnore - $(ExcludeCategories):InetAccess + $(ExcludeCategories):InetAccess $(ExcludeCategories):IgnoreInterpreter