Bump API snapshot submodule

Bump CoreFX

Update bockbuild. Fixes #57002

[runtime] Fix the size calculation for CustomAttributeType coded indexes, they can only point to the MethodDef/MemberRef tables even through 3 bits are used to encode them, only 2 values are used. Fixes #58210. (#5256)

[jit] Handle OP_VCALL_MEMBASE in get_simd_vreg (). Fixes #58344.

Bump bockbuild.

Bump roslyn to 2.3.1

Bump bockbuild.

Update CODEOWNERS

Merge pull request #5252 from alexrp/profiler-stress-fixes

[acceptance-tests] A few fixes for the profiler stress test suite.

Merge pull request #5253 from alexrp/profiler-lib

[Mono.Profiler.Log] Catch SecurityException for missing icalls.

[Mono.Profiler.Log] Catch SecurityException for missing icalls.

This is the exception that is thrown on .NET because icalls are not allowed in
non-GAC assemblies.

[acceptance-tests] Disable the ironjs-v8 test on arm32/arm64.

It times out even with a 24-hour timeout. Eventually we should investigate why,
but disable it for now so the lane can become green.

[acceptance-tests] Fix a crash in the profiler stress runner caused by racey modification of a StringBuilder.

[acceptance-tests] Set individual timeout for profiler stress tests back to 8 hours.

[acceptance-tests] Build profiler stress runner with debug info.

