Merge pull request #3749 from BrzVlad/fix-mips-fix

[mips] Fix mips

Merge pull request #3753 from henricm/enable-windowsbase-tests-on-windows

Enable WinowsBase tests on Windows

[System] Warnings cleanup

Merge pull request #3750 from marek-safar/socket

[System] More Socket pieces from referencesource

[mips] Disable div with mul on 32bit mips

[llvm] Avoid generating invalid IL for valuetypes+vphi+volatile variables in gsharedvt methods. Fixes part of #45270.

[corlib] Fix UdpClient tests on watchos

They were reenabled in 1fa85ad4afb4139e607876121a080a3be122a7a9,
but UdpClient throws PlatformNotSupportedException on watchos.

Merge pull request #3715 from kumpera/fix-44707

[corlib] Don't load the machine config file if it's missing. Fixes #44707

[cert-sync]: Change back to using the '--btls' argument.

Without arguments, 'cert-sync' now operates on the old certificate store
again, like it was before.  To test BTLS, you need to explicitly run it
with '--btls'.

(cherry picked from commit 46379bda6d13e6e216ca3fc35654eb910c891c41)

Merge pull request #3745 from akoeplinger/fix-bug44937

[io-layer] Correctly detect a PE32+ assembly as managed when starting process

[System.ServiceModel] Split unreliable tests

[System] Set more error details on failed https proxy connect. Fixes #45108

[mono] Fixes hardcoded stack limit in EnsureSufficientExecutionStack check

[System] More Socket pieces from referencesource

Enable WinowsBase tests on Windows

Merge pull request #3662 from henricm/fix-windowsbase-on-win

WindowsBase zlib, ZipSharp and Package fixes for Windows

Making sure stream is closed when exception is thrown

Stream will never be closed if we throw exception in Open
which leads to file being locked in Windows and that we
leak file handles.

Setting Culture to en-us for ToStringTest

Make ZipSharp work on Windows

This patch adds the zlib functions needed by ZipSharp to the VS
MonoPosixHelper.def export file.

This patch also changes the ZipSharp code to use different code paths
depending on the size of the C long type on the underlying platform. On
gcc/clang the C long type follows the bitness of the targeted architecture,
it's 32-bit on 32-bit systems and 64-bit on 64-bit systems. With the VS
compiler however, the C long type is always 32-bit regardless of the target
architecture. zlib and minizip uses C long in a number of different function
signatures and structs.

Merge pull request #3746 from ntherning/fix-SocketResponder-on-windows

Fix SocketResponder on Windows

Merge pull request #3747 from marek-safar/updclient

UpdClient from referencesource

[mips] Add missing define

[mips] Don't assert when decomposing longs

There is no point in asserting if we encounter an unhandled long op. These ops may still get removed, if emulated later on, or maybe from other passes. Asserting happens as a last resort when emitting the native code for the method.

[corlib] Add missing constructors to RNGCryptoServiceProvider on monotouch

Those were missed in cb46f78eef5263ce24ba12c583e4c72ac64734c1 because there's a separate
implementation for monotouch in an unrelated file.

[System]: Fix a typo in X500DistinguishedName.AreEqual().

(cherry picked from commit 1ac809aad7a2ca1d69c6d26e7ac18b36d5c90610)

