Merge pull request #3085 from lateralusX/jlorenss/win-x64-pinvoke-empty-struct

Fix for non GCC win x64 calling convention when passing empty structs over pinvoke.

Merge pull request #3139 from akoeplinger/dns-test

[System] Make ServicePointTest.DnsRefreshTimeout test more reliable

We are still seeing quite rare failures of this test:

```
HostEntry should result in the same IPHostEntry object.
  Expected: same as <System.Net.IPHostEntry>
  But was:  <System.Net.IPHostEntry>

+++++++++++++++++++
STACK TRACE:
at MonoTests.System.Net.ServicePointTest.DnsRefreshTimeout () [0x00045] in /Users/builder/jenkins/workspace/test-mono-mainline/label/osx-i386/mcs/class/System/Test/System.Net/ServicePointTest.cs:228
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /Users/builder/jenkins/workspace/test-mono-mainline/label/osx-i386/mcs/class/corlib/System.Reflection/MonoMethod.cs:309
```

Since the test is only checking if expiring of the cached DNS lookup works, we don't really need to use an external domain name for this as "localhost" works just fine as well and typically doesn't trigger a DNS query which makes the test more reliable against network issues.

@monojenkins merge

[mcs] Recognize indexer with accessors using different names. Fixes #41602

[System] Make ServicePointTest.DnsRefreshTimeout test more reliable

We are still seeing quite rare failures of this test:

```
HostEntry should result in the same IPHostEntry object.
  Expected: same as <System.Net.IPHostEntry>
  But was:  <System.Net.IPHostEntry>

+++++++++++++++++++
STACK TRACE:
at MonoTests.System.Net.ServicePointTest.DnsRefreshTimeout () [0x00045] in /Users/builder/jenkins/workspace/test-mono-mainline/label/osx-i386/mcs/class/System/Test/System.Net/ServicePointTest.cs:228
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /Users/builder/jenkins/workspace/test-mono-mainline/label/osx-i386/mcs/class/corlib/System.Reflection/MonoMethod.cs:309
```

Since the test is only checking if expiring of the cached DNS lookup works, we don't really need to use an external domain name for this
as "localhost" works just fine as well and typically doesn't trigger a DNS query which makes the test more reliable against network issues.

Merge pull request #3138 from akoeplinger/fix-win

[metadata] Fix Windows gcc build break

[mcs] Bump ikvm to fix assembly unification rules

[metadata] Fix Windows gcc build break

After 680fe21e6952ba6dc73ad89095b34ebb3a9bfc86 it failed with:

```
coree.c: In function '_CorExeMain':
coree.c:200:2: error: implicit declaration of function 'mono_runtime_run_main_checked' [-Werror=implicit-function-declaration]
  mono_runtime_run_main_checked (method, argc, argv, &error);
  ^
[...]
Makefile:2137: recipe for target 'libmonoruntime_la-coree.lo' failed
make[3]: *** [libmonoruntime_la-coree.lo] Error 1
```

[sgen] Scatterplot option for gcpauseviz.

Merge pull request #3135 from kumpera/enable_conc_gc_on_mainline_archs

[jenkins] Make the concurrent GC the default on mainline archtectures.

[jenkins] Make the concurrent GC the default on mainline archtectures.

Merge pull request #3090 from alexanderkyte/remove_deprecated_assertions

[mobile_static] Remove deprecated assertions

