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.

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.

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

[runtime] Build stripper and use with mkbundle

Preliminary linker support ran into dependency-finding bug
with the monolinker.exe binary. Fix postponed until later.

[runtime] Add target to mkbundle all tests before CI

[runtime] Move testing dll into profile

[runtime] Fix in-tree building for mkbundle

[runtime] Skip building System.Security, System.IdentityModel tests on testing_aot_full

[runtime] Use mkbundle in BCL tests

[runtime] Fix path resolving for mkbundle

[runtime] Add new dedup interface to mkbundle

[runtime] Fix mkbundle compilation on OSX

[runtime] Enable compiling aot with mkbundle

[runtime] Add aot arguments to mkbundle

Update Clang's ThreadSanitizer blacklist (#5467)

Bump api-snapshot

Call `g_utf8_to_utf16` with a proper argument (#5470)

* The sizes of `glong` and `gsize` differ on 64-bit Windows.
* In order to avoid uninitialized data in the high four bytes of the `bytes` argument to `mono_unicode_from_external`, call `g_utf8_to_utf16` with a `glong`, then copy the result.

Merge pull request #5469 from marek-safar/ilasm

[ilasm] Adds AggressiveInlining support

Revert "[runtime] Wait for the thread to really exit in Thread.Join (). (#5454)"

This reverts commit ec30afd9dcee60bea5ce4c221a9680a23449ab87.

It causes test failures on s390x.

Revert "[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. (#5408)"

This reverts commit f01f85481834e3d9cae2d6c757f9b694851c22aa.

It broke a bunch of tests with llvm.

[PEAPI] Don't add mscorlib reference when no type needs it

[ilasm] Adds noautoinherit option

[ilasm] Adds AggressiveInlining support

Remove leftover targets from Makefile.am (#5466)

win32getdeps and win32setup don't apply anymore.

[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. (#5408)

[runtime] Wait for the thread to really exit in Thread.Join (). (#5454)

[sgen] Move the mono_gchandle_ functions to sgen-mono.c for consistency, add sgen_gchandle () functions instead. (#5460)

[sdb] Make the IL in dtest-excfilter.il valid. (#5463)

[amd64] Make code patching smp safe for the case where we emit a mov + call_reg. Hopefully fixes #57959. (#5452)

[gc counters] avoid using Interlocked*64 functions as they are broken on some arches (#5451)

They are implemented via a global lock, which can lead to deadlooks in
some situations, e.g.

```
-0  0xb6c92e14 in sigsuspend () from /lib/arm-linux-gnueabi/libc.so.6
-1  0x0043f3b4 in suspend_signal_handler (_dummy=35, info=0xb4afe6d0, context=0xb4afe750) at mono-threads-posix-signals.c:177
-2  <signal handler called>
-3  0xb6de6bc4 in __pthread_mutex_unlock_usercnt () from /lib/arm-linux-gnueabi/libpthread.so.0
-4  0x00440f00 in InterlockedCompareExchange64 (dest=0x530700 <worker+64>, exch=0, comp=0) at atomic.c:578
-5  0x002f228c in InterlockedRead64 (src=0x530700 <worker+64>) at ../../mono/utils/atomic.h:398
-6  0x002f29a0 in COUNTER_READ () at threadpool-worker-default.c:194
-7  0x002f3320 in worker_park () at threadpool-worker-default.c:426
-8  0x002f3774 in worker_thread (unused=0x0) at threadpool-worker-default.c:488
-9  0x002e66b0 in start_wrapper_internal (start_info=0x0, stack_ptr=0xb4b00000) at threads.c:982
-10 0x002e6894 in start_wrapper (data=0x5a8f38) at threads.c:1044
-11 0xb6de31a8 in start_thread () from /lib/arm-linux-gnueabi/libpthread.so.0
-12 0xb6d3a1bc in ?? () from /lib/arm-linux-gnueabi/libc.so.6
Backtrace stopped: previous frame identical to this frame (corrupt stack?)

Thread 4 (Thread 0xb40ff440 (LWP 10198)):
-0  0xb6deb56c in __lll_lock_wait () from /lib/arm-linux-gnueabi/libpthread.so.0
-1  0xb6de584c in pthread_mutex_lock () from /lib/arm-linux-gnueabi/libpthread.so.0
-2  0x00440e6c in InterlockedCompareExchange64 (dest=0x559868 <gc_stats+8>, exch=2855648, comp=2844091) at atomic.c:570
-3  0x0039bdcc in InterlockedAdd64 (dest=0x559868 <gc_stats+8>, add=11557) at ../../mono/utils/atomic.h:392
-4  0x003a1b50 in collect_nursery (reason=0x5083f4 "Nursery full", is_overflow=0, unpin_queue=0x0) at sgen-gc.c:1865
-5  0x003a3794 in sgen_perform_collection (requested_size=4096, generation_to_collect=0, reason=0x5083f4 "Nursery full", wait_to_finish=0, stw=1) at sgen-gc.c:2539
-6  0x003a35f0 in sgen_ensure_free_space (size=4096, generation=0) at sgen-gc.c:2505
-7  0x00389db0 in sgen_alloc_obj_nolock (vtable=0xb4d174d0, size=64) at sgen-alloc.c:256
-8  0x0036e330 in mono_gc_alloc_vector (vtable=0xb4d174d0, size=64, max_length=12) at sgen-mono.c:1743
-9  0xb6c16a38 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
```

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

Merge pull request #5459 from HinTak/patch-5

More informative g_error message

Merge pull request #5458 from HinTak/patch-4

remove redundant duplicate word 'not'

More informative g_error message

Encountered this message while running the new mkbundle simple-cross functionality introduced recently. "Function not supported" is seen on both Mac OS X and Ubuntu. Will possibly file a separate bug since HAVE_MKDTEMP should be true... but it should be "Function mkdtemp not supported" in any case.

remove redundant duplicate word 'not'

Found a consecutive 'not' pair while searching the code for somethng else.

[runtime] Use the AC_FUNC_STRERROR_R() autoconf macro to distinguish between the two possible signatures for strerror_r (). (#5453)

Merge pull request #5455 from HinTak/patch-1

Fixing confusing typos

Fixing confusing typos

Fixing confusing typos - should be self-explanatory.

[jit] Assert that the trampoline code is called in gc unsafe mode, throw exceptions by returning the exception to the assembly trampoline code and throwing it from there. (#5409)

[amd64] silence compiler warning (#5435)

Silence a GCC warning about param_regs and return_regs not being
used.

[System] Disable SocketTest.SendAsyncFile for now (#5447)

It's one of the flaky tests that we have had for a very long time.

There's work in progress fixing it as part of
https://github.com/mono/mono/pull/5345 and https://bugzilla.xamarin.com/show_bug.cgi?id=43172
but I think we should disable it in the meantime for clean CI.

[configure.ac] Don't include the .build number in runtime-corlib version (#5449)

It's causing unnecessary complexity because we need to create a new
monolite when bumping the .build number in a major.minor.build
version like from 5.2.0 -> 5.2.1

corlib version bumps are very rare in those cases and even then
you can just increment the MONO_CORLIB_COUNTER variable.

[profiler] Correct the description of the 'legacy' option.

[System]: Cleanup debugging and use the same technique from the new web stack. (#5441)

* Cleanup debugging and use the same technique from the new web stack.

* The MARTIN_DEBUG conditional has been renamed into MONO_TLS_DEBUG.

* Moved the actual implementation into MNS.MonoTlsProviderFactory.

* We now use an environment variable called 'MONO_TLS_DEBUG' to
  enable debugging (so to get debugging, you need to both compile with
  -define:MONO_TLS_DEBUG and set that environment variable).

The new web stack has a conditional and environment variable called
'MARTIN_WEB_DEBUG'.

* Use [Conditional] instead of #if.

[msvc] Update csproj files (#5450)

[man] Cleanup mentions of monocov in mono.1 (#5446)

Coverage support was added to the log profiler in a5ab6c642e38cbb60cf0c4f63007652ebf567060
and the monocov profiler was removed with 16570265149730ec6a4760cc0fa34decc1a9d981.

Retrieve thread lazily and avoid special case for defer. We were trying to get thread when in defer mode even for EVENT_KIND_VM_DEATH and hitting assert since current thread was not available. (#5440)

Ensure exception out argument is initialized to NULL. (#5424)

[System] Fix TDS 7.0 prepared InputOutput parameter info

Fixes ParameterDirection.InputOutput type parameters not sending
the "output" modifier to server which creates a conflict resulting
in an exception server side.

Merge pull request #5420 from nealef/z13v5

Add support for z13 processor

[System]: Cleanup internal TLS APIs in preparation of adding renegotiation support. (#5426)

* [btls]: Bind new native APIs to support renegotiation.

* mono_btls_ssl_set_renegotiate_mode(), mono_btls_ssl_renegotiate_pending():
  New native methods.

* MonoBtlsSsl: Added SetRenegotiateMode() and RenegotiatePending().

* MonoBtlsSslRenegotiateMode: New internal enum.

* [System]: Cleanup internal Mono.Net.Security.MobileTlsContext APIs.

Use tuple `(int ret, bool wantMore)' return value instead of out variables.

* [System]: Check for pending exceptions in MobileAuthenticatedStream.

This should fix some issues where we're hanging instead of throwing
an exception if something went wrong.

Merge pull request #5438 from cherusker/cherusker-2017-08-24-complete-prototype

[libtest] Complete the function prototype

Merge pull request #5439 from alexrp/master

[profiler] Fix some stack imbalance issues due to missing method leave events.

[profiler] Add a missing method_leave event for the CEE_MONO_RETOBJ opcode.

[profiler] Add a missing method_tail_call event when the tailc optimization is enabled.

Complete the function prototype

[mcs] Update logic when checking for tuple element names when overriding inherited member

Revert "[runtime] Fix a warning."

This reverts commit f99617537ba31a046ede2753733d20c20a30e714.

Revert this as the original code was right, there is a problem with the defines.

Merge pull request #5415 from kumpera/wasm-support

WebAssembly port

[wasm] Don't define MONO_CROSS_COMPILE in the wasm configure.ac

[utils] Fix mono_valloc and mono_valloc_aligned malloc-based implementation.

[wasm] Add wasm stubs for object-offsets and sgen-archdep.

[wasm] Add wasm strings to mono-config.c

[wasm] Add comments explaining the magic strings in mono-dl-wasm.c

[readme] Document --enable-wasm

[wasm] Add threading, dl and hwcap support for wasm.