[wasm] Implement GC support. Embedder must provide main loop pumping function request_gc_cycle.

[mini] Add my test suite to the list of magic type assemblies.

[mini] Adjust some test suites to work under mobile.

[wasm] Disable test due to emcc bug.

[wasm] Fix test_0_conv_i under interp.

[mini] Fix bug categories.

[wasm] Add ff signature

[interp] Don't inline empty calls that take vt arguments.

[wasm] Disable tests that crashes the runtime for now

[wasm] Don't try mini_get_delegate_arg if the interpreter is in use.

[wasm] Add one more wasm_invoke entry.

[interp] Add some error handling to the interpreter

[wasm] Don't use capture_context fast path that requires function tramps.

[mini] Add MONO_ prefix to RETURN_ADDRESS macro.

Fix bug 57918:
Async Socket reads that need to perform multiple underlying read operations fail to return a correct total # of bytes read

Mark operations on `shutting_down` as known and accepted data races

[coop handles] Record backtrace when allocating handles when MONO_HANDLE_TRACK_OWNER is defined

A backtrace that leads to the allocation site is helpful for debugging.

[appdomain] Catch exceptions when invoking AppDomain.DoAssemblyResolve

Otherwise we may leak a coop handle for the requesting assembly, which may
lead to an assertion when we unload the current domain.

[test] Regression test for #58950

If a reflection only assembly resolve event handler throws an exception, catch
the exception in the runtime to ensure that no coop handles leak.

