Merge pull request #3731 from lambdageek/dev-handles-gbmc

[runtime] System.Configuration.InternalConfigurationHost:get_bundled_machine_config  needs HANDLES

Merge pull request #3729 from ntherning/fix-failures-in-msbuild-tests-on-windows

Fix failures in MSBuild tests on Windows

[runtime] Fix the blittable flag and the instance size for pointer types. Fixes #45131. (#3735)

Merge pull request #3737 from akoeplinger/fix-babysitter

[ci] Set the env var in a new subshell to fix babysitter processing

Fix failures in MSBuild tests on Windows

Merge pull request #3727 from henricm/fix-system-web-on-windows

Fixes for System.Web to work on windows

Enabling System.Web tests on Windows

Calling directly into Win API to get current process exe

On Windows the MonoToolsLocator tries to determine which mono.exe
to use by looking at the current process. Due to problems with
AppDomains and shadow copying (which happens in System.Web) we
could not use this approach. Instead we now call GetModuleFileName
directly using p-invoke to get the exe file of the running process.
This is only used when running on Windows.

[System] Use default parsing rules for Uri.IsWellFormedUriString. Fixes #45129

[ci] Set the env var in a new subshell to fix babysitter processing

Setting the MONO_TLS_PROVIDER env var in the testing script makes it opaque to the
babysitter and other processing scripts. By setting it in a new subshell we can
ensure that the scripts see the whole invocation.

Merge pull request #3730 from kumpera/async-reader-dispose-hardening

[System] Add some hardening when disposing a AsyncStreamReader as it will have things happening concurrently.

Merge pull request #3660 from lambdageek/bug-42271

[coop] Switch to GC SAFE for backtrace() and backtrace_symbols()

[runtime] give get_bundled_machine_config ves_icall_Class_Method-style names

[runtime] System.Configuration.InternalConfigurationHost:get_bundled_machine_config
needs HANDLES

The method (which also goes by
System.Configuration.DefaultConfig:get_bundled_machine_config) was
converted to use coop handles earlier, but we missed this declaration.

[System] Add some hardening when disposing a AsyncStreamReader as it will have things happening concurrently.

