Merge pull request #2903 from krytarowski/netbsd-support-4

Implement mremap(2) usage on NetBSD

Fix the loaded llvm build.

Merge pull request #3370 from ludovic-henry/fix-finalizer-wait

[gc] Fix crash when doing WaitForPendingFinalizers

[gc] Fix crash when doing WaitForPendingFinalizers

The crash would come from a use-after-free of stack memory: in coop_cond_timedwait_alertable, we would allocate the struct we pass to the mono_thread_info_install_interrupt function on the stack. But there is no guarantee as to when break_coop_alertable_wait is run. The fix is to allocate the data passed to mono_thread_info_install_interrupt on the heap, and free it in the callback, or if it hasn't been interrupted.

[x86] Only pinvoke methods are stdcall. (#3354)

For xamarin bug 30146.

[runtime] Fix the loaded llvm build.

Merge pull request #3356 from marek-safar/cecil-update

Cecil update

[runtime] Store the original invoke method in WrapperInfo for delegate invokes, use that to fix the check in find_aot_method_in_amodule ().

[monosymbolicate] Update project file

[Mono.Cecil] Add special handling of legacy 0.9.5 version

[mono-symbolicate] Better test failure reporting

[mdoc] Update project file

[Mono.Contract] Less obsolete warnings

[bcl] Reflect cecil breaking changes

Update Cecil to the latest master

Merge pull request #3363 from akoeplinger/fix-42938

[xbuild] Use the xbuild ToolsVersion even if another one is specified in the csproj

[arm] Fix aot support in the sdb soft breakpoint trampolines.

[ci] Don't build mobile profiles in a monolite build

The mobile profiles rely on system resgen2 which is not present in a monolite build.

Merge pull request #3359 from ntherning/sgen-avoid-scanning-stack-guard-page-on-windows

Prevent SGen from scanning the guard page used by Windows to dynamically grow the stack

Merge pull request #3361 from xmcclure/babysitter-xml

Add mono/tests support to CI babysitter

Merge pull request #3365 from ntherning/fix-gc-graystack-stress-on-win64

Fixes the mono/tests/gc-graystack-stress test on Windows x64

Merge pull request #3337 from akoeplinger/fix-alpine

Fix Alpine build

[ci] Add a CI_TAGS option for building with monolite

Also ensure that all the CI_TAGS check work with multiple values.

[ci] Run "make" in parallel in run-jenkins.sh

To speed up the build.

Merge pull request #3288 from ludovic-henry/iolayer-extract-threading

[io-layer] Extract threading code

Prevent SGen from scanning the guard page used by Windows to dynamically grow
the stack.

Windows uses a guard page before the committed stack memory pages to detect
when the stack needs to be grown. If we suspend a thread just after a function
prolog has decremented the stack pointer to point into the guard page but
before the thread has been able to read or write to that page, starting the
stack scan at the thread's ESP/RSP will raise a STATUS_GUARD_PAGE_VIOLATION and
the process will crash.

This patch uses VirtualQuery() to determine whether the thread's stack_start
value points into the guard page. If it does the stack scan will instead be
started at the following, non-guard page.

Bump binary-reference-assemblies

Fixes the mono/tests/gc-graystack-stress test on Windows x64

Windows x64 only has 1 MB of stack per thread which is less than other x86
64-bit OSes. Using 10000 for "depth" in this test causes a stack overflow on
Windows x64. This patch changes "depth" to 5000 only on Windows x64. 5000 will
fit within the default stack size.

Merge pull request #3355 from ludovic-henry/process-fix-exited

[process] Fix Process.Exited event triggered too soon

Remove stray comment

[arm] Fix the initialization of the variable holding the single step trampoline address when using soft breakpoints + aot.

Merge pull request #3360 from alexanderkyte/mini_fix_timeout

[runtime] Increase mini bitcode timeout and add parallelism

[mobile_static] Give bitcode mini tests parallelism and more time

[xbuild] Use the xbuild ToolsVersion even if another one is specified in the csproj

This is the recent behavior in .NET/MSBuild as well [1]:

> Starting in Visual Studio 2013, the MSBuild Toolset version is the same
> as the Visual Studio version number. MSBuild defaults to this Toolset
> within Visual Studio and on the command line, regardless of the
> toolset version specified in project file.

With this change xbuild will always use its own version instead
of the one specified in the project, unlessa ToolsVersion is forced
via the /tv switch on the command line.

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

[1] https://msdn.microsoft.com/en-us/library/bb383796.aspx#Anchor_1

Add NUnit XML support to test babysitter, also document the communication protocol better

For test harnesses which don't support the full babysitter protocol,
the test harness can now report back paths to NUnit-format XML files
containing test results, or the paths to those XML files can be given
by whoever invoked the babysitter script. Failures listed in the XMLs
are included in the babysitter log.

test-runner.cs has been modified to report its XMLs, so we should now
see mono/tests failures in the babysitter log.

Merge pull request #3357 from tritao/zip_delete_entries

[System.IO.Compression] Remove the entry from ZipArchive entries when its deleted

[process] Fix Process.Exited event triggered too early

The Process.Exited event would be triggered too early, leading to crash in code using this event to access the process exit code. This is because in Process.ExitCode, we first check that the process has really exited.

Merge pull request #3358 from ntherning/disable-sgen-red-zone-on-win64

Set SGen's REDZONE_SIZE=0 on Win64

Set SGen's REDZONE_SIZE=0 on Win64 since the ABI defines no red zone and the
ABI states that "All memory beyond the current address of RSP is considered
volatile".

On Windows the stack grows dynamically as the thread needs more stack space.
Windows places a guard page before the page at the top of stack and when this
guard page is accessed by the thread Windows will add a new page to the stack
and resume the thread.

Before this fix SGen on Win64 subtracted 128 from the stack pointer of a
suspended thread and started the stack scan from this address. In the case
when the stack pointer would be less than 128 from the start of the memory
page SGen would attempt to read from the previous page. If this happens to be
the guard page a STATUS_GUARD_PAGE_VIOLATION exception is raised and the
process crashes. The sgen-weakref-stress.exe triggered this bug on almost
every invocation.

[io-layer] Remove threads.h, thread-private.h and wthreads.c

[io-layer] Remove unused _WAPI_THREAD_CURRENT

[io-layer] Extract thread MonoW32HandleOps

[io-layer] Extract GetThreadPriority and SetThreadPriority

[io-layer] Extract wapi_create_thread_handle

[io-layer] Extract wapi_thread_own_mutex and wapi_thread_disown_mutex

[io-layer] Extract wapi_current_thread_desc

[io-layer] Extract wapi_ref_thread_handle

[io-layer] Remove _wapi_thread_cur_apc_pending

[io-layer] Extract wapi_get_current_thread_handle

[io-layer] Extract wapi_thread_handle_set_exited

[io-layer] Make _wapi_getpid, _wapi_thread_own_mutex, _wapi_thread_disown_mutex and _wapi_mutex_abandon functions public

[io-layer] Make _WapiHandle_thread struct public

[mono-threads] Split platform and suspend specific functions

We split mono-threads-{posix,windows,mach}.c functions in 2 categories:
 - platform: depends on the host the runtime is compiled on (possible values: posix or windows)
 - suspend: depends on the mono-threads suspend backend in use (possible values: posix, windows or mach)

[io-layer] Make wait.c not depend on _wapi_thread_cur_apc_pending

[System.IO.Compression] Remove the entry from ZipArchive entries when its deleted. Fixes #43022.

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

[mcs] Better error reporting for missing types during conversion

[build] Update order.xml data

[arm64] Assert that the displacement is not out of range int he b/bl macros, instead of in arm_get_disp (), so the assert location is more meaningful.

Merge pull request #3352 from alexanderkyte/mini_bitcode_fix

[runtime] Provide Mono.Simd.dll to llvmonlycheck+fullaotcheck

[runtime] Provide Mono.Simd.dll to llvmonlycheck+fullaotcheck

Merge pull request #3351 from kasthack/patch-2

Added missing DeliveryFormat property to MailClient

[arm] Initialize breakpoint_tramp later in full aot mode, its not available in arch_init ().

Added missing DeliveryFormat property to MailClient

Doesn't actually change the behavior but at least allows to build sources referencing it on mono.
https://msdn.microsoft.com/en-us/library/system.net.mail.smtpclient.deliveryformat(v=vs.110).aspx

Merge pull request #3347 from lambdageek/coop-abort-ignore-unhandled

[coop] Don't call unhandled exn hook for ThreadAbortException

[mcs] More tweaks in betterness improvements. Fixes #41724

[coop] Don't call unhandled exn hook for ThreadAbortException

Fixes the weird Jenkins logs for the coop job.

Merge pull request #3344 from alexanderkyte/propagate_mono_error_async

[runtime] Propagate async MonoError to AsyncResult

[bcl] Don't build not installed mobile assemblies

Merge pull request #3340 from lambdageek/dev/invoke-unhandled-exception-hook

[runtime] Move mono_invoke_unhandled_exception_hook to metadata

Merge pull request #3345 from alexanderkyte/fix_mini_aot_flags

[runtime] Pass invariant AOT options to mini aot tests

Merge pull request #3269 from alexanderkyte/silence_zero_len

[runtime] Disable noisy warning

[test] Fix sporadic Mono.Debugger.Soft ThreadpoolIOsinglestep failure.

ReadAsync size arg should not be greater than array.Length - offset.

Fix the build.

[runtime] Propagate async MonoError to AsyncResult

[runtime] Implement WaitForPendingFinalizers used cond vars instead of io-layer events. (#3315)

[threads] Call mono_invoke_unhandled_exception hook in mono_thread_internal_unhandled_exception.

When a GC or threadpool thread dies due to an unhandled exception,
invoke the unhandled exception hook to exit, instead of exiting directly.

[runtime] Pass invariant AOT options to mini aot tests

Bump to include the F# bockbuild patch

[arm] Use soft breakpoints on watchos.

Merge pull request #3342 from ludovic-henry/gc-domain-finalizer-sem

[gc] Replace domain finalizer request cond+mutex by a semaphore

Merge pull request #3335 from lambdageek/dev/system-reflection-assembly

[reflection] Move AssemblyName to managed

[llvm] In default_mono_llvm_unhandled_exception call mono_invoke_unhandled_exception_hook to exit

instead of exiting directly.

[Facades] Emit symbol files for lib like facades

[man] Fix typo in environment variable name

The section was added in 410a200ecd07f5e53ffea858d204b3bbf88d311d but with the wrong name.

[gc] Fix timeout in mono_domain_finalize

[gc] Ensure we clean the DomainFinalizationReq

[gc] Replace domain finalizer request cond+mutex by a semaphore

This is possible because we only wait for it at one place, and only post it at one other place.

[arm] Implement support soft breakpoints + AOT.

[sdb] Fix an aot assert.

[corlib] Bring AssemblyName(string) to the happy C# world.

[corlib] Move Assembly::GetName to be implemented in managed.

This involves a few steps.

First define MonoAssemblyName in RuntimeStructs.cs

Then add some helped to decode blob arrays and handle hex digits.

Finally, move the actual implementation from native to managed.

Merge pull request #3320 from lambdageek/dev/reflection-in-managed

[reflection] Marshal name strings from managed

[runtime] Move mono_invoke_unhandled_exception_hook to metadata

And call it consistently in thread start functions when the managed
thread start function returns with an exception.

(If coop is not enabled, the exception handler unwinds the thread and
when there's no handler invokes the unhandled exception hook via
jit_tls->abort_func, so this is just for the coop case.)

Fixes mono/tests/unhandled-exception-1.cs for coop and bitcode.

[reflection] Marshal name arg of RuntimeType.GetNestedTypes in managed code

Also do the name unmangling on the managed side.

[reflection] RuntimeType.GetNestedTypes in managed

[reflection] Marshal name arg to some RuntimeType search methods

Marshal the name argument of Get{Methods,Properties}ByName,Get{Fields,Events}
in managed code.

Cleanup the corresponding icalls.

[corlib] Add Mono.SafeStringMarshal struct

Allows safe allocation of native utf8 char* strings from managed
strings.

using (var h = new SafeStringMarshal (str)) {
   /* use h.Value IntPtr */
} // g_free'd by h.Dispose ()

[mcs] Recursive interface declaration check is needed for interfaces only. Fixes #42198

[mcs] Do exactly matching expression check for anonymous methods with successfully inferred return type only. Fixes #42611

Revert "[runtime] Add documentation to mono-os-{mutex,cond,sem} + Move to .c file"

This reverts commit 9ba062e5a183f3ace3a21e2e3d240adc3db0e5a4.