[acceptance-tests] Write list of CoreCLR tests into file and pass it to test-runner

This shaved about 30 seconds off the time until the test-runner started on my machine compared to
passing the tests in the command line directly (we have *lots* of CoreCLR tests).

[test-runner] Add a new --input-file option for reading the test list from a file

[test-runner] Fix bug in commandline option parsing

i+i doesn't make sense, it should be i+1 (the next argument after the current one).

[tests] Add coreclr GC stress tests

[tests] Try to fix finalizer-wait

[tests] Fix Math.Pow precision bug on Android

[profiler] Disable 'time went backwards' debug message.

While this would have indicated a bug in the past, it's perfectly normal now
due to how we synthesize TYPE_JIT events to ensure sane event ordering.

[runtime] Use Unwind_Backtrace only on platforms which support it.

[llvmonly] Add support for direct calls to icalls.

[System/TLS]: Don't crash when we're called without user-supplied certificate validator.

[acceptance-tests] Move CoreCLR submodule reference to Mono fork

Oops, there is no README.md

Oops, forgot to add to the top-level Makefile for mobile.

[test-runner] Filter out invalid XML chars before writing to XML

The test-runner crashed sometimes when running the CoreCLR testsuite with the following exception:
"System.ArgumentException: '', hexadecimal value 0x0F, is an invalid character." when trying to
emit the NUnit xml.

The stacktrace of one of the failing tests contained the character 0x0F which is invalid in XML,
we now filter out those invalid chars.

[Mono.Security.Providers.OldTls]: New assembly.

This provides a 'MonoTlsProvider' for Mono's existing TLS implementation.

It's simply a public frontend for the internal 'LegacySslStream' class.

[System]: Make internals visible to Mono.Security.Providers.OldTls.

[sgen] Fix marking of cards on 64bit

[sgen] Fix call to `InterlockedCompareExchange()`.

This fixes random crashes on `sgen-weakref-stress.exe`.

[linker] match type pattern with nested types

 - this way the following xml will match all the types in the assembly
   MyAssembly

    <linker>
        <assembly fullname="MyAssembly">
            <namespace type="*" />
        </assembly>
    </linker>

   before this change it will only match the types defined in main
   module and none of the nested types

[acceptance-tests] Append a suffix to the CoreCLR IL tests

Some of the tests have both a test.il and test.cs in the same folder, which means we would
generate an .exe with the same name and overwriting one of them.

This is avoided by appending a _il suffix to the IL tests.

Merge pull request #2163 from xmcclure/image-audit

Mempool reference-auditing feature in checked builds

This introduces three macros, to be used in place of a simple pointer assignment when working with images or imagesets:

CHECKED_METADATA_WRITE_PTR - Use this to assign a pointer when it points from memory hosted by an image or imageset to another.
CHECKED_METADATA_WRITE_PTR_LOCAL - Use this to assign a pointer when it points from memory noted by an image or imageset to itself.
CHECKED_METADATA_WRITE_PTR_EXEMPT - Use this to assign a pointer in memory hosted by an image or imageset, but which the audit feature cannot correctly check at this time (this is the case when an image must contain a pointer to something that is not located in a mempool, such as a constant string). This is so that at least we know where these exceptions to the audits are.

In non-checked mode, each of these is a simple assignment. In checked mode, they walk the graph of references between images to make sure invariants related to ref_count are not violated.

[runtime] Normalize include style.

Merge pull request #2175 from xmcclure/pre-image-audit

Mempool reference-auditing feature in checked builds (prep work only)

Simplify function interface for rework of image.c hashes, based on PR comments

[runtime] Fixed cross-compile LLVM build.

[test-runner] Make sure to close resources hold by Process

[Process] Use output/error stream instead of naked handle

This is to make sure the handle is closed, whenever we close the streams

[test-runner] Make the --testsuite-name option required

Defaulting to a value doesn't make sense since the NUnit xml would be overwritten if multiple tests used the same default.

[runtime] Added --with-cross-offsets configure option for including an explicit AOT cross offsets file.

This option will define MONO_OFFSETS_FILE which will be used by the rest of the code to include the right file when looking for cross offsets.

[mcs] Declared fields cannot have unmanaged check done before all types are defined. Fixes #35269