[SRE] Fixup the tokens in subtypes too. (#3721)

Merge pull request #3725 from ntherning/fix-symbolicate-tests-stack-trace-diff-on-windows

Fix symbolicate tests stack trace diff on Windows

[mempool] Fix indentation in mono_backtrace ()

[coop] Switch to GC SAFE for backtrace() and backtrace_symbols()

Fixes [#42271](https://bugzilla.xamarin.com/show_bug.cgi?id=42271)

[corlib] Add ValueTuple

Fix symbolicate tests stack trace diff on Windows

The symbolicate tests compare stack traces generated by the mono-symbolicate
tool against expected stack traces. The expected stack trace files have been
preprocessed to remove absolute paths and the Makefile does the same with the
output from the tool. However, the preprocessing doesn't take different
directory separator characters into account. This patch adds a 'tr' command to
the Makefile which converts \ -> / before the absolute paths are filtered out
from the actual output from the tool to make the comparison succeed on Windows
as well.

Ignoring CR in WebControls tests

Merge pull request #3707 from lateralusX/jlorenss/win-api-family-support-libmonoutils

Build libmonoutils under none desktop Windows API family.

Merge pull request #3720 from ntherning/fix-monodoc-tests-on-windows

Fix monodoc tests on Windows

Fix a typo.

Merge pull request #3714 from kumpera/fix_process_stream_dispose

[System] Fix Process::Dispose resource leak due to RS import.

[System] Dispose all streams in Process::Close if they were not exposed to the user. Fixes a RS import regression.

RS takes the stance that redirected streams should not be closed by the Process class because they might
be referenced from the outside.

That would be ok if it was true. If you use Process::BeginOutputReadLine, the output stream becomes inaccessible
with no way to dispose them by any mean but GC.Collect. You get an exception when accessing Process::StandardOutput
once async reading starts.

This change addresses the case where the streams are not accessible. This is tracked by the *StreamReadMode variables
that are set on stream getters or Begin*ReadLine.

[System.Net.Http] Wrap certain exceptions into HttpRequestException. Fixes #44978

Merge pull request #3684 from lambdageek/dev-monoerror-poolbox

[runtime] Use boxed MonoError for class failure.

Fix monodoc tests on Windows

These tests are sensitive to differences in line endings between Windows and
other platforms. This patch changes the diff commands used on Windows to
ignore changes in whitespaces at end of lines.

Merge pull request #3692 from lateralusX/jlorenss/win-api-family-support-libmono

Build libmono under none desktop Windows API family.

[mcs] Don't report version mismatch error for FX assemblies

Merge pull request #3709 from rolfbjarne/linker-mark-nested-types

[linker] We need to mark nested types even if the declaring type isn't marked.

[ci] Test System.dll with both the legacy and btls TLS provider

[BTLS]: Certificate cleanups and fixes. (#3706)

* [BTLS]: Allow fallback to MX.X509Certificate in X509CertificateImplBtls.

* [BTLS]: Fix X509Certificate2.MonoCertificate.

* X509Certificate2Impl.FallbackImpl: make abstract.

* X509Helper2.GetMonoCertificate(): new internal helper method.

* X509Certificate2.MonoCertificate: use X509Helper2.GetMonoCertificate().
  This is only used by X509ChainImplMono; we should investigate whether we
  could implement the missing APIs in BTLS.

* [BTLS]: Fully implement X509CertificateImplBtls.Import() and use correct semantics.

.NET's X509Certificate2.Import() uses several implicit fallbacks which we previously
did not handle in BTLS.

* you may provide a (unused) password when importing an unencrypted PKCS#8.

* when using null as password, it will attempt to use string.Empty if importing
  without password fails.

* [BTLS]: Implement X509CertificateImplBtls.PrivateKey.

* [BTLS]: Implement X509CertificateImplBtls.Export().

* [BTLS]: Implement X509CertificateImplBtls.Verify().

* [BTLS]: Fix X500DistinguishedName quoting.

* MonoBtlsUtils: don't quote special chars in UTF-16 strings.

* X500DistinguishedName: don't decode raw data when passed in a name.

* [ci] Test System.dll with both the legacy and btls TLS provider

* [ci] Only run the additional btls System.dll test run on OSX for now

Otherwise we might run it on platforms where btls isn't supported yet and get errors.

(cherry picked from commit 9638ee9ece023eb96f314de64624bf37ce4b7faf)

[System.Net.Htpp] Add test for large content range headers

Nowdays Content-Range From  could be much more than 2147483647

Merge pull request #3705 from BrzVlad/fix-sgen-internal-alloc

[sgen] Fix block size computation for allocation size

Merge pull request #3697 from directhex/fix-make-dist-missing-basic.exe

When basic.exe isn't in DISTFILES, it gets excluded from `make dist`

Merge pull request #3711 from akoeplinger/fix-fullaot-tests

Fix two test failures in the FullAOT Jenkins job

[runtime] Don't do logging from a signal handler. It's not signal safe.

I found this while trying to MONO_LOG_LEVEL=debug an process way bug and the removed log statements
would hang mono in a matter of seconds.

[MonoError] Don't overwrite computed TypeLoadException message with empty string

Fixes [#44729](https://bugzilla.xamarin.com/show_bug.cgi?id=44729).

[loader] Add descriptive error messages to class failure.

In cases where a class fails to load due to a related class failure,
extract the message from the related failure.

[runtime] Change how class failure is represented.

Use a single bit MonoClass::has_failure to represent whether a class has
failed.  Store detailed error message in MONO_CLASS_PROP_EXCEPTION_DATA property
in the MonoImage of the class.

Fix two test failures in the FullAOT Jenkins job

Disable WebOperationsContextTest.Current like on monotouch, it relies on dynamic code generation.
The namedmutex-destroy-race.cs runtime test doesn't work on mobile_static since we don't have named Mutexes.

Add mono-package-runtime to EXTRA_DIST

The file was added in 15b6ef60e45bb978203b57da4b956aa1d1f86f3c

[corlib] Enable more RuntimeHelpers tests

[linker] We don't need to mark nested types more than once.

Merge pull request #3693 from rolfbjarne/watchos-simplify-http-message-handler-selection

[System.Net.Http] Simplify default http message handler for watchOS since there's only one valid value.

Merge pull request #3704 from rolfbjarne/watchos-servicepointmanager

[System] Add back a few ServicePointManager properties that can be used by multiple networking stacks.

[linker] We need to mark nested types even if the declaring type isn't marked.

[loader] Make mono_class_get_failure static in class.c

[loader] Add mono_class_set_type_load_failure ()

- Hide mono_class_set_failure as a static function in class.c
- Use a MonoErrorBoxed to represent a MONO_EXCEPTION_TYPE_LOAD.

[loader] Use MonoErrorBoxed for mono_class_set_failure (..., MONO_EXCEPTION_INVALID_PROGRAM, ...)

Use a MonoErrorBoxed to represent an invalid program class failure.

[MonoError] add mono_error_vset_type_load_class function

A variant of mono_error_set_type_load_class that takes a va_list
argument instead of varargs.

[MonoError] Add MonoErrorBoxed for mempool-stored errors.

The usual MonoError is suitable as a short-lived stack-allocated error
with a lifetime confined to a function call.
Internally it performs allocations using g_malloc/g_free.

The MonoErrorBoxed is a mempool-allocated error whose lifetime is tied
to the lifetime of a MonoImage.

We provide two operations:

1. MonoErrorBoxed* mono_error_box (MonoError *ierror, MonoImage *image);

  Creates a new boxed error in the given mempool from MonoError.
  It does not alter ierror, so you still have to clean it up with
  mono_error_cleanup or mono_error_convert_to_exception or another such function.

2.   gboolean mono_error_set_from_boxed (MonoError *oerror, MonoErrorBoxed *from_box);

  Sets the error condition in the oerror from the contents of the given
  boxed error.  Does not alter the boxed error, so it can be used in a
  future call to mono_error_set_from_boxed as needed.  The oerror
  should've been previously initialized with mono_error_init, as usual.

[sgen] Fix block size computation for allocation size

The block size for a slot size is the minimum block size that can fit two slots. Make sure the computation accounts for the fact that even though a slot size might fit twice in the block, it still has to be allocated at aligned addresses, so it can end up using a higher block size.

Fixes 44922.

Merge pull request #3686 from lambdageek/dev-format-printf

Add MONO_ATTR_FORMAT_PRINTF macro

Build libmono under none desktop Windows API family.

Initial work to build libmono under none desktop Windows API families.

Build libmonoutils under none desktop Windows API family.

Initial work to build libmonoutils under none desktop Windows API families.

Add EXTRA_DIST listing all the files from btls

Apparently there's no way to get CMake to just spit out its own uncached variables, so we can't do this programatically.

Remove useless BTLS_DEPS variable

[System] Add back a few ServicePointManager properties that can be used by multiple networking stacks.

Merge pull request #3695 from ludovic-henry/fix-thread-test

[tests] Fix p/invoke test on windows

[sdb] Avoid adding the first file name in the debug info to the list of file names if a method has no seq points. Fixes #44982. (#3696)

Merge pull request #3700 from alexrp/profiler-c9-fixes

[profiler] Fix a regression that caused all buffers to have a zero thread ID.

[profiler] Use macro constants instead of passing a bunch of bools.

[profiler] Update a stale comment.

[profiler] Add some asserts.

[profiler] Simplify send_log_unsafe ().

[profiler] Fix a regression that caused all buffers to have a zero thread ID.

This was introduced in 928b840dad3c9cc1c39c3fe5820bf8394ae9b66b.

[SRE] Remove more dead code.

[SRE] Remove more unused icall declarations.

[SRE] Remove more unused icalls.

[btls] Add missing icall definition for mono_btls_x509_revoked_get_revocation_date

[SRE] Small code cleanup.

[SRE] Remove unused icall from inside a DISABLE_REFLECTION_EMIT block.

Merge pull request #3698 from alexrp/profiler-c9-fixes

More C9 profiler fixes

[SRE] Delay the creation of the MonoClass for dynamic types until CreateType () is called. Remove some icalls. (#3687)

[tests] Fix thread-native-exit test on windows

[metadata]: Unconditionally enable ves_icall_Mono_Btls_Provider_IsSupported().

The MonoBtlsProvider.IsSupported() icalls ves_icall_Mono_Btls_Provider_IsSupported()
needs to be available whether BTLS is available or not because it is used at
runtime to decide whether or not to enable BTLS.

(cherry picked from commit 7b2a96fbd372ad74c2595b473c00ac7e1ed9ca67)

[profiler] Fix BYTE_SIZE macro conflict on OS X.

[profiler] Fix PROF_TLS_FREE () on pthread TLS systems.

[profiler] Fix the build on Android (missing fcntl.h include).

Oops, actually correctly revert that commit.

(cherry picked from commit 04379d5d73838d12f772ac99cae4360dbd5d8c7a)

[System]: Temporarily switch the default TLS provider back to "legacy".

Also revert the Jenkins commit in c462f4e6032cd70d692461f09b295671d380eef8.

(cherry picked from commit cd1b0b8edf43ee61dc87ebc7edbc44f7d56204b5)

When basic.exe isn't in DISTFILES, it gets excluded from `make dist`

[Mono.Debugger.Soft] Move TypeLoadClass types to new file to improve testing

[System.Net.Http] Simplify default http message handler for watchOS since there's only one valid value.

[ci] Temporarily switch back to legacy TLS provider

[Mono.Debugger.Soft] Tweak ClassLocalReflection test

Merge pull request #3691 from ntherning/enable-tests-in-pr-builds-on-windows

Enable tests in Windows PR builds

[System*] Throw a PlatformNotSupported exception when using the managed networking stack on watchOS. (#3683)

* [System.Data] System.Data.SqlClient is not supported in watchOS.

System.Data.SqlClient is not supported in watchOS, because it depends on a
working managed networking stack, which we don't have on watchOS.

* [System] Throw PlatformNotSupportedExceptions in the networking stack for watchOS.

* [System] Don't reference/build Mono.Security.dll when building for watchOS.

This means we can't build Mono.Data.Tds.dll either, since it uses
Mono.Security.dll extensively. In any case Mono.Data.Tds.dll is useless
without a working networking stack, and since it's a Mono-only dll (i.e. not
part of netstandard or facades), just remove the entire assembly from the
watchOS profile.

System.Security.dll uses Mono.Security.dll as well, but the required classes
are available inside mscorlib.dll (as internal classes), so just open up
mscorlib internals to System.Security.dll.

* [System] Update tests after API change for watchOS.

* [System.Data] Update tests after API change in watchOS.

* Revert "Fix build break from 81fb20737f810def6dde88d87a1b2f23cdb736c3"

This reverts commit f4475ff01358f8490e7cd0878e8b39bed2a0def7.

* Fix missing trailing newline in monotouch_watch_System.dll.sources

It breaks the build on Linux if the newline is missing.

Enable tests in Windows PR builds

Also run the test suite in Windows PR builds. We disable all the test suites
in Windows PR builds that are known to fail.

[System] Server property of HttpWebResponse should never be null

[btls] Fix i386 build on Linux by setting -arch on Darwin only

The -arch option is an Apple extension, it caused failures on i386 Linux: "unrecognized command line option '-arch'"

Merge pull request #3689 from alexrp/profiler-c9-fixes

Profiler fixes for Cycle 9 (#2)

[profiler] Fix GC finalize object end event not getting invoked.

Revert "[domain] Report domain unloads to the profiler when finalizing the domain."

This reverts commit c8b20290808563f89d29b22d2fe35eb906cfe725.

This caused ordering problems with app domain events: https://bugzilla.xamarin.com/show_bug.cgi?id=44552

Conflicts:

mono/metadata/gc.c

Merge pull request #3688 from alexrp/profiler-c9-fixes

Profiler fixes for Cycle 9