[btls] Use UTC for the Unix epoch in the managed BTLS code (#3702)

The epoch is specified as being in UTC, but the DateTime(int, int, int) constructor
creates a DateTime with an 'Unspecified' timezone, which could lead to problems later on.

[BTLS]: Correctly handle certificates with negative serial numbers. (#3719)

In mono_btls_x509_get_serial_number(), first call i2c_ASN1_INTEGER()
which takes care of padding and computing the 2s complement for negative
numbers, then reverse the buffer.
(cherry picked from commit 1a55680bfaffa3fa00881889133e5313a6cd9ab4)

[System.Core] Remove unused files

Fix SocketResponder on Windows

There seems to be a race in some tests in the System test suite that use
SocketResponder on Windows. Some tests fail with an exception that the socket
has been closed before they get a chance to see the response sent by the
SocketResponder. The same happens occasionally when using SocketResponder on
.NET. This patch changes the order of the Socket.Shutdown() calls and adds a
Socket.Receive() before Shutdown() is called to give the sender a chance to
actually send any data before the responder shuts down and closes its side of
the connection.

[System] Enable more tests

[System] UdpClient from referencesource

[System.Data] Use same socket family for data receiver as it has its client. Fixes #44624

Merge pull request #3740 from Unity-Technologies/gc-options-command-line

GC debug and params command line options

[io-layer] Correctly detect a PE32+ assembly as managed when starting process

We weren't checking whether a PE file is in PE32 or PE32+ format,
causing us to use the PE32 offsets on the newer format as well.

This means that using Process.Start() on a managed assembly didn't
always work when a PE32+ file was in play. Such a file is created
when targeting x64 e.g. via the -platform:x64 csc.exe/mcs option.

The reason why we probably didn't notice until now is that in an
assembly produced by mcs there happens to be some data at the PE32
CLR header offset even in a PE32+ file, causing the check to "work".

This doesn't apply to csc.exe/roslyn though and so we couldn't
execute those assemblies via Process.Start() even though they're
perfectly managed. Since the .NET Core project.json toolchain
produces x64-targeted assemblies by default more people ran into
the issue trying to run those on Mono: https://github.com/cake-build/cake/issues/1247

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

PE/COFF spec at https://msdn.microsoft.com/en-us/library/windows/desktop/ms680547(v=vs.85).aspx

Merge pull request #3739 from akoeplinger/add-more-netstandard-api

[bcl] Expose a few more APIs on mobile to match with netstandard

[bcl] Expose a few more APIs on mobile to match with netstandard

See https://github.com/dotnet/standard/issues/47.

Merge pull request #3653 from ntherning/alertable-socket-wait

Make blocking socket calls on Windows interruptible

Merge pull request #3744 from ntherning/enable-system-security-tests-in-pr-builds-on-windows

Enable System.Security tests in PR builds on Windows

Enable System.Security tests in PR builds on Windows

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

[profiler] Start the helper thread before the writer thread.

[profiler] Start the helper thread before the writer thread.

This fixes a regression introduced in 0db428169a42373a73cd5c502bd40033629f3f8d
which caused the profiler to not report the correct command port in the file
header on startup.

[SRE] Set the wastypebuilder flag on completed generic param builder … (#3712)

* [SRE] Set the wastypebuilder flag on completed generic param builder classes to remove a workaround from mono_type_get_object_checked ().

* [SRE] Remove unused verify_safe_for_managed_space () function.

[llvm] Make sure the initlocals_bb is not treated as part of a try block if one starts at IL offset 0, since it could mess up the control flow if it contains calls. (#3718)

Merge pull request #3741 from akoeplinger/fix-aot-DoAssemblyResolve

[aot] Fix AppDomain.DoAssemblyResolve runtime invoke wrapper

[aot] Fix AppDomain.DoAssemblyResolve runtime invoke wrapper

In https://github.com/mono/mono/pull/1528 the DoAssemblyResolve method was changed
from two parameters to three, but the code in aot-compiler.c was not updated to match.

Document --gc-debug and --gc-params command line options

Add --gc-params and --gc-debug command line options

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

Make blocking socket calls on Windows interruptible

This second commit adds functions to socket-io.c which uses
WSAEventSelect()/WSAWaitForMultipleEvents() or overlapped I/O to make
potentielly blocking socket calls interruptible (alertable).

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

Make blocking socket calls on Windows interruptible

This first commit changes the signatures of the icalls for blocking socket
calls in Socket to propagate whether the socket call should block or not. This
is required since it's not possible on Windows through any native API to
determine whether a socket is in blocking or non-blocking mode.

This commit also adds the necessary RegisterForBlockingSyscall() guards to the
Socket.SendFile_internal() method which takes a WSABUF array. Without this
calls to this method won't be signaled when the Socket is closed.

This commit also fixes the error handling in the Socket.SendFile() method to
not rely on WSAGetLastError(). It's likely to have been overwritten between
when the error is generated and the WSAGetLastError() call from managed code.

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

[corlib] RemotingConfiguration now try to use the bundled machine.config first.

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

[corlib] Don't load the machine config file if it's missing. Fixes #44707

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.