[llvmonly] Fix EH when llvm is disabled.

[llvmonly] Fix an assertion for methods with many arguments.

[llvmonly] Fix the computation of the generic context used to initialize shared methods, previously we would be using the context of the class of the receiver, but that wouldn't work when its a non-generic class inheriting from a generic class. Use the context of the method which was used to lookup the shared method in the first place instead.

[sgen] Fix binary protocol limit default.

`size_t` is unsigned, so using `-1` by default will always impose a
limit, and make binary protocol file names numbered, which is not
usually what you want or expect.

[Process] Close process in/out/err handle on Process dispose

This would lead to leaked handle, and that would be visible when running
the test suite on Jenkins, with a "Too many open files" exception.

That also reverts the behavior to before 789808b507fe16941a31acc43ffc6bc8e0cdae75

Rework “image hashes” in image.c to segregate by-path and by-name lookups into two hashes

This is part of the continued cleanup work for the mempool reference auditing feature. Separating the hashes prevents redundant work when iterating over hashes, something the reference auditor does a lot. External interface to image.c still behaves the same.

Continued cleanup in preparation for mempool reference auditing feature

- Fixes to comments and the mempool rework, based on pull request comments
- Additional comments overall

[llvm] Fix aot support in non-llvmonly mode.

[llvmonly] Avoid disabling llvm for methods with pinvoke signatures in llvmonly mode.

[llvm] Register the llvmonly icalls even if llvm is not enabled.

Fix the non-llvm build.

[tests] Add timeout before failing

[tests] Increase timeout before failing

[mcs] Support auto-property with private setter with member definition from external assembly. Fixes #35236

[tests] Increase timeout before failing

[tests] Add --testsuite-name to avoid overwritting test result XML files

[tests] Use test-runner to run unhandled-exception tests

Add another missing file.

Add a missing file.

[llvm] Move the EH support code into mini-exceptions.c, and the c++ functions used by it into a new llvm-runtime.cpp file so llvmonly code works in a runtime which doesn't have llvm enabled.

[llvm] Fix support for asmonly+llvmonly by emitting the bitcode into the file given by the llvm-outfile aot option.

Merge pull request #2166 from ermshiperete/FontHeight

[WinForms] Check for null font Height

[profiler] Create a static version of libmono-profiler-log which doesn't link against eglib, to prevent duplicate symbols errors when its linked with the runtime. Fixes part of #29745.

[arm] Add a comment for a hack.

[arm64] Implement mono_threads_core_begin_async_resume () for arm64.

Merge pull request #2171 from lambdageek/dev/fix-marshal

[coop] One more pair of coop gc marshal fixes.

[sgen] Documentation about future-proofing the binary protocol.

We want to use the binary protocol to measure GC pause times in the
benchmarking suite.  To make that easier and be able to go back and
(re-)benchmark older versions, we'd like the binary protocol format to
be backwards compatible.

Add a new Moono_GetStackTraces method to Thread which can be used to obtain stack traces for all running threads.

[coop] One more pair of coop gc marshal fixes.

Missed a pair of
`mono_threads_reset_blocking_start`/`mono_threads_reset_blocking_end`
calls in 991e3ef3e47f0ccf9981a791f7da680b4ee9c4c6

Remove a few more 'using System.Reflection's.

[Mono.Security]: Use the new 'NoReflectionHelper' instead of reflection in 'MonoTlsProviderFactory'.

[sgen] Add some bridge documentation.

Merge pull request #2170 from alexanderkyte/mini_llvm_archfix

[runtime] Fix support for amd64 in mini-llvm-cpp.cpp

[runtime] Fix support for amd64 in mini-llvm-cpp.cpp

[System]: Add internal 'Mono.Net.Security.NoReflectionHelper' class to avoid reflection use.

Remove 'using System.Reflection'.

[Mono.Security]: Access System.dll's internals directly instead of using reflection.

[System]: Make internals visible to Mono.Security.

[test-runner] Fix a few more hardcoded test suite names

Missed those in 04bdf3b9332a4707d196b65deda1fdfba1b88a30

[test-runner] Wait synchronously for process exit

There was still a race for the last processes, where we could not
correctly wait for them to exit, before closing the StreamWriter for
stdout and stderr.

