Update CODEOWNERS

[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.

[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.