[corlib] Fix tests for watchOS. (#3131)

[llvm] Enable support for the atomic store intrinsics when running with llvmonly.

Merge pull request #3117 from alexrp/master

Fix a few socket test failures on Android

Merge pull request #3093 from lambdageek/dev/monoerror-exec

[runtime] Mark mono_runtime_{run,exec}_main external only.

[sockets] Correctly report that Linux supports double binding for UDP, even on old kernels.

[mcs] Improve betterness changes in b12385e9fdedd5b146d00a56687c886a647e1865

[System] Fixes compiler warning

[System] Make SocketAsyncTest.SendAsync give more helpful failures.

[build] Add llvm folder to SUBDIRS/DIST_SUBDIRS

This ensures all the right files are packaged in the tarball.

Merge pull request #3122 from akoeplinger/fix-41530

[System] Make TimerTest.EnabledInElapsed more reliable

Merge pull request #3120 from esdrubal/syscall_details

Fixes SendAsyncFile flaky failures

[runtime] Only print out the value of MONO_EXTRA_CONFIGURE_FLAGS when it is set.

Merge pull request #3118 from kumpera/fix_getname

[corlib] Handle user Assembly subclass and Assembly::GetName.

[System] Make TimerTest.EnabledInElapsed more reliable

We were seeing intermittent failures on devices like:

```
[FAIL] TimerTest.EnabledInElapsed :   #1 loss of events
  Expected: True
  But was:  False

  at MonoTests.System.Timers.TimerTest.EnabledInElapsed () <0x33b168 + 0x0014f> in <filename unknown>:0
  at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object , BindingFlags , System.Reflection.Binder , System.Object[] , System.Globalization.CultureInfo ) <0x66d400 + 0x000b7> in <filename unknown>:0
```

Since OS scheduling and GCs can influence the exact timing of the timer intervals sleeping for 200ms is likely just too short.
Instead of bumping the timeout, I decided to rewrite the test to not rely on Thread.Sleep ().
I also added another test which verifies setting AutoResetEvent=false stops the timer event from firing after the first one,
which was implicitly tested before by the other test.

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

Merge pull request #3114 from alexanderkyte/mono_conf_gc

[runtime] Add configuration for the default sgen gc mode

Merge pull request #3112 from akoeplinger/test-runner-nofile

[test-runner] Write stdout/stderr into StringBuilder instead of an intermediate file

More updates

[corlib] Handle user Assembly subclass and Assembly::GetName.

Remove UnprotectedGetName and replace with overriding of GetName.

dotnet throws NIE if you subclass Assembly and call GetName on it.
We hard crashed.

Fix make dist

It got broken when the versions.py file was moved around in d110f526706b7c4880ca916e2dfbab93138871cd.
Also add the new locations to EXTRA_DIST so they get distributed too.

Merge pull request #3119 from kumpera/fix-getname-and-shadowcopy

[corlib] Take shadow copying when computing CodeBase in Assembly::GetName.

Merge pull request #3082 from alexanderkyte/no_alloc_klass_error

[runtime] Set klass-error-based MonoError state without allocating

[mobile_status] Fix nullable type ambiguity that nunit-lite complains about

[mobile_static] Remove use of CollectionAssert, missing in nunit-lite

[mobile_static] Remove uses of deprecated Assertion.cs nunit helper.

Merge pull request #3110 from BrzVlad/fix-complex-conc-scan

[sgen] Fix concurrent scan of complex objects

[Socket] Check for SocketOptionLevel.Socket before checking for SocketOptionName.ReuseAddress.

Without doing this, the check would consider a valid option name like
IpTimeToLive to be the same as ReuseAddress, as they share the same enum
value.

Fixes SocketTest.TtlChange and SocketTest.UdpMulticasTimeToLive on Android and
other platforms that don't support ReuseAddress.

Merge pull request #3124 from mono/revert-2734-master

Revert "Enhance log tracing"

Revert "Enhance log tracing"

[runtime] Add environment variable to pass additional configure options

[runtime] Add new configure option for default sgen GC mode

[amd64] Fix an off-by-one in the dyn call code.

Llvm submodule (#3108)

* Move the versions.mk infrastructure to scripts/submodules so it can be used elsewhere too.

* Add a conditional submodule for llvm.

* Fix commit messages emitted by the bump- targets.

* Use a anonymous git url for llvm.

[runtime] Fixed interruptible wapi_sendfile.

Before closing a socket we try to interrupt and wait for all socket
pending system calls to return. For that we set the socket as non
blocking which makes the system calls to return with EAGAIN\EWOULDBLOCK.

This issue was caught by System.Net.Sockets.SocketTest.SendAsyncFile
that was intermittently failing, we were only able to reproduce it while
running multiple test suites in parallel. We now can say that it only failed when
the the socket was closed and the sendfile was waiting for the OS to
allocate enough resources to send the file.

This was fixed by changing wapi_sendfile to return on EAGAIN.

[System] Improved "Could not abort" exception.

Stack traces of blocked threads are now displayed on the exception.

The threads stacktraces are only stored when MONO_TESTS_IN_PROGRESS env
var is set to yes.

[runtime][mobile_static] Add flag for rgctx_fetch_trampolines and use it (#3088)

* [runtime] Add commandline option for rgctx fetch trampolines.

* [mobile_static] Increase the number of trampolines

Smaller adjustments based on PR feedback.

[runtime] Call mono_class_init () for classes in mono_class_is_subclass_of (). Fixes #41466.

[corlib] Take shadow copying when computing CodeBase in Assembly::GetName.

Merge pull request #3116 from alexrp/master

Some improvements to the profiler makefile

[sgen] Make gcpauseviz histograms more meaningful.

There were too many bins, because the x axis was not logarithmic, and
there were no meaningful labels.

[sgen] Fix gcpauseviz.

It didn't detect starts of concurrent collections anymore because of how
the binary protocol outputs it.

[profiler] Don't install the coverage suppression file if the profiler is disabled.

[profiler] Don't run the test suite when the profiler is disabled.

Also clean up the makefile logic surrounding tests a bit.

Merge pull request #3113 from akoeplinger/boehm-fixes

[ci] Turn off Boehm on Windows x64 gcc build

[offsets-tool] Generate offsets for SgenClientThreadInfo.

[sgen] Fix define argument

The code worked by accident because the function in which the define was expanded had an argument with the name desc.

[sgen] Fix race between complex descriptor allocation and complex object scanning

When we allocate the complex descriptor for a new type we might have the concurrent marker scan objects. While the concurrent marker scans the bitmap for a complex object, which resides in the complex_descriptor array, another thread might allocate a new complex descriptor, thus reallocate the complex_descriptor array, which would lead to corruption of the bitmap used by the concurrent marker.

We reuse the SgenArrayList which never reallocates, using instead a list of exponentially increasing arrays.

Only define Boehm variables in configure.ac when it is enabled

Fix the message that is printed in the configure summary at the end to contain only "sgen" when Boehm is disabled.

[ci] Turn off Boehm on Windows x64 gcc build

It was never supported there, turn it off to fix the build on Jenkins.

[amd64] Fix a loop variable reuse in a58c2ef804779be70832c6b0e092fbca1edaebf1.

Merge pull request #3100 from lambdageek/dev/monoerror-exns

[runtime] Pass MonoError to exception creation utilities

[sgen] Make sure we will scan the bits correctly for complex objects

[sgen] Add missing memory type description

Merge pull request #3104 from lambdageek/dev/monoerror-resolve_vcall

[llvm] Raise MonoError using mono_llvm_throw_exception

[jit] Fix the emission of Selector.GetHandle () implementation to avoid the creation of string objects, which could cause GCs don't work in a cross compiler. Fixes #41431.

[test-runner] Write stdout/stderr into StringBuilder instead of an intermediate file

We don't need to write out and then read back in the contents. Additionally, this will prevent
an unhandled exception we've seen on Jenkins due:

```
Unhandled Exception:
System.IO.IOException: Sharing violation on path /media/ssd/jenkins/workspace/test-mono-mainline-linux/label/debian-8-arm64/mono/tests/finalizer-wait.exe.stdout
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share, System.Int32 bufferSize, System.Boolean anonymous, System.IO.FileOptions options) <0x7f7eb4a740 + 0x00650> in <filename unknown>:0
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share, System.Int32 bufferSize, System.IO.FileOptions options, System.String msgPath, System.Boolean bFromProxy, System.Boolean useLongPath, System.Boolean checkHost) <0x7f7eb4a620 + 0x00053> in <filename unknown>:0
  at (wrapper remoting-invoke-with-check) System.IO.FileStream:.ctor (string,System.IO.FileMode,System.IO.FileAccess,System.IO.FileShare,int,System.IO.FileOptions,string,bool,bool,bool)
  at System.IO.StreamReader..ctor (System.String path, System.Text.Encoding encoding, System.Boolean detectEncodingFromByteOrderMarks, System.Int32 bufferSize, System.Boolean checkHost) <0x7f7e9b2170 + 0x000f3> in <filename unknown>:0
  at System.IO.StreamReader..ctor (System.String path, System.Text.Encoding encoding, System.Boolean detectEncodingFromByteOrderMarks, System.Int32 bufferSize) <0x7f7e9b2120 + 0x00037> in <filename unknown>:0
  at System.IO.StreamReader..ctor (System.String path, System.Boolean detectEncodingFromByteOrderMarks) <0x7f7e9b2040 + 0x0003b> in <filename unknown>:0
  at System.IO.StreamReader..ctor (System.String path) <0x7f7e9b2010 + 0x0001f> in <filename unknown>:0
  at (wrapper remoting-invoke-with-check) System.IO.StreamReader:.ctor (string)
  at System.IO.File.ReadAllText (System.String path) <0x7f7eb48c70 + 0x0003b> in <filename unknown>:0
  at TestRunner.DumpPseudoTrace (System.String filename) <0x7f7cfc6450 + 0x00023> in <filename unknown>:0
  at TestRunner.Main (System.String[] args) <0x7f8128b770 + 0x01ed3> in <filename unknown>:0
```

[test-runner] Print timeout value when test times out

This is useful when multiple timeouts are in place e.g. like in the runtime testsuite.

Merge pull request #3111 from directhex/sed-before-sign

Move call to sn.exe to *after* cil-stringreplacer.exe

Merge pull request #2734 from nealef/master

Enhance log tracing

Move call to sn.exe to *after* cil-stringreplacer.exe

This makes sure mscorlib.dll still has a valid signature (fixes nightly builds on Linux)

Merge pull request #3070 from kumpera/runtime-handles

Local handles initial drop

Merge pull request #3083 from alexrp/gc-pointless-memory-barrier

[sgen] Remove a pointless memory barrier from the middle of the managed allocator.

[System] Override SupportsHeaders in all WebResponse overrides

[amd64] Enable dyncall tests on amd64.

[amd64] Support stack args in dyn calls.

[amd64] Handle ArgValuetypeAddrInIReg in dyncalls.

[mini] Set pending MonoError in mono_vcall_trampoline (#3107)

Remove unused header.

[coop] New local handles implementation.

This is the initial code drop of a new handles implementation.

It differs from the previous one is a few ways:

- It keeps a separate stack of handles which is shared by all "arenas".
This eliminates malloc in the majority of the cases without the need for any tuning

- Custom gc scanning code for the handles stack.
The original implementation registered the chunks as a GC roots. Which is a terribly
slow operation as it takes the GC lock.

Instead of allocating a new handle arena for each icall, we compute a stack mark,
which is the position in the handles stack at icall entry. On exit, we simply restore that
position, which is just a couple of stores.

Since the stack is implemented using arraylets, addressed are stable in the face of
expansion.

Only the typed handles function/macros are in as having both typed and untyped handles
adds extra complexity for no clear value.

The macros only handle what was needed to port a sizeable chunk of locales.c.

This commit just sets the stage for the feature, there's quite some work left
before we can start using it, as it can be seen in the big TODO in handle.c

[sgen] Fix scan thread data job to set ops so it can do precise scanning.

[mono-error] Fix error_init macro to work with MonoError pointers.

[llvm] Propagate MonoError out of mini_resolve_imt_method

To the enclosing llvm jit icalls

Use %Y-%m-%d %H:%M:%S for Windows log message header

Merge pull request #3040 from xmcclure/debugger-step-recursive

Debugger stepping does not understand recursion (bug 38025)

[genproj] Update project files

[jit] Pass MonoError to resolve_vcall

[System] Flush few colliding changes

[runtime] Use new mono_error_set_for_class_failure where applicable.

We were seeing an issue where attempts to create the MonoClass for an
exception in a no-exec context was causing failure. No-exec will forbid
the constructor call. Our solution was to realize that there was no
need to allocate an exception, as MonoError could already encode the
errors.

[runtime] Make helper to set MonoError directly from MonoClass error

Correct typo

Correct for widechar on Windows

[mcs] Fixes NRE during error reporting

[System] Remove duplicate file entry

Several corrections thanks to Johan Lorensson

[WinForms] Fix build broken by 5d54803af7af4c78cfeac020e0e2046b1282af91

Correcty initialize image ref_count. Ensures image is correctly unloaded during appdomain unload. (#3084)

Merge pull request #3102 from alexrp/master

[System] Add back IWebProxyScript to the mobile profile.

[System] Add back IWebProxyScript to the mobile profile.

SWF - Fixed exiting modal runloop. (#2980)

Fixes a bug in exiting modal runloop that caused not really exiting it
(not calling XplatUI.SetModal(..., false).

It caused issues with menu, for example, if you first ran a modal
dialog (a wizard), and then you started the application.