Merge pull request #2168 from BillSeurer/llvm

[ppc] Activate llvm for power

[ppc] Activate llvm for power

Merge pull request #2165 from lambdageek/dev/marshal-fix

[coop] update native call marshalling to pass thread state transition functions a stack ptr arg.

[socket] Use syscall abort mechanism properly

The crash only occurs on Posix platforms.

The assertion error in `mono_threads_begin_global_suspend`
"pending_suspends != 0" arised because of a unproper use of the syscall
abort mechanism.

The signal would have to be send out of a global suspend context, but
in case the call to `mono_threads_pthread_kill` would fail in
`mono_threads_core_abort_syscall`, then we would add a pending
operation, thus increasing the `pending_suspends` counter. But as we
wouldn't call for `mono_threads_wait_pending_operations`, we would never
have a chance to decrement this counter. So next time we would try to
call `mono_threads_begin_global_suspend` we would hit this assertion
error.

The fix is simply to call the abort from inside a global suspend
context, and making sure to wait on pending operations afterwards.

[socket] Enable Shutdown on socket disposal only on specific platforms

This breaks Linux, because a shutdown will cause a recv/recvfrom on the
remote host to return without an error.

This would trigger in System.Net.Sockets.SocketTest.ConnectedProperty.

[msvc] Manually update .defs, don't use update-defs Make target

Revert "[msvc] Update the DLL export definition files."

This reverts commit 90cb172eb8c160f46781d189d3baf25f06496b1d.

[msvc] Update the DLL export definition files.

[WinForms] Check for null font Height

DrawButtonText was throwing ArgumentException when
button.Font.Height was invalid.

[coop] update native call marshalling to pass thread state transition functions a stack ptr arg.

mono_threads_prepare_blocking, mono_threads_finish_blocking,
mono_threads_reset_blocking_start, mono_threads_reset_blocking_end all
take a stack pointer argument.  Update the native and managed wrappers
to pass it to them.

[runtime] Add a check for PE images to mono_method_body_get_object().

Fix the build.

[runtime] Move the MONO_ENV_OPTIONS parsing code to driver.c

Export mono_image_load_cli_header/load_metadata from image.c

Merge pull request #2162 from alexrp/master

[profiler] Disable the hack to attempt to get code start/size info.

[corlib] Add non-overlapping buffer MemoryCopy implementation

Cleanup in preparation for mempool reference auditing feature in checked builds

- Rework in mempool.c: Have USE_MALLOC_FOR_MEMPOOLS work by tweaking existing rather than using an entire different path. Remove unused mempool field “rest”, stop assigning fields “pos” and “end” where unused. Removed unused function mono_mempool_empty. Fix off-by-one bug (>= vs >) in mono_mempool_contains_addr.
- Fix misc bug spotted by Rodrigo in aot-runtime.c: image of MonoGenericParam was always being set to corlib instead of proper image
- Add comments and replace inline constants with named constants in various places

[profiler] Disable the hack to attempt to get code start/size info.

This is causing crashes by poking into runtime internals, and since we no
longer need this for anything critical, disable it.

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

Merge pull request #2158 from akoeplinger/fix-winforms-tests

[WinForms] Update tests to match recent changes to Control Foreground+Background

Merge pull request #2154 from BrennanConroy/bug_30018

More robust get_address_info

[threads] Print more information in case of error in global suspend

Merge pull request #2149 from directhex/absolute-path-for-xcopy

[msvc] Use fully qualified path for xcopy

[WinForms] Update tests to match recent changes to Control Foreground+Background

The colors were updated in https://github.com/mono/mono/commit/f412b5529e560bca594e0c29a47f32dcfb252820,
but the tests weren't. This fixes them.

[Mono.Security]: Mark 'HttpsClientStream' as obsolete.

[System]: Remove some dead internal code.

Merge pull request #2122 from lambdageek/dev/bug-26384

In Type.FullName, escape ,+&@*[]\ with blackslash. Closes #26384

Merge pull request #2155 from akoeplinger/bug-34334-pr

[interpreter] Relax check for forcing compilation for built-in structs

[System]: Use MonoTlsProvider API in WebConnection.

[System]: Use MonoTlsProvider API in FtpWebRequest and SmtpClient.