Merge pull request #3802 from lambdageek/dev-reference-attr-take3

[loader] Re-enable the reference assembly attribute check

Bump boringssl submodule to the correct commit

It was accidentally reverted in https://github.com/mono/mono/commit/95b0698304cbc54fdfd2eac00692c9c8c3c8875d

[corlib] SharedStatics from referencesource. Fixes #46175

Merge pull request #3851 from akoeplinger/nunitlite

Switch BCL testing from nunit to nunitlite

[System] Disable a test that reads a file on mobile

We don't have a way to pass the file to our mobile test runners at the moment.

[System] Fix monodroid build when BTLS is disabled

[nunitlite] Port babysitter work from nunit24

See https://github.com/mono/mono/commit/1dc0598badaed161efa92fd261de7e555d1a2b13

[System.ServiceModel] Make looper thread in ChannelDispatcher a background thread and join instead of abort

This showed up when switching to nunit-lite since it runs all tests in the same AppDomain,
meaning it won't exit when any foreground threads are still running. This was the case
with the thread created by ChannelDispatcher which was still waiting on a connection.

Also try to Join() the thread for a bit so it exits gracefully before trying to abort,
might resolve issues like in https://github.com/mono/mono/pull/3819

[System.Web.DynamicData] Fix references to nunit.mocks

nunitlite.dll does not include namespace NUnit.Mocks and it is not necessary.

[System.Web] Remove Explicit attribute from test

With nunitlite the test is executed even though it's marked as ignored when the attribute is there.
This is probably the same bug as https://github.com/nunit/nunit/issues/741 (we're passing -exclude for some categories to nunit).

[WinForms] Disable the ApplicationTest.RestartNotSupportedExceptionTest for now

It causes an infinite loop because nunit-lite restarts itself over and over when running the test
as the tests don't run in a separate AppDomain so the host process is restarted.

[Microsoft.Build] Ignore ThreadAbortException in BuildNodeManager

It showed up after test runs with nunit-lite because it caused writes to Console.Error.

[System.Xml/System.ServiceModel/Dataflow] Use more specific type in assert

Apparently nunitlite is more strict here so we need to allow subclasses as well
and test for the more specific type.

[System.Xml] Fix embedded assembly name in XmlCodeExporterTests

nunitlite doesn't run the test in a separate AppDomain so we get the name of the console host.

[System.Runtime.Remoting] Fix double call to ChannelServices.UnregisterChannel

It is already called in TestFixtureTearDown, and calling it twice results in an exception in UnregisterChannel
the second time because the channel is no longer registered.

[System.Data.OracleClient] Replace config file lookup of ConnectionString with env var

This is easier with the new nunitlite config file handling.

[System.Data.Linq] Fix "build" subdirectory to work with nunitlite

TODO: we should refactor this, seems crazy to have a frankencopy of the test machinery in this folder.

[corlib/System/System.Configuration] Fix tests relying on the current directory to be the test folder

nunitlite runs tests shadow-copied in a temp folder, so looking up the resource based on the current dir doesn't work.

[corlib] Disable a test that is incompatible with how nunitlite runs tests

[bcl] Replace deprecated usage of Assertion class and Is.InstanceOfType usages.

nunitlite does not contain Assertion class.

nunitlite does not implement Is.InstanceOfType, replaced usages of:
Assert.That (type, Is.InstanceOfType(typeof(T))) with Assert.IsInstanceOfType.

[bcl] Add NUnitHelper.cs with API not in nunit-lite

NUnitHelper.cs adds CollectionAssert, FileAssert, StringAssert.

Also adds AssertHelper class with methods that are not part of nunitlite
Assert class.

[bcl] Add patching of nunit-lite-console.exe.config file

Because nunit-lite runs everything in the same AppDomain, we can't use dll.config files for settings anymore.
Since a few test suites rely on being able to read those settings we need to patch them into the main
nunit-lite-console.exe.config file instead before the test.

Remove all _test.dll.config files and replace with nunit-lite patcher equivalent

[bcl] Replace nunit24 with nunit-lite

Get rid of TEST_HARNESS_VERBOSE and enable verbose logging by default

nunit-lite doesn't have the "dot when test runs" progress indicator anymore,
so we need to enable "verbose" output by default otherwise we don't get
any progress until the test run is over.

Also remove the TestResult_$(PROFILE).log and nunit-summary.xsl files,
they're not needed anymore.

Merge pull request #3852 from monojenkins/update-csprojs

[msvc] Update csproj files

