Aleksey Kliger (λgeek)

Results 129 comments of Aleksey Kliger (λgeek)

Breaking when the AOT compiler is compiling these two tests: ``` aot-compile: compiling /__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Interop/Interop/AnsiBStrTest.dll; MONO_PATH: /__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Interop/Interop:/__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Tests/Core_Root Mono Ahead of Time compiler - compiling assembly /__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Interop/Interop/AnsiBStrTest.dll ``` and ``` aot-compile: compiling...

We just started running the LLVM aot runtime tests again after the recent slate of PRs to make the LLVM compilation use less memory, so this was probably missed because...

I guess it's in this PR: https://github.com/dotnet/runtime/pull/92057

Actually I take it back. it's failing in `runtime-extra-platforms` on other `main`, too: https://dev.azure.com/dnceng-public/public/_build/results?buildId=562296&view=results https://dev.azure.com/dnceng-public/public/_build/results?buildId=562296&view=logs&j=d25dc2c0-518b-5583-f661-5ac905106679&t=47cb0142-4d6b-5edf-a0a8-cf81fa3f7e05&l=1453

Stack trace: ``` mono_os_mutex_destroy: pthread_mutex_destroy failed with "Resource busy" (16) ================================================================= Native Crash Reporting ================================================================= Got a SIGABRT while executing native code. This usually indicates a fatal error in the...

Might be related to https://github.com/dotnet/runtime/issues/85960#issuecomment-1844892277

Next steps are to do less cleanup in `ep_rt_mono_fini`. As Johan mentioned in https://github.com/dotnet/runtime/issues/85960#issuecomment-1844892277 > `ep_rt_mono_fini` assumes that all threads that might run EventPipe code has been stopped, so if...

@davmason you're probably not missing anything. I don't think we have a way to know when all the existing managed threads are really stopped/gonge (we used to have`mono_thread_suspend_all_other_threads` in "classic"...

I fixed the merge conflict. So it should be building again. I think this is an interesting test of the profiler and of the inliner (and their interaction), so it...