[bockbuild] Avoid overwriting self.local_configure_flags in some packages, extend them instead. (#5239)

Merge pull request #5248 from alexrp/profiler-api

[profiler] Add back some legacy profiler API functions.

[mempool] Annotate functions for clang's ThreadSanitizer (#5191)

* [mempool] Annotate functions for ThreadSanitizer
Whitelist `mono_mempool_alloc`, `mono_mempool_new_size` and `mono_mempool_destroy` according to https://bugzilla.xamarin.com/show_bug.cgi?id=57936

* [fixup!] Test the NO_SANITIZE_THREAD macro

* [fixup!] Merge comments / documentation

[acceptance-tests] Temporarily increase the profiler stress timeout to 24 hours.

This is temporary and is just to see how long ironjs-v8 actually takes to complete on ARM. We may need to just exclude it there.

Bump API snapshot submodule

Implement ServiceEndpoint.EndpointBehaviors

This follows the reference source in understanding `EndpointBehaviors` to be an alias of `Behaviors` with a more general type.

Note that although an alias might not seem necessary to implement, when using netstandard1.3 the [`System.ServiceModel.Primitives`](https://www.nuget.org/packages/System.ServiceModel.Primitives) package exposes `EndpointBehaviors` but not `Behaviors`.

[msvc] Update project files with new profiler-related source files.

[Marshal.IsComObject] Make this predicate return false instead of throwing an exception which is what the calling code expects (#5246)

Reenable MonoTests.Remoting.IpcChannelTest.Bug609381 (#5245)

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

[msvc] Update def files.

[profiler] Add back some legacy profiler API functions.

These are used by XA/XI/XM and because XM in particular needs to work with
both old and new versions of system Mono, we need to make sure these functions
still exist.

[profiler] Rename mono_profiler_install to mono_profiler_create.

Merge pull request #5198 from BrzVlad/fix-binprot-stats

[sgen] Parallel gc improvements

[jit] test builtin types (#5208)

* [builtin-types] import test suite

List of original contributors:

Zoltan Varga <vargaz@gmail.com>
Rodrigo Kumpera <kumpera@gmail.com>

* [builtin-types] import updated definitions from xamarin-macios

https://github.com/xamarin/xamarin-macios/blob/8fbbfa5ca7154fcfd553f7a7b242ffee9a95f240/src/NativeTypes/Primitives.tt

* [builtin-types] make regression suite work

* [builtin-types] disable broken tests

* [fixup] disable failing tests on arm64

[interp] reuse finally_ips infrastructue for fault clauses

Tweak the Unreal profile

Merge pull request #5243 from alexrp/profiler-stress-fixes

[acceptance-tests] Don't mark msbiology as a failure if it OOMs on 32-bit.

[sgen] Fix function signature

[sgen] Compact allocated block list

It helps with its repeated traversal and improves work distribution when using parallel modes.

[sgen] Improve card table parallelization

Improve caching by scanning consecutive blocks and avoiding full block list travesal.

[sgen] Run a single worker cycle in major parallel finish case

This was not the case when we were doing a forced concurrent finish.

[sgen] Avoid logging unnecessary binary protocol events

[sgen] Add explicit binary protocol stats for collections

Using the cardtable scan and finish_gray_stack start and end events is cumbersome to parse and require a lot of logging, especially in the case of the parallel collector.

[sgen] Include mod union times to stats

[sgen] Use parallel minors for pause mode

It should makes pauses less influenced by workload spikes

[sgen] Avoid waiting for workers to report their own finish

If a worker is stuck doing work in another context, it might not be very responsive in transitioning its own state from WORK_ENQUEUED to WORKING and finally to FINISHING. When joining worker threads, we check whether we have any threads working in the corresponding context and, if there aren't any, change their state directly from the gc thread.

We can do this because work is distributed from centralized pools (worker_distribute_gray_queue and the job_queue), instead of per worker. Therefore we don't need all the workers to perform actual work in the context.

Update CODEOWNERS

[acceptance-tests] Don't mark msbiology as a failure if it OOMs on 32-bit.

This is known to happen on some platforms.

[threads] Add checks around libc calls (#5240)

[threads] Make mono_thread_detach_internal static (#5241)

[acceptance-tests] Further increase the individual test timeout in the profiler stress runner.

It seems ironjs-v8 takes even longer to run on ARM than fsharp, msbiology, and roslyn.

[bcl] Fix the parallel build in the tools dir. (#5236)

[offset-tool] fix assignment for GeniOS

fixes this compilation error

MonoAotOffsetsDumper.cs(279,59): error CS0201: Only assignment, call, increment, decrement, await, and new object expressions can be used as a statement

[btls] Remove create-object-library.sh script which is no longer used. (#5232)

Fix build of "unreal" profile

Merge pull request #5224 from alexrp/profiler-multiple-args

[profiler] Better support for multiple profilers.

[System.Net.Http]: Close request stream when HttpClientHandler.SendAsync() is done writing data. (#5226)

[acceptance-tests] Increase individual timeout of profiler stress tests to 8 hours.

fsharp, roslyn, and msbiology take a very long time to run on ARM. Hopefully this is enough.

[llvm] Disable an optimization in LLVM where we deleted and recomputed the MONO_INST_INDIRECT flags during codegen. This doesn't work since the INDIRECT flag prohibits the insertion of phi nodes, so its later removal leads to the generation of LLVM IR which is missing phi nodes. (#5227)

[msbuild] Bump msbuild, fix build breakage

[llvm] Set the MonoJitInfo.from_llvm flag for methods loaded from llvm, it got broken by 1debf3934120547b3003c0ec4ec90bae4b08ee13. (#5223)

[offsets-tool] Add a --gen-ios argument which can be used together with --targetdir= to generate ios offsets without having to specify the macos path. (#5218)

[msbuild] Bump msbuild to fix build breakage

.. specifically get msbuild@81687aa7dd4ad9b76c52c1efa841b4527e0f45f9

[profiler] Fix a crash in the AOT profiler when the output option isn't given.

[mini] Support loading multiple profilers with the --profile option.

The runtime has ~always supported having multiple profilers loaded, so it
didn't make much sense that only the last --profile option given would be used.

[Mono.Profiler.Log] Remove coverage event support.

I intend to move code coverage support to a separate profiler module, so these
events will eventually be removed from the log format. As such, I'd rather not
commit to having these event classes in the API.

This won't affect users of the library who don't enable code coverage.

[profiler] Profiler entry point name must now include the module name.

So mono_profiler_init becomes e.g. mono_profiler_init_log.

This allows us to link together multiple profilers into a single module and
load them selectively. This is especially useful for XA/XI/XM.

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.