[msbuild] Bump msbuild to fix build breakage

.. specifically get msbuild@81687aa7dd4ad9b76c52c1efa841b4527e0f45f9

Merge pull request #5194 from xmcclure/orbis-freeswept

Do not major_free_swept_blocks on Orbis.

[msbuild] Bump to get new changes on xplat-master

Merge pull request #5206 from alexrp/profiler-gc-base-init

[profiler] Don't call mono_gc_base_init () before loading profilers.

[profiler] Don't call mono_gc_base_init () before loading profilers.

Calling this function could cause us to miss some profiler events.

In the past, we needed to call this function so that SMR and mono-threads were
initialized. Since then, 8dec9848f6208531ae14ea08f123c01da0357c42 changed
profiler initialization to happen later during startup, so this is no longer
necessary.

Merge pull request #5216 from alexrp/profiler-runtime-settings

[profiler] Introduce runtime control API.

Add SoapAction to Content-typ for SOAP 1.2 (#5219)

[msvc] Update csproj files (#5220)

Bump corefx (#5217)

[System] Suppress a few instances of CS0618 obsolete warnings

[corlib] Preserve List<T>.ConvertAll in TypeTest (#5215)

TypeTest.GetGenericMethodDefinitionOverInflatedMethodOnGTD accesses
List<T>.ConvertAll() via reflection but doesn't use the method
elsewhere so the linker can remove it. Added some usage to avoid that.

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

[profiler] Use g_strerror () instead of strerror ().

[profiler] Change sample frequency to a 32-bit int.

The unit is hertz, so values above 100-1000 are already extremely unusual.
Changing it to a 32-bit int avoids word tearing on 32-bit platforms.

[profiler] Introduce runtime control API.

Merge pull request #5210 from alexrp/profiler-runtime-settings

[profiler] Housecleaning

[profiler] Move heapshot logic to the correct GC event.

MONO_GC_EVENT_START is actually sent _after_ all the 'stop world' events. The
heapshot logic being in this switch case only worked by accident previously.

[profiler] Clean up MonoProfilerGCEvent enum.

* Remove the mark/reclaim events as they've been broken for a long time and
  aren't particularly useful.
* Reorder the enum member declarations to the order they actually arrive in.

[bcl] Remove the ValueAdd and InetAccess NUnit categories (#2212)

ValueAdd was added over a decade ago and all the tests marked with this category are passing now.

We have *tons* of tests that rely on working DNS/Internet that aren't marked with
InetAccess (and they actually provide good value so we don't want to exclude them).

Since these categories don't really make sense and just causes confusion
it's better to just remove them.

[profiler] Add a heapshot-on-shutdown option.

[pedump] Print failure cause message when a class cannot be initialized

Add @luhenry and @kumpera to CODEOWNERS for some files

[llvm] Reserve the proper number of got entries in the llvm got instead of 16 which used to be enough in the past. Hopefully fixes #56111. (#5207)

[metadata] Encode public key length correctly for keys longer than 128 bytes

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

For keys longer than 1024 bits, we encoded the length incorrectly as
   0x80 (length & 0xff)
instead of the correct algorithm:
  if (length < 0x80)
    length
  else if (length < 0x4000)
    0x80|(length>>8) (length & 0xff)
  else
    0xc0|(length>>24) (length>>16)&0xff (length>>8)&0xff length&0xff

which is provided by mono_metadata_encode_value.

See ECMA-335 II.24.2.4 #US and #Blob heaps

N.B.: we now always allocate the memory for the key as (keylen + 4) which
over-allocates by a couple of bytes in the cases where the key is shorter.

[tests] Add regression test for InternalsVisibleTo with long PublicKey

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

We build a library that has an InternalsVisibleTo attribute that references an
executable that is signed with a long (>128bytes) public key.  Previously we
were using an incorrect method of encoding the key length.

[tests] Use test runner for test-internalsvisibleto

Also add the source files to EXTRA_DIST

[Microsoft.Build] Relax check in BuildSubmissionTest a bit more

[Mono.Profiler.Log] Fix LogProcessor.ReadObject ().

[msvc] Update VTune project files.

[profiler] Fix sampling thread shutdown logic to handle idle mode correctly.

[profiler] Use tabledefs.h constants for method attributes.

[profiler] Move the log format description to the header.

[acceptance-tests] Update the profiler-stress runner for removed options.

[profiler] More options reworking/refactoring.

This commit does a few different things.

First, metadata event options have been removed. We now always emit these
events. This is done because they contain important data that must be present
in order for dynamic event mask changes at runtime to result in useful data.
Also, almost all uses of the profiler require this data to be present, so
letting it be turned arguably just adds confusion for users.

Second, the PROFLOG_*_FEATURE flags have been removed. These made the code a
bit confusing because it was unclear whether code in log.c should be checking
the PROFLOG_*_FEATURE flag or the PROFLOG_*_EVENTS flag for a particular kind
of event. For heapshot in particular, the heapshot mode is now a separate enum
configuration variable. For enter/leave and coverage, switching those on and
off in the log profiler at runtime is not something we want to support at this
time.

As a result of the above change, some options are now processed manually rather
than being part of the event_list array.

Third, the way we turn off GC root events when needed is now by setting the
callback to NULL. This is more efficient as it signals to the runtime that it
doesn't have to gather root information. This will especially matter later on
when the profiler root info revamp lands, as it collects a lot more data than
we do today. Also, GC roots are no longer gathered unless we're doing a
heapshot.

Fourth, the profiler now always starts the sampling thread in idle mode. This
is so that we can reconfigure and enable/disable it at runtime as needed.

Finally, the alloc alias option now includes GC move events, and GC resize
events are only emitted if the gc option is given.

[w32error] Add ENXIO error case (#5199)

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

[Microsoft.Build] Make BuildSubmissionTest.EndBuildWaitsForCompletion more reliable (#5209)

We've seen this test failure a few times:

```
MESSAGE:
  #2
  Expected: greater than or equal to 00:00:01
  But was:  00:00:00.9544900

+++++++++++++++++++
STACK TRACE:
  at MonoTests.Microsoft.Build.Execution.BuildSubmissionTest.EndBuildWaitsForSubmissionCompletion () [0x000d8] in /mnt/jenkins/workspace/test-mono-mainline-linux/label/ubuntu-1404-amd64/mcs/class/Microsoft.Build/Test/Microsoft.Build.Execution/BuildSubmissionTest.cs:110
  at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00032] in /mnt/jenkins/workspace/test-mono-mainline-linux/label/ubuntu-1404-amd64/mcs/class/corlib/System.Reflection/MonoMethod.cs:305
```

Rewriting the test to use Stopwatch instead of DateTime
for measuring elapsed time should make it more reliable.

[profiler] Clarify docs for mono_profiler_set_sample_mode ().

[profiler] Use mono_cpu_count () from utils/proclib.

[utils/proclib] Export mono_cpu_count () for use in the profiler.

[profiler] Update file headers.

[profiler] Use EXIT_LOG instead of EXIT_LOG_EXPLICIT where possible.

[profiler] Clean up counter data structures a bit.

[profiler] Consolidate log profiler state.

Previously, the profiler's state was spread all over the place. Some stuff was
in the MonoProfiler structure, some stuff was in global variables littered
throughout log.c. Now everything is stored in MonoProfiler and ProfilerConfig.

[profiler] Drop support for the old-style sample option.

[profiler] Remove the need to pass a MonoProfiler pointer around everywhere.

[profiler] Clean up logging to use the new logging functions.

[profiler] Remove support for the output=-FILENAME option.

[profiler] Remove the debug-coverage option.

[utils/logger] Export logging functions for the profiler.

[profiler] Use explicit values for enum members.

[profiler] Remove unused TYPE_END enum member.

[profiler] Update sample hit size calculation.

[llvm] Add support for fault clauses. Fixes #58114. (#5203)

Add myself to a few paths in CODEOWNERS

Also fix a few broken markdown links in contributing.md that that I noticed.

@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

Do not major_free_swept_blocks on Orbis.
Also, the existing major_free_swept_blocks exemption for Windows is more appropriate for Windows *hosts* than Windows *targets*.

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.