UdpClient.Receive parameter refers to remote EP (#3842)

Instead of listening to a local end point, listen to any remote endpoint answering to our request.

[msvc] Update csproj files

[threads] Replace use of W32Handle by MonoOSEvent for MonoThreadInfo exited event (#3659)

* [mono-threads] Make thread wait platform specific

* [mono-threads] Replace thread w32handle use by event

This allows use to remove the MonoThreadInfo dependency on the w32handle on posix.

* [w32handle] Move them to metadata

w32handle are purely for managed support, and they shouldn't be used in the runtime.

* [mono-threads] Factor open/close thread handle

* [mono-threads] Factor exit thread

* [mono-threads] Factor platform register/unregister thread

* [mono-threads] Use ThreadHandle on all platforms

* [threads] Fix mutex abandonning for main thread

According to the comment, the intent is to abandon the mutex hold by the current thread. This intent was conveyed by the previous code when the threading code was still in the io-layer.

[tests] Build AOT prereqs for reference-loader test

More metadata cleanup (#3792)

* [runtime] Introduce a new mono_class_init_sizes () function which initializes only the instance fields without initializing fields if possible. Use it in a few places instead of mono_class_init ()/mono_class_setup_fields ().

* [runtime] Simplify mono_class_has_references (), it doesn't have to special case the init_pending case.

* [runtime] Use a TLS list instead of the 'init_pending' flag to detect recursion in mono_class_init ().

* [runtime] Reduce the size of the critical section in mono_class_init () by doing most of the work outside the lock.

* [runtime] Avoid loading cached class info for pointer etc. types.

* [runtime] Reduce the locking inside setup_interface_offsets ().

[sre] Remove some dead code.

Merge pull request #3847 from rolfbjarne/btls-remove-unused-pinvokes

[System] Remove unused BTLS P/Invokes.

[bcl] Disable BTLS on xammac and xammac_net_4_5 profiles

[btls] Fix library install

We need to ensure the directory is created and doesn't have an extraneous '/' char.

[threads] Fix suspend on watchos

Because certain platform (such as WatchOS) do not support syscall abort, we need to ensure that it is supported before we call the `mono_threads_suspend_abort_syscall` function.

configure.ac: fix incorrect description

[gc] Remove mono_gc_set_stack_end (#3845)

* [sgen] Add stress test

This hope to suspend a thread while it is in the process of attaching or detaching.

* [sgen] Fix unified suspend

This would cause a crash with unified suspend: if a thread would be suspended at mono-threads.c:1168, the stack_start would be greater than the stack_end, triggering the g_error at sgen-stw.c:351. The stack_end would initially be initialized in `sgen_client_thread_register`, but it would be overriden in start_wrapper at threads.c:893.

Merge pull request #3846 from monojenkins/update-csprojs

[msvc] Update csproj files

Merge pull request #3844 from tritao/deflate_fixes

[System.IO.Compresion] DeflateStream fixes

[System] Remove unused BTLS P/Invokes.

They point to internal functions that don't exist, causing compilation
problems for Xamarin.Mac apps.

Example test/compile failure: https://gist.github.com/rolfbjarne/43cf2228816bbe8a863f331da3e07c0d

[msvc] Update csproj files

[corlib] Disable a few tests that are not supported on MOBILE_STATIC for now

They were added in 895f3f5ae8e22c14511baa8c2a5c3f1b17009c02.

[llvm] Add the 'notail' flag to non-tailcalls when using the 3.9 based llvm.

[System.IO.Compression] Fixed potential bug when keeping track of total read bytes of input stream.

This didn't manifest as a bug, to to be on the safe side, replicate the behavior
introduced in https://github.com/mono/mono/commit/b4d5016b5e42fec226d93fd13260b6cac80eb384
and use the same approach when calculating the total of the read bytes of the input stream.

[System.IO.Compression] Fixed DeflateStream input stream EOF condition.

What was happening is that we set the stream EOF flag to true when we
could not read any more input data, and that made us early out before
fully finishing the processing of the compressed data.

Now the only EOF flag is set to true is if zlib's inflate() has actually
finished processing the data.

Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=44994#c2.

[coop handles] Don't scan null object references

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

[System.IO.Compression] Fixed DeflateStream inflate() decompression loop.

This reworks the fix from https://github.com/mono/mono/commit/7c4d41a518aaf0370b882f9430752ebcd5544c10.

That fix introduced a regression when handling some inputs, which lead us to ignore
some left-over data from zlib's inflate output buffer.

Now we only break from the loop if inflate() returns Z_BUF_ERROR.

This makes sure all data is fully processed before returning to managed code.

Fixes one of the bugs in https://bugzilla.xamarin.com/show_bug.cgi?id=44994#c2.

Merge pull request #3769 from evincarofautumn/fix-verify-before-allocs

[sgen] Fix verify-before-allocs.

[btls] Remove the statically linked btls code, the dynamic version should work on all platforms. (#3829)

[bcl]: Fix 'Mono.Btls.Interface' build logic.

(cherry picked from commit 43558a209d64e5e91f137df0bfc7caec80fee114)

Merge pull request #3800 from madewokherd/mingwbuild

Use lower-case names for Windows headers.

[man] Typo fixes for the man pages

disco.1: fix typo, closes #3841

csharp.1: fix typos, closes #3840

crlupdate.1: fix minor typo, closes #3839

certmgr.1: fix typos/grammar, closes #3838

ccrewrite.1: fix minor grammatical errors, closes #3837

ccrewrite.1: fix copyright, closes #3836

Following comment changed the copyright:
https://github.com/mono/mono/commit/5a3fb39ca6a46250aec3afef21c85bef30239fc5

Reflect same change in the comments.

Merge pull request #3826 from monojenkins/update-csprojs

[msvc] Update csproj files

Merge pull request #3831 from rolfbjarne/watchos-fix-defaultproxy-test

[System] Tweak define to exclude System.Net.WebProxy.CreateDefaultProxy and update corresponding test.

Merge pull request #3832 from microMolvi/patch-2

cccheck.1: fix a typo

[runtime] Fix a test so it works with bitcode, stack traces might contain wrapper frames.

[llvmonly] Add gsharedvt out wrappers for icalls, the icall wrappers have a different signature (this/no this).

cccheck.1: fix a typo

[suspend] Remove sgen specific suspend code (#3640)

* [suspend] Remove sgen specific suspend code

* [sgen] Remove unused SgenClientThreadInfo->stopped_domain field

* [sgen] Remove unused SgenClientThreadInfo->stopped_ip field

* [threads] Unsplit code for syscall abort

* [threads] Reuse posix suspend signal handler function for abort

* [mono-threads] Move debug printf to separate file

This is to avoid recompiling half of the runtime for every change to these printf.

* [stw] Slightly improve logging

* [mono-threads] Fix unified thread suspend

`mono_threads_suspend_begin_async_suspend` return FALSE iff it coudn't successfully suspend the thread, not if the thread is not in the correct state. The latter is checked by calling `mono_threads_suspend_check_suspend_result` which will return FALSE if we couldn't initialize the state of the suspended thread.

* [mono-threads] Fix unified thread suspend

If we try to suspend a thread that is still attaching, we might suspend it after it allocated the MonoInternalThread and MonoThread, but before it set the current domain or lmf. This would lead us to not scan the thread stack, which would lead to collect the newly allocate MonoInternalThread/MonoThread, leading to a crash. By letting sgen deal with the thread, even if the domain or the lmf is not set, we avoid this bug.

This makes unified suspend have the same behaviour as the previous sgen specific suspend.

* [mono-threads] Fix compilation

[System] Tweak define to exclude System.Net.WebProxy.CreateDefaultProxy and update corresponding test.

[mono-config] fix warning (#3820)

```
mono-config.c:324:14: warning: passing 'const char *' to parameter of type 'void *' discards qualifiers [-Wincompatible-pointer-types-discards-qualifiers]
                                        g_free (libdir);
                                                ^~~~~~
../../eglib/src/glib.h:125:20: note: passing argument to parameter 'ptr' here
void g_free (void *ptr);
                   ^
1 warning generated.
```

[llvm] Use the global llvm context instead of creating a new one for each aot module, since the latter seems to cause random memory corruption/crashes.

[btls] Install the btls shared library on make install. (#3825)

[mcs] Add pathmap option

[msvc] Update csproj files

[mobile-static] Increase trampoline limits even more.

[mobile-static] Increase the number of imt trampolines to 2000 to prevent running out of them in the System.Core tests.

[runtime] For delegate invoke wrappers, store the generic invoke method in the wrapper info and inflate it when inflating the wrapper. Fixes ipy failure. (#3821)

Merge pull request #3823 from mono/fix-btls-managed

[System] Don't build managed BTLS code on monotouch

[bcl] Only build Mono.Btls.Interface and btls tool when BTLS is enabled

[System] Don't build managed BTLS code on monotouch

After 2fb07d6c6d5b3915ef4665391febbb7b8be09fb5 BTLS can be used as a shared lib,
but this caused an issue in some monotouch tools which grepped the P/Invokes for `__Internal`
since these icalls wouldn't resolve on monotouch since BTLS is disabled there.

Instead, we now completely leave out building the managed parts of BTLS when
BTLS is not enabled.

Fix compilation errors in mono/utils/os-event-win32.c with Mingw/GCC (#3818)

This was introduced in 92be5357d12ecf0cc4c51fbcb76fc0514b375d1c. The patch
also adds the os-event-win32.c file to the libmonoutils Visual Studio project.

[BTLS]: Fix a ref-counting leak in X509ChainImplBtls.  Fixes #45687. (#3811)

* [BTLS]: Fix a ref-counting leak in X509ChainImplBtls.  Fixes #45687.

* Set 'untrusted = null' in Dispose().

(cherry picked from commit c9a67ffc5addabe74015f7d8f33977f4ab416075)

[System] Clean up some warnings

[runtime] Ignore bit 14 in typedef export table during verification. It seems to be used by some WinRT predefined types

Merge pull request #3817 from monojenkins/update-csprojs

[msvc] Update csproj files

MonoLinker: Fix a Typo in the README (#3816)

[msvc] Update csproj files

Merge pull request #3812 from rolfbjarne/watchos-no-cfnetwork

[System] Remove any CFNetwork usage from the watchOS profile. Fixes #45847.

Merge pull request #3815 from microMolvi/patch-1

caspol.1: fix minor grammatical errors

[x86] Clear the fp stack when exceptions are thrown. Fixes #45841. (#3814)

caspol.1: fix minor grammatical errors

[runtime] Fix an assert message.

[System] Remove any CFNetwork usage from the watchOS profile. Fixes #45847.

The MacProxy class uses CFNetwork, but since CFNetwork is not a public
framework on watchOS, we can't use it.

So remove MacProxy completely (it only contains internal classes), and throw
PlatformNotSupportedException in any API that used it (the managed networking
stack is not supported on watchOS anyway, so this should be safe).

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

[runtime] Add MonoOSEvent

Make mkbundle use SDKs instead of a single runtime for cross compilation

Make mkbundle use SDKs instead of a single runtime for cross compilation

[tests] Missing custom attr on an assembly should not crash mono.

Merge pull request #3801 from akoeplinger/netstandard20-updates

[bcl] Move a few types from the Facades into the framework for netstandard20 compatibility

Merge pull request #3810 from lateralusX/jlorenss/add_missing_assert_include

Explicit include of assert.h.

Explicit include of assert.h.

[configure.ac] Only disable -Wno-zero-length-array on clang

The $GCC variable in autoconf will be yes for clang as well [1].

[1] https://lists.gnu.org/archive/html/autoconf/2014-09/msg00023.html

Merge pull request #3442 from lateralusX/jlorenss/win-atexit-commands

Add support on Windows to install atexit handler(s) early in bootprocess.

Merge pull request #3808 from monojenkins/update-csprojs

[msvc] Update csproj files

[mono] Add some strong types for a few variables, document some methods (#3777)

[msvc] Update csproj files

[btls] Convert BTLS icalls to pinvokes by invoking them using [DllImp… (#3799)

* [btls] Convert BTLS icalls to pinvokes by invoking them using [DllImport("__Internal")], which will make it easier to redirect them to a separate dylib in the future.

* [btls] Add a --enable-dynamic-btls configure flag to enable compiling btls into a separate shared library instead of embedding it into the runtime.

There's a G_BREAKPOINT in mini-exceptions.c which could be hit during user execution. This will not be caught by our normal crash reporter, so we should use g_error here instead. (#3791)

[runtime] Fix the native-to-managed marshalling of null blittable arrays. Fixes #45788.

[bcl] Move a few types from the Facades into the framework for netstandard20 compatibility

See https://github.com/dotnet/standard/pull/76.

Also moved IncrementalHash from System.Core.dll to mscorlib.dll as per discussion with Wes.

[tests] Re-enable reference-loader.exe test

This reverts commit c08acd2eb48596cec148183fd828403b77074b01.

[loader] Set image assembly before reference attribute assembly check.

In mono_assembly_load_from_full, the call to
mono_assembly_has_reference_assembly_attribute has to come after
image->assembly is set.  This is because image->assembly->base_dir is
used for probing for referenced assemblies, and the reference assembly
attribute check will probe for assemblies referenced by the custom
attributes on the assembly in image being loaded.