@jonpryor reviews Mono.Options (#5204)

Merge pull request #5201 from alexrp/code-owners

Add initial CODEOWNERS file.

Add initial CODEOWNERS file.

[amd64] Mark RBP as 'same value' in the unwind info for the epilog so the unwind info is correct if the method is interrupted after cleaning up its frame. (#5197)

Incorrect parameters passed to GetDiskFreeSpaceEx causing random memory corruption. (#5196)

The call to GetDiskFreeSpaceEx passes uninitialized pointers to GetDiskFreeSpaceEx
instead of pointers to stack variables. This will cause random memory corruption and random
crashes on Windows builds.

This is a regression caused by the refactoring work done in this PR:

https://github.com/mono/mono/pull/4255

Merge pull request #5190 from alexrp/profiler-roots-deprecation

[profiler] Hide the GC roots callback behind a macro and mark it obsolete.

Remove partial path for ml.exe and ml64.exe in mono VS runtime builds. (#5195)

Location of ml.exe and ml64.exe is different in VS2017 compared to VS2015.
By removing the partial path from the command, the VS environment will locate
the tool depending on selected build configuration and VS version.

This should work for both VS2015 and VS2017.

[System.Drawing] Remove tab from .sources

Check return value of asn1_generalizedtime_to_tm ()

[bitcode] Disable tests which use filter clauses when running with bitcode.

Merge pull request #5189 from alexrp/profiler-sample-type-field

[profiler] Remove the type field from TYPE_SAMPLE_HIT.

[profiler] Remove the type field from TYPE_SAMPLE_HIT.

It's unlikely that we'll add support for other kinds of sampling in the future
as we've dropped support for perf on Linux.

[profiler] Hide the GC roots callback behind a macro and mark it obsolete.

Add new version of SocketTest.ConnectedProperty that works on all platforms. The current version is broken on Linux FullAOT and on Mac, so this adds a new version tuned to work reliably on Windows, Linux and Mac (including FullAOT). This addresses https://bugzilla.xamarin.com/show_bug.cgi?id=52157.

Revise new ConnectedProperty test

Merge pull request #5187 from alexrp/debugger-fix-callback-signature

[sdb] Fix signatures of thread-related profiler callbacks.

[sdb] Fix signatures of thread-related profiler callbacks.

This was an unintended change in the profiler API overhaul.

Bump api-snapshot

Merge pull request #5180 from alexrp/profiler-zero-freq

[profiler] Don't enable sampling if frequency is zero.

Merge pull request #5179 from alexrp/profiler-thread-name

[profiler] Report main thread name after attaching.

[System]: minor cleanups. (#5186)

* [Mono.Security]: Add missing IMonoSslStream.WriteAsync() method.

* Cosmetic.

* [System]: MonoTlsStream: put implementation inside #SECURITY_DEP, not the entire class.

[System.ServiceProcess] Remove "browser" service from dependencies in test (#5183)

LanmanWorkstation and SamSs services don't depend on the Browser
service anymore, at least in the Windows Server build that our
Azure VM agents on Jenkins are using now. This started failing a
few days ago so I assume a new build was rolled out.

Merge pull request #5184 from mono/gtk-bump

Bump bockbuild for gtk# API compat fixes.

Bump bockbuild for gtk# API compat fixes.

[corlib] Add test for deleting non-existing file

It shouldn't throw in that case.

Helps cover https://bugzilla.xamarin.com/show_bug.cgi?id=57629
(note that we don't explicitly test on a readonly FS as that is hard
to do in a unit test, but running this test on Android should suffice).

[metadata/corlib] Add apfs to list of fixed drive types

Also added some tests for DriveInfo since we didn't have any.

[profiler] Don't enable sampling if frequency is zero.

Otherwise we would get a SIGFPE at runtime.

Merge pull request #5181 from alexrp/master

[profiler] Fix a type name conflict.

[profiler] Fix a type name conflict.

[arm64] Really fix handler block trampolines.

[profiler] Report main thread name after attaching.

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.