[Socket] Fix Socket.BeginConnect to multiple IPAddress (#4960)

We would previously mix 2 SocketAsyncResult, and in case we would succeed asynchronously, we would return one that would not be updated afterwards.

The solution consists in using only 1 SocketAsyncResult that we ensure to update accordingly.

Merge pull request #4935 from lambdageek/dev-handles-may

[coop handles] w32event, w32handle and DNS icalls

[runtime] Avoid calling mono_class_from_mono_type () for sre arrays, it could initialize unfinished element types. Create a normal MonoType structure instead. Fixes #56611. (#4952)

[System] Adds watch specific HttpListenerWebSocketContext

Bump bockbuild to include gtk# bump (#4959)

Bump api-snapshot

[unaligned] relax test regarding floating point precision

[interp] use memmove for some primitve types, because it takes care of unaligned addresses

relevant on architectures which do not support unaligned load/stores.

Bump corefx

[build] Fixes profile path for aot compilation

Bump cecil

[System] Add missing configuration properties to the list of properties (#4954)

After looking at https://github.com/mono/mono/pull/4949 I did a
quick pass through the other config sections and found two more
cases where we missed adding to the properties list.

[MacSDK] Bump xamarin-gtk-theme.py to latest revision from private bockbuild

[MacSDK] Port two msbuild fixes from private bockbuild

https://github.com/xamarin/bockbuild/commit/58467b852e13b4bdfc75a2a0f9b9507b417b0694:

    [msbuild] Create symlinks from lib/mono/xbuild/Microsoft/NuGet/*
    msbuild install script cannot do this as the source files are installed
    by mono, which are in a different staging directory.

https://github.com/xamarin/bockbuild/commit/fb9656a7613bab2d77c8ce7e05d673c5abbbdac5:

    Partially Revert "[msbuild] Remove redundant msbuild logic from mono_master.py"

    Use 15.0 instead of mono version! msbuild's install script cannot
    copy these since bockbuild installs msbuild to a separate staging
    directory from mono. So, it needs to be done here.

Bump bockbuild to include gtk+ patch

[System] Add useDefaultCredentialsProp to the list of properties. (#4949)

[runtime] Don't make the domain id allocatable before we clear the domain (#4942)

Otherwise, while we are clearing the domain, another domain might be created that has the same id as the current one. This would lead to crashes due to releasing of data from the newly created domain (ex gchandles).

Merge pull request #4938 from kumpera/optimize_ref_queries

Add concurrent read version of MonoGHashTable and use it with MonoDomain::refobject

[win32] Add new files.

[w32socket] Remove dead code

[w32socket] Use coop handles for System.Net.Dns.GetHostByAddr_internal

[w32socket] Use coop handles for System.Net.Dns.GetHostByName_internal

[mcs] Don't emit param-array attribute on method overrides, it's ignored by C# compiler. Fixes #53244

Bump bockbuild

[S.R.I.RuntimeInformation] Fix FrameworkDescription on Android (#4946)

* [S.R.I.RuntimeInformation] Fix FrameworkDescription on Android

It turns out using `DllImport ["__Internal"]` doesn't work on
XA (and other cases where Mono is embedded) because it searches
for the method in the host which embeds Mono and doesn't find it,
resulting in an EntryPointNotFoundException.

We need to switch to an icall instead. However there's already
an existing icall+BCL wrapper: Mono.Runtime.GetDisplayName().

Added IVT from corlib to S.R.I.RuntimeInformation so we can call it.

* Update API snapshot

[mcs] Reset catch state variable after it has been checked. Fixes #54322

[mcs] New property/indexer cannot have accessors implementing an interface in base type. Fixes #56627

Bugfix Ping.cs (#4936)

[interp] implement RELOC_LONG_BRANCH.  fixes test/imt_big_iface_test.exe

[runtime] Handle mono_method_body_get_object returning NULL on success.

[Facades] Update to match latest contracts (#4939)

* [Facades] Update to match latest contracts

* Update API snapshot

[CI] ignore appdomain-unload-asmload.exe on interp and full-aot

[android] add hardware specific workaround for Nexus9 in armv7 mode

we got a couple of bug reports, all with the same failure:

https://bugzilla.xamarin.com/show_bug.cgi?id=44907
https://bugzilla.xamarin.com/show_bug.cgi?id=46482
https://bugzilla.xamarin.com/show_bug.cgi?id=51791

The bugs are private, therefore here what I wrote:

```
Thank you Matthias, this was very helpful.

>>> 09-30 12:24:50.347: A/DEBUG(6211): pid: 6191, tid: 6209, name: Thread-4  >>> com.distech.x50.ui.droid <<<
>>> 09-30 12:24:50.347: A/DEBUG(6211): signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x8
>>> 09-30 12:24:50.347: A/DEBUG(6211):     r0 d6b58728  r1 00004001  r2 00000000  r3 d6b58738
>>> 09-30 12:24:50.347: A/DEBUG(6211):     r4 00000008  r5 ea97427c  r6 00001f3c  r7 00000015
>>> 09-30 12:24:50.347: A/DEBUG(6211):     r8 00001f38  r9 00000015  sl d6b58728  fp d67fed90
>>> 09-30 12:24:50.347: A/DEBUG(6211):     ip ea9742c0  sp d67fed60  lr ea7a4504  pc ea716a58  cpsr 800e0010

The crash happens on this assignment:
https://github.com/mono/mono/blob/de1865dad5c0350f391fedcaa08f02f610530d3f/mono/mini/mini-generic-sharing.c#L418
Here the according disassembly:
https://gist.github.com/lewurm/b1094749027c9e5ea19fdc4fac7905a7

The crash happens in the last loop iteration (`r9=i`, `r7=slot`).  Looking
at the machine code it just _cannot_ happen, which is confirmed by the C
code as well: `*oti` successfully happens in the if check, but after
returning from `alloc_oti()`, `oti` doesn't contain a valid address anymore.
I suspect some weird hardware issue that fails to restore all registers
properly from the stack.
```
[...]

```
Thanks again Matthias.  Unfortunately, I'm out of ideas, and I can't
blame anything but the hardware. The situation we see is too weird. We
segfault at offset `0xdda60: str sl, [r4]`, but really we should
already segfault at offset `0xdda2c: ldrge   sl, [r4, #8]!"`.  So I
suspect two things why this could happen:

(1) The instruction at `dda2c` fails to do the post-increment correctly
    for *whatever* reason.

(2) Something along the execution path corrupts the stackslot, where
    `r4` is saved, in such a way that it *exactly* masks it with `0xf`.
    Everything else on the stack looks fine, so this is sort of very
    unlikely to be honest.

This only happens on a very specific device: The Nexus 9 is the only
device that was ever shipped with the Tegra K1 T132.  I suspect an issue
in the binary translation layer of `armv7` instruction set to the internal
micro-ops of the CPU.  I tried to stress test the instruction in
question (see https://github.com/lewurm/ldrinsntest), however I was not
able to trigger a crash.  So either, I'm missing some context in order
to trigger the bug or I'm on a completely wrong track.

That said, even if we could proof that it is indeed a hardware issue,
the workaround is also non-trivial (it would then either need a fix in
gcc or require a microcode update by the chip vendor).
```

And then we saw:
https://github.com/golang/go/issues/19809#issuecomment-290804472

Another hint that this device is buggy.

Bump bockbuild

[runtime] Change MonoDomain::refobject_hash to use MonoConcGHashTable.

This change massively improves reflection query scalability.

Given this benchmark and my 4 cores laptop:

```
const int T_C = 4;
var list = new List<Thread> ();
bool done = false;
int count = 0;
for (int i = 0; i < T_C; ++i) {
var t = new Thread(() => {
while (!done) {
typeof (Driver)
.GetMethods (
BindingFlags.Public | BindingFlags.NonPublic |
BindingFlags.Instance | BindingFlags.Static);
Interlocked.Increment (ref count);
}
});
t.Start ();
list.Add (t);
}

Thread.Sleep (5000);
done = true;
foreach (var t in list)
t.Join ();
Console.WriteLine ("Got {0} lists done", count);
```

This change makes us go from 166.725 queries/s to 5.140.821 queries/s. Or 30x faster.

[runtime] Add concurrent read variant of MonoGHashTable name MonoConcGHashTable.

Use the same design as MonoConcHashTable.

Aaaaand confusing name will confuse.

[mcs] Improves imported destructor detection. Fixes #54159

Bump bockbuild

Bump bockbuild to fix wrench build

Merge pull request #4904 from kumpera/more_opts

[runtime] Change jit_info_table_num_elements to O(1).

Merge pull request #4928 from kumpera/ptr_to_struct_intrinsic

[mini] Intrinsify Marshal.PtrToStructure<T>

Interp sdb (#4911)

* [interpreter] Add beginnings of debugger support.

Handle debugger support similarly to the JIT:
- Add a MINT_SDB_SEQ_POINT instruction to the debugger IR. Replace it with a MINT_BREAKPOINT instruction when a breakpoint is placed.
- Store the sequence point information in the existing sequence point data structures.

* [interp] Add support for debugger stacktraces.

* [interp] Add support for debugger frame info.

* [interp] Connect seq points inside the same basic block so simple single stepping works.

* [interp] Link sequence points in different basic blocks the same way the JIT does it.

* [interp] Allocate some bitsets from a mempool.

* [interp] Rewrite the patching in the transform pass to make it a bit easier to understand.

* [interp] Add interruption support for sdb.

* [interp] Add support for mkrefany,refanytype and refanyval.

* [interp] Fix debugger failures.

* Add support for setting byref variables.
* Fix thread interruption when the top frame is an interpreter frame.
* Allow stack walks when a thread is suspended while transforming a method.
* Implement method entry/exit events.
* Fix ldstr+dynamic methods.

* [interp] Implement support for Debugger.Break.

* [interp] Change the offsets in MonoJitInfo/seq points into byte based from short based to reduce the amount of special casing needed in other parts of the code.

* [interp] Factor out the push/pop lmf code into a pair of helper functions.

* [mixed] Fix mixed mode EH.

* [interp] Implement support for MONO_VERBOSE_METHOD. Improve sequence point placement.

* [interp] Implement support for SetIP ().

Pass the target thread as an additional argument to mono_interp_set_resume_state (). Check the resume state after single
stepping/breakpoints in the interpreter.

* [interp] Disable the usage of the STLOC_NP opcodes when running under the debugger, it doesn't work if the ip is changed when the execution is stopped.

* [interp] Fix async stack walks in the debugger when the thread is in native code called from interpreter code.

* [interp] Fix the disable interpreter build.

* [interp] Fix the calculation of native offsets in the line number info.

* [interp] Fix the build.

* [interp] Make CMD_STACK_FRAME_GET_THIS behave the same when called on managed-to-native frames, with the interpreter, frame->has_ctx is set.

* [interp] Fix typos, remove some dead code, use g_print instead of printf in a few places, move some interpreter stubs to interp-stubs.c.

[runtime] Fix a race in mono_class_get_virtual_methods () which happens when klass->method is NULL initially, but gets initialized later. Use the lowest bit of the iterator pointer to distinguish between the two cases. Fixes #56824. (#4933)

[runtime] Change jit_info_table_num_elements to O(1).

On very large apps such as XS, this looking can take half a second of CPU time when opening a big solution.

[Mono.Posix] Adding .Net Core 2.0 support. (#4774)

[w32socket] addrinfo_to_IPHostEntry_handles

- change addrinfo_to_IPHostEntry to use the handles version
- next commits will update the callers to use the _handles version directly

Re-bump bockbuild for libtiff URL change

[System] Use coop handles for System.Net.Dns.GetHostName_internal

Revert "Bump bockbuild", as it breaks the build.

This reverts commit 30f9a22f0869d727c663cc4ba2c283ce75dc73f2.

Merge pull request #4931 from kumpera/com-interop-fix

Fix cominterop bug

[mini]Add test for PtrToStruct<T>

Merge pull request #4840 from kumpera/unaligned-access

Another round of cleanups plus fix how the JIT emit unaligned access.

[runtime] Use coop handles for ves_icall_System_Threading_Mutex_OpenMutex_internal

- add mono_w32mutex_open for w32mutex-unix.c

[runtime] Use coop handles for ves_icall_System_Threading_Mutex_CreateMutex_internal

[runtime] Use coop handles for ves_icall_System_Threading_Events_OpenEvent_internal

[w32event] Use coop handles for System.Threading.NativeEventCalls.CreateEvent_internal

- Make a new mono_w32event_create_full on unix
- Convert the MonoString to utf8 (unix) or utf16 (win32) in the icall, then
  call an implementation func.
- transition to GC safe mode around the win32 api call on Windows

Bump bockbuild

[mcs] Codegen for optimized new with valuetype load and awaited arguments. Fixes #56081

[coop handles] Add mono_string_handle_pin_chars

[w32handle] Const-ify arg of mono_w32handle_namespace_search_handle

[remoting] Use coop handles for ves_icall_System_Runtime_Activation_ActivationServices_EnableProxyActivation

[remoting] Use coop handles for ves_icall_IsTransparentProxy

[arm] use r8 instead of r5 in icall trampoline, because r5 is used as IMT reg

[arm] respect iOS specific ABI for frame pointer in icall trampoline

[mini] Intrinsify Marshal.PtrToStructure<T>

[runtime] Add support for using COM types from corlib.

This can be reproduced by trying to load a type that implements a COM interface corlib ships.

[mini] Fix the llvmonlycheck test.

[aot] Avoid printing out AOT NOT FOUND messages for icall methods which have no body.

[arm] Do not use the ARM Run-time ABI in cross-compilers (#4914)

[remoting] refactor ves_icall_InternalExecute

Pull the FieldGetter and FieldSetter logic into separate functions

[runtime] Use coop handles for System.Runtime.Remoting.RemotingServices.GetVirtualMethod

Merge pull request #4926 from BrzVlad/fix-tls-v5

[arm] Don't use fast tls on v5 arm

Fix mono build with disabled sgen (#4915)

Build of the Mono failed with the Boehm GC only (without SGEN)
if it configured like this: --with-gc=included --withsgen=no

sysctl takes 3 name components (#4883)

Adapted from:

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=212259

https://github.com/mono/mono/pull/3480

https://svnweb.freebsd.org/ports?view=revision&revision=423356

Move <sys/types.h> higher for integer types (#4884)

Fix build on FreeBSD 8 and maybe others
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=198404#c6

[arm] Don't use fast tls on v5 arm

The thread coprocessor registers appear only in v7 specification.

Merge pull request #4923 from lambdageek/bug-56694

[appdomain] Don't leak MonoReflectionAssemblyHandles firing events

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

Merge pull request #4872 from kumpera/add_test_for_eloop

[runtime] Fix runtime behavior when dealing with missing symlinks

Merge pull request #4845 from lambdageek/dev-coop-delegates

[marshal] Use coop handles for delegates icalls

Bug #56499: unreadable /etc/localtime should not cause an exception (#4881)

* Bug #56499: unreadable /etc/localtime should not cause an exception

* Fix build

[sgen] Scan the handle stack for dangling ptrs during domain unload

There should be no handles pointing to objects in the domain that is unloading
in any thread's handle stack once mono_gc_clear_domain () is called.

[coop handles] Fix wrong #ifdef

[appdomain] Don't leak MonoReflectionAssemblyHandles in callbacks to managed

[tests] Regression test for #56694

When an AssemblyLoad event fires in a domain, a System.Reflection.Assembly
object is created in that domain for the assembly being loaded.  The bug left
this object alive after the domain was unloaded, which lead to a GC crash.

[runtime] Implement interior handles as a special case to simplify and speed up the more common handle code. (#4921)

[corlib] Fixes warning

[mcs] Parsing of nested unbound types. Fixes #55400

[corlib] Remove unused field

Improve error messages to help users diagnose problems when using custom mode (#4922)

* Improve error messages to help users diagnose problems when using the custom mode

* Update mkbundle.cs

[runtime] Convert a g_warning to a type load failure. (#4920)

Fix make dist.

Fix the CpBlk and InitBlk intrinsics.

Update test for correct behavior.

[runtime] Fix multiple issues around symlink handling.

After fixing my windows setup, I managed to properly test MS's behavior, which is a lot saner and it's the following:

Opening fails with ERROR_CANT_RESOLVE_FILENAME (1921).
Testing and Deleting works.

Fixing File.Delete required removing the lstat we do before deleting, which no longer serves any purpose.

Fixing File.Exists requires handling ELOOP when stat fails and we retry with lstat.

[System]: Don't call OnCompleted(this) twice in SocketAsyncEventArgs.FinishWrapperConnectSuccess(). (#4894)

[bcl] Fix compiling xunit-based tests

We moved System.Runtime.InteropServices.RuntimeInformation to a facade.

[runtime] Add specialized versions of mono_icall_handle_new () for internal/non-internal pointers. (#4918)

Fix full aot (#4919)

* [runtime] Fix the generation of invalid IL in the return marshalling of stringbuilders.

* [runtime] Disable some failing fullaot tests.

[runtime] Use coop handles for System.Delegate.AllocDelegateLike_internal

[marshal] Use coop handles for System.Delegate.GetVirtualMethod_internal