[arm64] Really fix handler block trampolines.

Merge pull request #5088 from alexrp/profiler-lightweight-coverage

[profiler] No longer use enter/leave instrumentation for code coverage.

[profiler] Don't enable GC and thread events for code coverage.

[profiler] No longer use enter/leave instrumentation for code coverage.

This was only enabled to populate a hash table that wasn't actually used.

Merge pull request #5101 from alexrp/profiler-api

[runtime] New profiler API.

[msvc] Update def files after profiler API changes.

[utils] Mark mono_dl_open () as MONO_API.

This is needed on Windows.

[metadata] Mark mono_loader_register_module () as MONO_API.

This is needed on Windows.

[runtime] Rework process waiting. (#5175)

Instead of using fragile lockless code in the sigchld signal handler, wake up the finalizer thread and make it call back
into the process code to signal finished processes.

[arm] Fix the handler block trampoline, it didn't actually call the helper function.

Revert "[arm] Fix the handler block trampoline, it didn't actually call the helper function."

This reverts commit 745a6bec4a1fab5a5f56e00fc8efda84341b0634.
Revert this as it wasn't tested properly.

[arm] Fix the handler block trampoline, it didn't actually call the helper function.

[arm64] Fix the handler block trampoline, it didn't actually call the helper function.

[profiler] Add MONO_PROFILER_API_VERSION macro.

[profiler] Split shutdown callback into early/late callbacks.

[runtime] New profiler API.

* Profiler callbacks can now be changed and disabled at any point.
* API users no longer have to set event flags. The API instead uses a counter
  internally for each type of callback.
* Filter functions for enter/leave instrumentation can be installed, and they
  can choose whether to instrument the prologue, epilogue, or both.
* Managed allocators can now be instrumented for allocation profiling.
* A profiler must now declare that it wishes to use allocation profiling and/or
  sampling in its init function.
* Sampling parameters can now be changed at any point, and the sampling thread
  can be put into an idle mode when no sampling is needed.
* Only one profiler can have control over the sampling parameters. Whichever
  profiler enables sampling first gets control.
* Adding new events is now very easy: One line in profiler-events.h and one
  line wherever the event should be raised.
* Lifted the restriction that enter/leave instrumentation would cause an abort
  in full AOT mode.
* Support for call chain sampling has been removed.
* Support for the old, platform-specific code coverage mode has been removed.
* The new profiler module entry point is mono_profiler_init. If a module has
  a mono_profiler_startup symbol (the old entry point), a warning will be
  printed and the module won't be loaded.
* Updated the profiler test suite to work with instrumented managed allocators.

Merge pull request #5172 from kumpera/proc-handles-pt2

[System] Fix handle leak in Process::GetProcess.

[System] Fix handle leak in Process::GetProcess.

The runtime returns a inc'd handle to managed, so the SafeHandle should decrement on dispose.

[sgen] Use an OS mutex for memory governor log entries

The mutex is used by sgen worker threads that are not attached to the mono
runtime - they cannot use a coop mutex.

[tools] Fixup 098de0b8, inadvertently deleted the wrong csproj

[msvc] Update csproj files (#5170)

[packaging] Fixup mistake from f7c9be

[tools] Cleanup unused files in mdoc

mdoc now lives in https://github.com/mono/api-doc-tools

[offsets-tool] allow to set targetdir for iOS targets, so it's possible to dump offsets without maccore

[jit] Optimize ldloc+ldfld operations for valuetypes, so they don't require making a copy of the valuetype. Hopefully fixes #56452. (#5166)

Revert "[jit] Optimize ldloc+ldfld operations for valuetypes, so they don't require making a copy of the valuetype. Hopefully fixes #56452. (#5166)"

This reverts commit 025ee6399bfa229fedee3c2b8527cd9d1b9886e7.

Revert this as it seems to break the f# build.

[packaging] Add Windows hostfxr.dll to MSBuild in MSI

Part of https://bugzilla.xamarin.com/show_bug.cgi?id=57930

Revert "[packaging/msbuild] Bump to get new changes on xplat-master (#5145)"

This reverts commit 94dcde72917979d9b25b3786e214f08cbc4d2299.

Bump corefx (#5167)

* Bump corefx

* Bump API snapshot submodule

[jit] Optimize ldloc+ldfld operations for valuetypes, so they don't require making a copy of the valuetype. Hopefully fixes #56452. (#5166)

[ci] Tell babysitter where profiler stress result XML lives

This allows it to parse the XML and provide better results
about which tests failed.

[Microsoft.Build] Improve assert in BuildSubmissionTest.EndBuildWaitsForSubmissionCompletion

It started failing more frequently, let's see if we can get better data.
Also did the same change to a similar test.

[jit] Fix mono_debug_count () so it doesn't confuse COUNT=0 with COUNT not being used.

[System.ServiceModel] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight-saving time

[Mono.Posix] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight-saving time

[System] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight-saving time

[corlib] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight saving time

[jit] Use the right type when handling shared versions of RuntimeHelpers.IsReferenceOrContainsReferences (). The previous version only worked by accident. (#5160)

[utils] Fix CppCheck warning in mono-threads-posix.c (#5161)

After the refactoring in 9aaa0083d0def5b940372dbd9375336b036249fd
CppCheck complained about using 'result' uninitialized (looks like
it can't deduce that g_error will abort the process anyway).

[msvc] Update csproj files (#5162)

[mcs] C#7 tuples var deconstruct

[mcs] Adds missing file

[mcs] C# 7 tuple (foundation only).

Still couple of advanced features not implemented and parser cannot
deal with unnamed tuple type at type declaration level.

Merge pull request #5159 from MaurGi/patch-1

Console.Error.WriteLine in System.Net.NetworkInformation

Console.Error.WriteLine in System.Net.NetworkInformation

Console.Error.WriteLine ("Got a bad hardware address length for an AF_PACKET {0} {1}",...

This Console.WriteLine interferes with Apache Storm that reads from the console I/O.

See bug: https://bugzilla.xamarin.com/show_bug.cgi?id=57887

Merge pull request #5144 from kumpera/handle-fixes-part1

[io-layer] Fix two race conditions, one on FD handle creation and one on interrupt install.

Merge branch 'master' into handle-fixes-part1

Merge pull request #5156 from lambdageek/bug-57850+57851

[sre] Deal with ResolveEventHandler returning an AssemblyBuilder (Fixes #57850, #57851)

Fix a couple of bugs in the situation where a `ResolveEventHandler` happens to return an `AssemblyBuilder`.

* [#57850](https://bugzilla.xamarin.com/show_bug.cgi?id=57850) - if the `ResolveEventHandler` returns a reflection only assembly, throw a `FileNotFoundException`
* [#57851](https://bugzilla.xamarin.com/show_bug.cgi?id=57851) - don't look for a `ReferenceAssemblyAttribute` on assemblies returned by a `ResolveEventHandler` if they happen to be an `AssemblyBuilder`. (the native code path to get there is to try and instantiate a custom attribute in some other assembly, hence the fairly convoluted test in `assemblyresolve_event5` and its helper files)

[loader] Don't try to look for ReferenceAssemblyAttribute on dynamic assemblies (Fixes #57851)

.NETFramework seems to ignore this custom attribute and allows instantiating
types from on assembly builders.

[loader] Throw exn if ResolveEventHandler returns a refonly assembly (Fixes #57850)

(Also depends on previous commit to ensure that MonoAssembly:refonly is set
correctly for dynamic assemblies)

[sre] Mark AssemblyBuilderAccess.ReflectionOnly assemblies as refonly

This will ensure they show up in AppDomain.ReflectionOnlyGetAssemblies(), and
won't be allowed to execute

[sre] use AssemblyBuilderAccess bitmask to set MonoDynamicImage flags

In particular this makes sure that AssemblyBuilderAccess.ReflectionOnly aren't
allowed to run.  And AssemblyBuilderAccess.CollectAndRun aren't allowed to save.

[sre] Add MonoAssemblyBuilderAccess enum in native code

in sync with System.Reflection.Emit.AssemblyBuilderAccess

[mono-error] Add mono_error_set_file_not_found

To throw System.IO.FileNotFoundException

[tests] ResolveEventHandler that returns an AssemblyBuilder shouldn't crash mono

Regression test for https://bugzilla.xamarin.com/show_bug.cgi?id=57851

[runtime] Use pthread_cond_timedwait_relative_np () function on osx if available. (#5147)

[aot] Load aot images only in the root domain. (#5151)

[tests] Throw an error if ResolveEventHandler returns a reflection only AssemblyBuilder

Regression test for https://bugzilla.xamarin.com/show_bug.cgi?id=57850

[tests] Add some missing files to EXTRA_DIST

[runtime] Make runtime invoke wrappers for dynamic methods non-shareable, so they can be freed when the dynamic method is freed. (#5152)

[System] Request 32-bit capabilities

This fixes a crash that happened when running in the debugger.
This fixes Xamarin-21261.

Change-Id: I0ccda9f3c11ec02457a932073665721be5e4e83e

Merge pull request #5140 from alexrp/profiler-stress-fpe-fix

[acceptance-tests] Don't pass a zero sampling/heapshot frequency in the profiler stress runner.

[msvc] Update csproj files (#5149)

[packaging/msbuild] Bump to get new changes on xplat-master (#5145)

[bcl] Drop a bunch of unused _2_1.cs files (#5112)

* [bcl] Drop a bunch of unused _2_1.cs files

They weren't used anywhere, not even mobile, so we can remove them.

Revert "[runtime] Use pthread_cond_timedwait_relative_np () function on osx if available. (#5143)"

This reverts commit 616ea50718aba3af237637f77d9c58acdca9a61f.

Revert this as it fails to build on wrench.

[runtime] Use pthread_cond_timedwait_relative_np () function on osx if available. (#5143)

[io-layer] Fix two race conditions, one on FD handle creation and one on interrupt install.

mono_w32handle_new_fd: We must hold scan_mutex while initializing an unallocated handle otherwise
mono_w32handle_foreach might think it's a valid handle.

mono_w32handle_timedwait_signal_handle: We must increment the refcount before installing the interrupt handle
as the other end can be called right after we finish installing and would trigger a failure.

[w32handle] Replace mono_w32handle_{ref,unref} by mono_w32handle_{duplicate, close} (#5106)

* [w32handle] Fix handle dumped ref

* [w32handle] Replace exposing mono_w32handle_{ref,unref} by mono_w32handle_{duplicate,close}, it is closer to what the win32 API exposes.

* [w32handle] Remove unrefing of handles on shutdown

If one of the handle is a file handle for example, then w32file has already be cleaned up, so we cannot guarantee that the handle we are going to close is not going to call into w32file code.

Implement GC roots ourselves in Boehm rather than relying on their concept of roots which is mainly for static data segments in binaries and not implemented on all platforms (cannot remove roots on Windows) (#564)

[packaging] Remove git_branch from msbuild and mono-extensions (#5141)

We're checking out a specific commit for both now so the git branch
is no longer required.

Also fixes warnings that a 2017-06 branch doesn't exist in mono-extensions.

Manually invoke Xvfb, don't depend on the xvfb plugin (#5132)

[build] Add a new profile

[nullgc] When building nullgc, don't try to link Boehm libs

Previous nullgc commit (52d9a55a861527ed4027f8e10a97e3cc880baf4a) didn't actually work in a clean tree.

[acceptance-tests] Don't pass a zero sampling/heapshot frequency in the profiler stress runner.

Also change the code to make it more likely that we actually do some runs
without sampling or heapshot enabled.

[w32handle] Add some debugging code to help track down handle wait problems. (#5136)

[w32handle] Zero out the handle specific data in the handle dtor to help debugging. (#5135)

[msvc] Update csproj files (#5131)

[man] Fix AOT temp path option description

It's `temp-path` in the code, not `temp_path`

Merge pull request #5120 from lambdageek/bug-57744

[reflection] Convert correct MonoError to an exn in Assembly.GetTypes ()

Fix various compiler warnings (#5064)

* Fix compiler warnings (w32)

- add explicit (long long (unsigned)) type casts

- change %ld to %lld once, to match the general style of these logging messages

* Fix compiler warnings (threadpool)

- add explicit (void *) type casts

* Fix compiler warnings (boehm)

- add an explicit (long) type cast; the values are defined as gint64, an upgrade to at least "%ld" seems like a good idea

- about dropping const: const does not seem to be necessary as g_getenv() returns pointers to freshly allocated buffers whenever called. const might have been used to protect the return value of g_getenv() on unix systems with earlier versions of Mono (as a comment in gmisc-unix.c suggests). keeping const here causes a compiler warning, as the const qualifier gets dropped when *env is being used with g_free(). that led to the decision to drop the qualifier altogether.

* Fix compiler warnings (sgen)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

* Fix compiler warnings (misc - metadata)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

* Fix compiler warnings (misc)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

- add an explicit (void *) type cast; changing the return type to char* might be more usable and could be discussed

- ret is not being used and can be removed; fixes a -Wunused-but-set-variable warning

* Fix compiler warnings (mini)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

* [fixup!] Correct indentation

* [fixup!] Correct (void *)
- use GUINT_TO_POINTER and MONO_NATIVE_THREAD_ID_TO_UINT instead of (void *) to cast mono_native_thread_id_get () to %p

* [fixup!] Correct indentation of boehm-gc.c and add FIXME

* include <inttypes.h> and test the G_GINT64_FORMAT macro

* [fixup!] Use G_G*_FORMAT and PRIx32 macros instead of type casts wherever possible

[packaging/msbuild] Bump to get new changes on xplat-master (#5127)

[jit] Avoid emitting op_load_got_addr opcodes on x86 when using llvmonly, its not needed. (#5125)

[reflection] Convert correct MonoError to an exn in Assembly.GetTypes () (Fixes #57744)

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

[bcl] Delete our ReflectionTypeLoadException

We use the referencesource version

[cil-strip] Move comment out of .sources file

I forgot that we only run gensources.sh on libraries but not
tools so the comments don't get stripped out...

[runtime] Add more logging in case mono_os_cond_timedwait () fails.

Bump API snapshot

[System.Core] Implement MemoryMappedFile.OpenExisting

Reuse the existing icall infrastructure to give the same handle back.

This PR implements all OpenExisting methods (with the access rights parameter discarded) and the SafeHandle property.
In the runtime, we don't check the capacity parameter if we try to open a mmap file, and we fallthrough the handle refcount case.

Revert "[cil-strip] Remove old private copy of Mono.Cecil"

(reverted from commit 4ea31227587b9df3a244d9c32a5a87a7afa70436)

Revert "[cil-strip] Upgrade to latest Mono.Cecil API"

(reverted from commit 0c9eb70a4646364f4b425f69b366be766043bef9)

Using newer Cecil means metadata tokens will be different which
breaks AOT compilation: https://github.com/mono/mono/pull/5102#issuecomment-310801767

[msvc] Update csproj files

[llvm] Align aot variables to 8 bytes, parts of the aot code depend on it.

[null_gc] Get null GC building again.

- Had to borrow a gchandles implementation from mono/metadata/boehm-gc.c
- Added coop handles support for NULL GC
- Fixed typos in configure.ac
- To use: `./configure --with-libgc=none --without-sgen --enable-boehm`
  You get a mono-boehm which actually is not collecting anything.

[packaging/msbuild] Bump to get new changes on xplat-master (#5113) (#5114)

[System] Fix compilation on watchos (#5108)

[cil-strip] Remove old private copy of Mono.Cecil

No longer needed now that we're using the new Cecil.

[cil-strip] Upgrade to latest Mono.Cecil API

This greatly simplifies the code and as a bonus also shaves off
about 300ms when stripping mscorlib.dll.

Merge pull request #5103 from alexrp/profiler-signal-ack-fix

[profiler] Do the signal ack after checking that the thread is detached.

[aot] Fix aot image loading messages in static mode.