[threads] Store MonoInternalThread in MonoThreadInfo for use when detaching (#5058)

* [threads] Make mono_thread_attach_full static

* [threads] Remove  prefix on MonoThreadInfoCallbacks callbacks

* [threads] Rename mono_threads*_init functions to mono_thread_info*_init

* [threads] Remove dead method mono_runtime_is_critical_method

* [threads] Move MonoThreadInfoCallbacks to metadata/threads

* [threads] Store MonoInternalThread in MonoThreadInfo for use when detaching

When the native thread would exit, the TLS keys are destroyed. There is a TLS destructor for MonoThreadInfo, but not for MonoInternalThread. If MonoInternalThread is destroyed before MonoThreadInfo, then MonoThreadInfoCallbacks.thread_detach wouldn't successfully detach the current MonoInternalThread as mono_thread_internal_current would return NULL. This would lead to MonoInternalThread still laying around, while their corresponding MonoThreadInfo has already been destroyed. This fixes this problem, since we now rely on MonoThreadInfo to store the MonoInternalThread for detach.

Merge pull request #5070 from alexrp/profiler-version-bump

[profiler] Bump log profiler version to 2.0 due to breaking argument changes.

[profiler] Bump log profiler version to 2.0 due to breaking argument changes.

Merge pull request #5062 from alexrp/profiler-library

[profiler] Introduce the Mono.Profiler.Log library. (take 2)

[profiler] Introduce the Mono.Profiler.Log library. API still unstable.

Merge pull request #5063 from alexrp/profiler-format-changes

[profiler] A few small format changes

[profiler] Bump format version to 14.

[profiler] Update format description comment.

[profiler] Encode the address in TYPE_SAMPLE_UBIN as a pointer.

[profiler] Add an exception object field to TYPE_CLAUSE.

[profiler] Add an image pointer field to assembly load events.

[profiler] Simplify the encoding of monitor events.

The way the event was encoded was highly irregular with respect to the rest of
the format.

[jit] Fix a warning.

Merge pull request #5059 from lambdageek/tbt

[tests] Rework TypeBuilderTest SetUp/TearDown

[arm64] Remove an assert which is hit when mono_arch_get_call_target () is called with a 'b' instead of a 'bl'.

Merge pull request #5027 from kumpera/damn-you-strerror

[eglib] Replace strerror with strerror_r as the former is not thread safe.

[jit] Treat the mono_threads_attach_coop () icall specially, since it can be called from native-to-managed wrappers on unattached threads, so calling it shouldn't go through the trampoline infrastructure. (#5056)

[tests] Don't run SRE saved assembly in current process

Windows doesn't like deleting the file, and the original bug was a problem in
SRE, not in the AssemblyBuilder.Save ()

[interp] Implement runtime invokes by executing the runtime-invoke wrappers using the interpreter. This saves some code and allows better interop with JIT exceptions. (#5057)

[MacSDK] Bump msbuild

[packaging] Add a few more missing files to Windows MSI

Also add a safeguard so we don't accidentally leave files with
OSX paths in the package.

Bump cecil and api-snapshot

[btls] Fix a small memleak in btls-x509-lookup.c (#5065)

Noticed this while looking at the Clang Analysis Report[1] as the
only memory leak in the report and thought why not fix it :)

[1] https://jenkins.mono-project.com/job/test-mono-mainline-staticanalysis/Clang_Analysis_Report/

[packaging] Add a few missing msbuild files to Windows MSI

[profiler] Add an exception clause callback that receives the exception object.

[sdb] Fix invokes to string ctors. Fixes #57197.

[w32handle] Unify WaitHandle.Wait{One,Any,All} icalls (#5051)

[tests] Rework TypeBuilderTest SetUp/TearDown

- Create a unique temporary directory for each test in TypeBuilderTest and delete
  it on TearDown.

- Change tests to use tempDir instead of Path.GetTempPath ()

- Change IsSerializable () test to how it was 11 years ago and have it write to a
  file with a simple name (but now in a directory where noone else is writing)

- Change FieldsWithSameName (), FieldsWithSameNameAndType () and
  MethodsWithSameNameAndSig () to use the default temp assembly (now in a
  unique tempDir) instead of using the CreateTempAssembly ()

- Remove CreateTempAssembly function. It is dead code.

[threads] Rework MonoThreadInfoCallbacks.thread_{register,detach,unregister} callbacks (#5055)

* [threads] Use MonoThreadInfo.{stack_start_limit,stack_end} in more places

* [threads] Remove dead code

* [threads] Rename MonoThreadInfoCallbacks.thread_{register,detach,unregister} to MonoThreadInfoCallbacks.thread_{attach,detach,detach_with_lock} respectively

[runtime] Add a mono_marshal_get_runtime_invoke_full () function with an extra need_direct_wrapper argument to allow generation of runtime-invoke wrappers which directly call a method without using the 'addr' argument. (#5053)

[interp] Implement CEE_RETHROW properly by allocating a separate exception variable for each clause, and not overwriting stack traces on a rethrow. (#5040)

Merge pull request #5014 from vkargov/vk-kasha

[jit] Always cache the latest image set in case of a cache miss.

[msvc] Update csproj files (#5052)

[packaging] Add back IronLangs to Windows MSI

It was just temporarily removed from the Mac package to work around
a build break. This was fixed so we need to add iron* back.

Bump API snapshot

Merge pull request #5010 from Unity-Technologies/boehm-gc-alloc-fixed-sre

Boehm GC Fixes - SRE and Handle Stack

Merge pull request #5049 from mono/fixSdbTests

Fixed Mono.Debugger.Soft unit tests

Enable translation of WSEACCESS (10013) on mobile (#5050)

This error can occur on mobile devices during network operations
as evidenced in https://bugzilla.xamarin.com/show_bug.cgi?id=56653
and having an actual description of the error printed in this case
makes it easier for the user to discover what's going on.

Fixed Mono.Debugger.Soft unit tests and added new `new String('a', 3);` unit test after `return;`

Merge pull request #5042 from mono/sdbSyncDebuggerLibs

Upstreaming changes from mono/debugger-libs to Mono.Debugger.Soft

Bug 44266 - Unable to change variable values whilst debugging
https://github.com/mono/debugger-libs/commit/ac17c08938408abc7f5aaf0f862d83bbd2b56d42

caching thread state
https://github.com/mono/debugger-libs/commit/e3e665525a59823563c23babaf5d0e29a44a644a

Updated SourceLocation with end line and end column
https://github.com/mono/debugger-libs/commit/479a84ce506e7cf25d6aac4353d59b7fc6526a17

Fix one extra allocation on MethodMirror.FullName
https://github.com/mono/debugger-libs/commit/cfea89011ef690fe2cd9fefccf6b34ee1d4b1584

Bug 36053 - Broken enum values conversion
plus moved some code to correct location
https://github.com/mono/debugger-libs/commit/d65261617baddb5b41f665b0eb717cd1f81873b6

Fixed fetching Enum values on older runtimes
https://github.com/mono/debugger-libs/commit/525c3a685d52eb6ed053ee23c358e54dc091d9cb

[Debugger] Set the encoding of the error/output readers.
Bug 40556 - Output pad no longer displays unicode characters
https://github.com/mono/debugger-libs/commit/c10de4dd44493156a112fdc0d315db75e0457a2d

Merge pull request #5032 from kumpera/profiler-reorder-heap-events

Fix how HEAP_START / HEAP_END works

Add back missing TypeConverter that got removed in last commit

Also bump API snapshot with the latest changes.

Use CoreFX sources for System.Printing (#5041)

Use CoreFX sources for System.Drawing.Imaging (#5048)

* Use CoreFX sources for System.Drawing.Imaging

* Bump API snapshot

In mono_thread_info_suspend_lock in non-coop, don't assert MonoThreadInfo* is non-NULL

mono_thread_info_suspend_lock () can be called from boehm-gc.c
on_gc_notification before the new thread's start_wrapper calls
mono_thread_info_attach but after pthread_create calls the start wrapper.

As a result, the asserts in mono_thread_info_suspend_lock_with_info can catch
the new thread in an inconsistent state.

[msvc] Update csproj files

Use Drawing2D classes from CoreFX

Ensure monodoc/Test/ exists, with an empty file

This prevents build failures building monodoc tests, as git cannot check out empty directories, but Test/ must exist for `make check` to work

Initialize new handle chunks on in mono_handle_stack_alloc ()

Use Brush classes from CoreFX (#5039)

Revert "[win] set HAVE_ONLINE_VES" (#5038)

This reverts commit 501e77f91215d31c56aa63de0c99c91d8b13bfa5.

superseded by https://github.com/mono/mono/pull/5000

[arm64] Fix the patching of MONO_PATCH_INFO_METHOD_JUMP so it handles thunks properly. Allocate thunk space for tail calls. Fixes asserts in the System.Numerics xunit tests. (#5026)

[packaging] Add missing .pc files to Windows folder

It got accidentally ignored by the root .gitignore.

[profiler] Handle zero object heap dumps in mprof-report and fix the test suite.

[profiler] Emit post-gc sync point with the world stopped to avoid racing with allocating threads not stopped on the GC lock.

[sgen] Don't report GC moves with the world running and before a GC happened - it makes no sense!.

[sgen] Move STW GC events to sgen-stw.c so they are properly reported.

[profiler] Move HEAP_START and HEAP_EVENTS to happen outside of heap shots.

They now are produced every time a GC spills heap events such as roots, moves and objects.

Additionally, I changed GC syncpoints to happen in case any GC events are active.
It's a safer choice with little penalty. Figuring out the actual combinations that would require them is non-trivial.

[eglib] Linux needs errno.h for ERANGE

[packaging] Fix wrong argument in Windows script

[jit] Add a ExceptionTraceIp structure to represents entries in MonoException.trace_ips, making it easier to add more information to it in the future. NFC. (#5025)

[eglib] Handle ERANGE in g_strerror

Merge pull request #5020 from lambdageek/fix-strict-loader-image-cache-hit

[loader] If using strict strong names, recheck loaded assm before returning it.

[packaging] Fix prerequisite in Windows script

Use CoreFX sources for System.Drawing.Text (#5036)

Use CoreFX sources for System.Drawing.Imaging (#5035)

* Use CoreFX sources for System.Drawing.Imaging

* Bump API snapshot

Use source from CoreFX for System.Drawing.Drawing2D (#5034)

[packaging] Remove msiexec prerequisite from Windows script

We're using lessmsi now.

[packaging] Exit Windows packaging script on error

Also quote passed in mdk path.

Use GC memory for handle stack under Boehm so it's scanned. Cleanup handle stack on thread unregister.

[metadata] s/HAVE_ONLINE_VES/ENABLE_ILGEN/g

[configure] move HAVE_ONLINE_VES away from autotools to a header file

Bump corefx

[Mono.Security] Fix typo in CryptographicException message

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=52621

[loader] If using strict strong names, recheck loaded assm before returning it.

Suppose we have two versions of an assembly: AsmA 1.0.0.0 and AsmA 2.0.0.0 that
we want to load into the same domain at the same time (for example we want to
load AsmB and AsmC that statically reference the two versions, respectively, or
maybe we used reflection).  Suppose version 1 got loaded first (and we found it
on our MONO_PATH in "somepathdir/AsmA.dll").  Now we want to load version 2.
The problem is that if version 1 is on MONO_PATH first, we'll find a filename
that matches ("somepathdir/AsmA.dll") and try to open that image.  Then we'll
end up here in mono_assembly_open_predicate() after the mono_image_open_a_lot()
call hits the image cache.  So we get back an image and image->assembly is
version 1.

The old behavior would just return any non-NULL image->assembly.  The new
behavior is to check the predicate and reject opening the propose file if the
predicate doesn't match.  (If we're not using the strict loader, or if there's
no predicate set, return the existing image->assembly as before).

[tests] Strict assembly loading of two version of the same assm should work.

In particular, we should be able to use classes/methods that are unique to each
version.  We should not get only one version loaded.

[MacSDK] Remove quotes in nuget wrapper script

None of the other wrapper scripts quote the Mono path and this
broke the Windows packaging script which extracts nuget from the
Mac package.

Revert "[exceptions] only do SIGSEGV related actions for SIGSEGVs in mono_handle_native_crash"

This reverts commit a27bc024f1732edb6b1edddd96d6c90f4a2983f8.

We never want the crash handler to be recursive.

[eglib] Replace strerror with strerror_r as the former is not thread safe.

This change would be nicer if the code was moved to the runtime so we could use our PAL.

[aot] fix compilation for targets that do not set MONO_ARCH_HAVE_FULL_AOT_TRAMPOLINES (#5017)

Merge pull request #5011 from kumpera/quiet-mode

Don't emit profiler events we don't need to.

[profiler] Don't emit sync point on restart the world unless gc moves are enabled.

[profiler] Add + prefix to file names to append the pid. This makes enabling it with env vars doable.

Bump bockbuild (#5023)

[ci] Get rid of remaining usages of WORKSPACE env var

Most of it was already removed in ea519ada8c38cae67f56e27ca70d32923ce9f23a

Bump corefx

[packaging] Fix path to wrapper.bat for Windows MSI

Merge pull request #5002 from BrzVlad/feature-sgen-modes

[sgen] SGen modes

Merge pull request #5021 from BrzVlad/fix-parallel-fixes

[sgen] Fix parallel fixes

[profiler] Fix issue found by review.

[packaging] Simplify Windows packaging scripts

We don't need to build in a separate checkout of mono anymore.
Also use .pkg passed in from upstream build.

[packaging] Move Windows MSI scripts into repository

They lived in https://github.com/mono/release before, moving them
into the main repo allows us to version them more easily.

Note: I intend to do more cleanup on those scripts in the future,
this is just the first step.

[man] Add entries for sgen modes and new major/minor types