Implement stack overflow handling on Win64. Call _resetstkoflw to restore stack protection before calling handler, and then jump to handler via a cached MonoContext. (#5472)

[TSan] Interlock and unlock MonoJitStats (#5437)

[TSan] Interlock and unlock MonoJitStats

This is the first part of telling TSan what to do with `MonoJitStats`; I dealt with all the `gint32` values and partly touched the `gboolean` value (see `FIXME`). However, `MonoJitStats` also contains `gdouble` which can (1) probably be interlocked and (2) will be updated in a later step, together with the `FIXME`, as soon as an updated version of the atomic interface exists.

I did my best to distinguish between perf. critical parts and parts where `Interlocked* ()` seems to work perfectly fine. As before, if I misjudged anything, I would kindly ask to comment the affected operations, and I will update them accordingly.

More (technical) details can be found in the commit messages.

[System.Xaml] Fix deadlock on exception in XamlBackgroundReader

The ManualResetEvent wouldn't be signaled when an exception happens
in the reader loop so we'd deadlock in Read().

I'm still investigating where the exception comes from but this
should at least help unblocking CI.

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

Merge pull request #5504 from ntherning/wait-for-native-thread-to-die-in-Thread-Join

Wait for the native thread to die in Thread.Join() on Windows

Update CONTRIBUTING.md (#5542)

[F#] Reinstate _DebugFileExt as it's needed for mobile debugging. (#5534)

Also added a mechanism for users to override DebugType

Merge pull request #5428 from kumpera/wasm-support-p2

Mono Wasm support

[threads] Fix "[threads] Fix leaking threads: SGen Worker and Finalizer (#5284)" (#5532)

* Revert "[threads] Fix leaking threads: SGen Worker and Finalizer (#5284)"

The sgen-specific cleanup cannot be run before the sgen-agnostic cleanup

This reverts commit 7db0fb0c886f5157066e26c2e2ae2d39c338cf6b.

* [threads] Make sure we do not leak the finalizer and sgen threadpool threads

F# portable debugging (#5531)

* Switch F# builds over to portable pdb

* Add portable pdb metadata fix from https://github.com/Microsoft/visualfsharp/pull/3553

[Tests] Fix Mono.Data.Tds tests on iOS (#5501)

* [Tests] Fix Mono.Data.Tds tests on iOS

The modified tests failed on iOS following PR:
https://github.com/xamarin/xamarin-macios/pull/2580 that adds support
for Xharness to run the Mono.Data.Tds on Xamarin.iOS. With the changes
the tests run a expected.

* Close Listener when done.

[TSan] Tackle MonoPerfCounters (#5413)

[TSan] Tackle MonoPerfCounters

With `MonoPerfCounters` taking part in many races, I thought it's finally time to attend to it. If I am not mistaken, most (if not all) load / store operations can be interlocked. If I am wrong, I would kindly ask to quickly comment the respective perf. critial operations and I will change them to `Unlocked* ()`.

Furthermore, a few details:

* updated all fields to signed types as `Interlocked* ()` functions exist for these types already
* upgraded a few values to `64-bit` as other `64-bit` values are assigned to them
* fixed a few `FIXME`s in `predef_writable_update ()` that were connected to `Interlocked* ()` functions

Update man page

Shorter

Toggle the turbo button

Remove lingering bits from removed prj2make tool (#5526)

Bump Roslyn

[msvc] Update csproj files (#5524)

Use `Unlocked* ()` to mark racy counters (#5500)

[TSan] Mark racy counters in metadata.c

- using `gint32` instead of `int` should not be an issue
- `img_set_count` is interlocked by `image_sets_lock ()` - `UnlockedIncrement ()` is just used for completeness / documentation
- `++img_set_cache_(hit|miss)` is called quite often and is connected to caching: it seems to me as if interlocking these counters would be a bad idea (at least for cache hits)

[threads] Fix leaking threads: SGen Worker and Finalizer (#5284)

* Fix leaking threads: SGen Worker and Finalizer
- https://bugzilla.xamarin.com/show_bug.cgi?id=58317
- stop SGen workers before the finalizer stops
- clean up SGen workers with the Finalizer
- remove `mono_thread_join (GUINT_TO_POINTER (gc_thread->tid));` as we detach the Finalizer with `mono_threads_join_threads ()` within `mono_gc_cleanup ()`

* [fixup!] Move counting variable into the loop initialiser and add a `(gpointer)` cast

[TSan] Reorganise / clean up counters logic in mini-generic-sharing.c (#5502)

* Reorganise counters:
- use `Interlocked* ()` and `Unlocked* ()` to fix / mark (possible) races
- change the initialisation location and the visbility of some counters
- fix a few typos

* Extend the warning when re-registering counters

* also print counter address if we have a duplicate

Updates for CODEOWNERS

Update reference-assemblies (#5519)

Bump roslyn to 2.3.2

Merge pull request #5512 from rodrmoya/fix-mono-profiler-lib

[Mono.Profiler.Log] Initialize LogEvent's properties

[Mono.Profiler.Log] Initialize LogEvent's properties

Make sure all events have all fields initialized. This was missing setting
Buffer and Timestamp properties when creating the different LogEvent* subclasses.

[WinForms/Drawing] Remove some leftover test scripts (#5518)

They're no longer used.

[Mono.Security]: Remove some unused `Mono.Security.Interface` classes. (#5509)

* [Mono.Security]: Remove some unused `Mono.Security.Interface` classes.

* Mono.Security.Interface: removed `IBufferOffsetSize` and `BufferOffsetSize`
  (these were not general-purpose BOS APIs, but depend on the internal
  `SecretParameters` class).

* Mono.Security.Interface: removed `SecretParameters`, `SecureBuffer`,
  `IMonoTlsEventSink`, `TlsBuffer` and `TlsMultiBuffer`.

* Bump API snapshot submodule

[amd64] fix tailcall insn size (#5483)

commit
https://github.com/mono/mono/commit/9a634c1810aad46d30a674f3a97ab263dcd4272e
introduces a regression for
https://github.com/mono/coreclr/blob/mono/tests/src/JIT/Methodical/Invoke/25params/25param1c.il

```
Testing method of 25 parameters, all of int data type, tail.call
mono-sgen(46435,0x7fff9d34b3c0) malloc: *** error for object 0x101045400: incorrect checksum for freed object - object was probably modified after being freed.
*** set a breakpoint in malloc_error_break to debug
Process 46435 stopped
* thread #1, name = 'tid_307', queue = 'com.apple.main-thread', stop reason = breakpoint 1.1
    frame #0: 0x00007fff945c915f libsystem_malloc.dylib`malloc_error_break
libsystem_malloc.dylib`malloc_error_break:
->  0x7fff945c915f <+0>: pushq  %rbp
    0x7fff945c9160 <+1>: movq   %rsp, %rbp
    0x7fff945c9163 <+4>: nop
    0x7fff945c9164 <+5>: nopl   (%rax)
(lldb) mbt
* thread #1
  * frame #0: 0x00007fff945c915f libsystem_malloc.dylib`malloc_error_break
    frame #1: 0x00007fff945c5e81 libsystem_malloc.dylib`szone_error + 406
    frame #2: 0x00007fff945c7925 libsystem_malloc.dylib`small_free_list_remove_ptr_no_clear + 766
    frame #3: 0x00007fff945c7cb2 libsystem_malloc.dylib`free_small + 881
    frame #4: 0x00000001004d2110 mono-sgen`monoeg_g_free(ptr=0x0000000101083c00) at gmem.c:66
    frame #5: 0x0000000100007b64 mono-sgen`mono_codegen(cfg=0x000000010108c800) at mini.c:2300
    frame #6: 0x000000010000aedf mono-sgen`mini_method_compile(method=0x0000000100910310, opts=370239999, domain=0x000000010090ebd0, flags=JIT_FLAG_RUN_CCTORS, parts=0, aot_method_index=-1) at mini.c:3829
    frame #7: 0x000000010000edd0 mono-sgen`mono_jit_compile_method_inner(method=0x0000000100910310, target_domain=0x000000010090ebd0, opt=370239999, error=0x00007fff5fbfe048) at mini.c:4156
    frame #8: 0x000000010001459e mono-sgen`mono_jit_compile_method_with_opt(method=0x0000000100910310, opt=370239999, jit_only=0, error=0x00007fff5fbfe048) at mini-runtime.c:2127
    frame #9: 0x0000000100013e6d mono-sgen`mono_jit_compile_method(method=0x0000000100910310, error=0x00007fff5fbfe048) at mini-runtime.c:2173
    frame #10: 0x0000000100131eea mono-sgen`common_call_trampoline(regs=0x00007fff5fbfe128, code="H\x8bй\x01", m=0x0000000100910310, vt=0x0000000000000000, vtable_slot=0x0000000000000000, error=0x00007fff5fbfe048) at mini-trampolines.c:704
    frame #11: 0x00000001001313b7 mono-sgen`mono_magic_trampoline(regs=0x00007fff5fbfe128, code="H\x8bй\x01", arg=0x0000000100910310, tramp="����\b\x10\x03\x91") at mini-trampolines.c:835
```

turns out we smashed our code buffer.

Also, transform this comparison:

```
if (G_UNLIKELY (offset > (cfg->code_size - max_len - EXTRA_CODE_SPACE))) {

if (G_UNLIKELY ((offset + max_len + EXTRA_CODE_SPACE) > cfg->code_size)) {
```

we deal with unsigned values here, and if `max_len` is bigger then
`cfg->code_size`, we won't resize the buffer.

Interlock `inflated_signatures_size`, `memberref_sig_cache_size`, `methods_size` and `signatures_size` (#5489)

[TSan] Interlocking loader.c

`loader.c` contains four statistics counters and all of them are invovled in data races. This PR focuses on dealing with them:

- All of them can be `gint32` as they are registered as `MONO_COUNTER_INT` anyways.
- Since there is already a lot of locking involved in `loader.c` (including an approved usage of `Interlocked* ()` with with `mono_stats`), I think that `Interlocked* ()` can be used here instead of `Unlocked* ()`. If this assumption is wrong, I would kindly ask to correct me by commenting the affected rows and I will change it to `Unlocked* ()`.

[bcl] Use RuntimeHelpers.GetHashCode () for getting the hash code of … (#5511)

* [bcl] Use RuntimeHelpers.GetHashCode () for getting the hash code of the delegate target to avoid calling user defined hash code implementations which can throw exceptions. Fixes a regression introduced by a48bc439850869e565833d7fe6330c289beffe40.

* add test for #59235

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

* [fixup] RuntimeHelpers.GetHashCode handles null

[sdb] Don't set tls->context when interrupting a thread, it could race with the thread processing a breakpoint at the same time. Fixes #58612.

Add z14 support and extend z13 support

[utils] Fix inet_pton fallback.

Wait for the native thread to die in Thread.Join() on Windows

This is the behavior of .NET. After this patch the code on Mono for Windows
will make sure the underlying native thread of a managed thread has died
before Thread.Join() returns.

This PR builds on PR #5454 but makes it Windows specific. PR #5454 was
reverted since it caused crashes on some Unix platforms.

[ci] Add the coverage collection script.

More codeowners update

Use `Unlocked* ()` for operations on `joinable_thread_count`
- mark the (harmless) data race of the fastpath

Fix racy counter and unify the styles

Clarifying the usage of the --library option

I found that multiple libraries need to be specified in dependency order. That was not obvious from the man page. Make it explicit.

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

[ci] Retry flaky tests only on PRs (#5492)

So we can still see failures in master and release branches.

[wasm][sgen] Introduce DEFAULT_SWEEP_MODE, move DEFAULT_MAJOR to sgen-config.h and make wasm defaults serial.

[wasm] Wasm actually has inet_pton but configure doesn't detect it.

[wasm] Make sure we don't use the posix dl backend with wasm

[wasm] Disable posix threading backend and fix wasm's.

[wasm] Move RETURN_ADDRESS macros to mini.h, use them in mini-exceptions.h and add wasm version.

[runtime] Replace a mono_class_vtable with mono_class_vtable_full.

[wasm] Fix eglib issues with WASM.

Merge pull request #5486 from kumpera/fix_array_dispatch_perf

[mini] Handle magic interfaces in common_trampoline as if they were variant.

[mini] Remove never defined MONO_ARCH_STACK_GROWS_UP (#5491)

[mini] Handle magic interfaces in common_trampoline as if they were variant.

With the introduction of magic array interfaces, they must be handled as if they
were variant if they are being dispatched against an array.

This was not being done in common_trampoline and resulted in the IMT thunk never
getting rebuilt and getting decent perf.

With this patch, dispatch to simple methods such as Array::Count over a magic interface
in a covariant case (such as IList<object> dispatch onto string[]) will accrue gvi's.

On a trivial benchmark of executing Count 1M times, perf goes from 760ms to 50ms.

Improve `class.c` for TSan instrumentation: (#5485)

- use `Unlocked* ()` for `class_gparam_count`, `classes_size` and `inflated_methods_size`
- dead code: `class_ext_size`
- fix a typo: "the the" -> "let the"

[bockbuild] Blacklist more llvm libraries/binaries. (#5488)

[ci] Reenable xunit tests on Windows (#5487)

Should be fixed now that https://github.com/mono/mono/pull/5477 was merged.

Merge pull request #5444 from hifi/fix-tds-inputoutput

[System] Fix TDS 7.0 prepared InputOutput parameter info

mkbundle: add console confirmations of actions (#5457)

Almost all the other "maker.Add()" are accompanied by console messages, except these two. User might worry about why they are missing. (I did!)

Fixing another confusing typo

The file being added is the config of the last assembly read, not the runtime name.

update comment

xunit on windows doesn't like ln to xunit.execution.desktop.dll, provide a copy

[w32file] Add 'kbfuse' filesystem type to known drive types array.

It's the https://keybase.io desktop client filesystem

[socket] Reduce handle manipulations in native (#5378)

Merge pull request #5479 from marek-safar/cecil-bump

Bump cecil and linker

Bump API snapshot submodule

Add retrying of failed tests by default to reduce flaky tests (#5461)

Whenever a test fails we retry it five times and then decide the
overall result based on that. If three or more of the five retries
fail too then the overall result is FAIL, two or lower means PASS.

We're modifying nunitlite here so this only works for those tests
right now, we'd need to add it to the other test runners too.

[cominterop] Get function ptr from interface before switching to blocking

The cominterop_get_function_pointer icall has a wrapper which has a
trampoline.  When we enter the trampoline mono_marshal_emit_native_wrapper
already switched us to blocking.

[marshal] Don't use a wrapper for mono_marshal_set_last_error

This fixes assertions due to d0e103fcb4ed3f0ba867052f9b5a4ac71faa0226 in
cooperative GC mode.

mono_marshal_set_last_error is called from a P/Invoke wrapper just after the
pinvoked method returns before we transition the thread state back to running
from blocking.  If mono_marshal_set_last_error had a wrapper, we would invoke
the trampoline (mono_magic_trampoline) to JIT the wrapper while in blocking
mode, which would trigger the assertion.

[llvm] Add support for building with more recent versions of the mono llvm branch, in particular https://github.com/mono/llvm/tree/2017-07-06-5a1ffa96376e8a3ee895b782790591e5e7814ccf, which is based on llvm 5.0. (#5474)

Bump cecil and linker

[sdb] Fix support for async debugging in optimized mode, roslyn generates valuetype IAsyncStateMethod implementations. Fixes #58728. (#5476)

[bcl] Fix a few hardcoded socket ports in tests (#5471)

* [System] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

* [System.Runtime.Remoting] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

Additionally remove a copy of RemotingServicesTest.cs from corlib
which also exists in System.Runtime.Remoting in updated form.

* [System.ServiceModel] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

* [System.Net.Http] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

The additional "keep-alive" header in Send_Complete_NoContent() and
Send_Transfer_Encoding_Chunked() is because we were using the same
port for all tests before so other tests would clear the header.
This doesn't happen now that we use distinct test ports.

* [System.ServiceModel.Web] Fix a few hardcoded socket ports in tests

Those could've shown up as "address already in use" in CI.

[wasm] Hack around AC_CHECK_SIZEOF bug and don't hardcode -Os in CFLAGS

Revert "Merge pull request #5330 from alexanderkyte/dedup_mkbundle"

This reverts commit 9a287c04126d095e7371afee32632febd0dafd93, reversing
changes made to 4a79280b3bef8d5f15da9ddd2a2af3a03e194b03.

It breaks tons of tests.

Merge pull request #5330 from alexanderkyte/dedup_mkbundle

[runtime] Add Dedup Support to MkBundle

[Mono.Security]: Add 'MonoTlsProvider.SupportsCleanShutdown' and 'MonoTlsSettings.SendCloseNotify'. (#5465)

Only send close_notify when using BTLS and explicitly requested via 'MonoTlsSettings.SendCloseNotify'.

[runtime] Document mkbundle AOT options in man pages

[runtime] Fix tracking of dedup module in mkbundle

[runtime] Add mkbundle support for dedup

[runtime] Clean up temp mkbundle aot directory

[runtime] Fix mkbundle internationalization test results, build all

[runtime] Provide config to mkbundle

[runtime] Don't re-aot when running BCL tests

[runtime] Skip not supported tests with mkbundle