Merge pull request #3705 from BrzVlad/fix-sgen-internal-alloc

[sgen] Fix block size computation for allocation size

Merge pull request #3697 from directhex/fix-make-dist-missing-basic.exe

When basic.exe isn't in DISTFILES, it gets excluded from `make dist`

Merge pull request #3711 from akoeplinger/fix-fullaot-tests

Fix two test failures in the FullAOT Jenkins job

[MonoError] Don't overwrite computed TypeLoadException message with empty string

Fixes [#44729](https://bugzilla.xamarin.com/show_bug.cgi?id=44729).

Fix two test failures in the FullAOT Jenkins job

Disable WebOperationsContextTest.Current like on monotouch, it relies on dynamic code generation.
The namedmutex-destroy-race.cs runtime test doesn't work on mobile_static since we don't have named Mutexes.

Add mono-package-runtime to EXTRA_DIST

The file was added in 15b6ef60e45bb978203b57da4b956aa1d1f86f3c

[corlib] Enable more RuntimeHelpers tests

Merge pull request #3693 from rolfbjarne/watchos-simplify-http-message-handler-selection

[System.Net.Http] Simplify default http message handler for watchOS since there's only one valid value.

Merge pull request #3704 from rolfbjarne/watchos-servicepointmanager

[System] Add back a few ServicePointManager properties that can be used by multiple networking stacks.

[sgen] Fix block size computation for allocation size

The block size for a slot size is the minimum block size that can fit two slots. Make sure the computation accounts for the fact that even though a slot size might fit twice in the block, it still has to be allocated at aligned addresses, so it can end up using a higher block size.

Fixes 44922.

Merge pull request #3686 from lambdageek/dev-format-printf

Add MONO_ATTR_FORMAT_PRINTF macro

Add EXTRA_DIST listing all the files from btls

Apparently there's no way to get CMake to just spit out its own uncached variables, so we can't do this programatically.

Remove useless BTLS_DEPS variable

[System] Add back a few ServicePointManager properties that can be used by multiple networking stacks.

Merge pull request #3695 from ludovic-henry/fix-thread-test

[tests] Fix p/invoke test on windows

[sdb] Avoid adding the first file name in the debug info to the list of file names if a method has no seq points. Fixes #44982. (#3696)

Merge pull request #3700 from alexrp/profiler-c9-fixes

[profiler] Fix a regression that caused all buffers to have a zero thread ID.

[profiler] Use macro constants instead of passing a bunch of bools.

[profiler] Update a stale comment.

[profiler] Add some asserts.

[profiler] Simplify send_log_unsafe ().

[profiler] Fix a regression that caused all buffers to have a zero thread ID.

This was introduced in 928b840dad3c9cc1c39c3fe5820bf8394ae9b66b.

[SRE] Remove more dead code.

[SRE] Remove more unused icall declarations.

[SRE] Remove more unused icalls.

[btls] Add missing icall definition for mono_btls_x509_revoked_get_revocation_date

[SRE] Small code cleanup.

[SRE] Remove unused icall from inside a DISABLE_REFLECTION_EMIT block.

Merge pull request #3698 from alexrp/profiler-c9-fixes

More C9 profiler fixes

[SRE] Delay the creation of the MonoClass for dynamic types until CreateType () is called. Remove some icalls. (#3687)

[tests] Fix thread-native-exit test on windows

[metadata]: Unconditionally enable ves_icall_Mono_Btls_Provider_IsSupported().

The MonoBtlsProvider.IsSupported() icalls ves_icall_Mono_Btls_Provider_IsSupported()
needs to be available whether BTLS is available or not because it is used at
runtime to decide whether or not to enable BTLS.

(cherry picked from commit 7b2a96fbd372ad74c2595b473c00ac7e1ed9ca67)

[profiler] Fix BYTE_SIZE macro conflict on OS X.

[profiler] Fix PROF_TLS_FREE () on pthread TLS systems.

[profiler] Fix the build on Android (missing fcntl.h include).

Oops, actually correctly revert that commit.

(cherry picked from commit 04379d5d73838d12f772ac99cae4360dbd5d8c7a)

[System]: Temporarily switch the default TLS provider back to "legacy".

Also revert the Jenkins commit in c462f4e6032cd70d692461f09b295671d380eef8.

(cherry picked from commit cd1b0b8edf43ee61dc87ebc7edbc44f7d56204b5)

When basic.exe isn't in DISTFILES, it gets excluded from `make dist`

[Mono.Debugger.Soft] Move TypeLoadClass types to new file to improve testing

[System.Net.Http] Simplify default http message handler for watchOS since there's only one valid value.

[ci] Temporarily switch back to legacy TLS provider

[Mono.Debugger.Soft] Tweak ClassLocalReflection test

Merge pull request #3691 from ntherning/enable-tests-in-pr-builds-on-windows

Enable tests in Windows PR builds

[System*] Throw a PlatformNotSupported exception when using the managed networking stack on watchOS. (#3683)

* [System.Data] System.Data.SqlClient is not supported in watchOS.

System.Data.SqlClient is not supported in watchOS, because it depends on a
working managed networking stack, which we don't have on watchOS.

* [System] Throw PlatformNotSupportedExceptions in the networking stack for watchOS.

* [System] Don't reference/build Mono.Security.dll when building for watchOS.

This means we can't build Mono.Data.Tds.dll either, since it uses
Mono.Security.dll extensively. In any case Mono.Data.Tds.dll is useless
without a working networking stack, and since it's a Mono-only dll (i.e. not
part of netstandard or facades), just remove the entire assembly from the
watchOS profile.

System.Security.dll uses Mono.Security.dll as well, but the required classes
are available inside mscorlib.dll (as internal classes), so just open up
mscorlib internals to System.Security.dll.

* [System] Update tests after API change for watchOS.

* [System.Data] Update tests after API change in watchOS.

* Revert "Fix build break from 81fb20737f810def6dde88d87a1b2f23cdb736c3"

This reverts commit f4475ff01358f8490e7cd0878e8b39bed2a0def7.

* Fix missing trailing newline in monotouch_watch_System.dll.sources

It breaks the build on Linux if the newline is missing.

Enable tests in Windows PR builds

Also run the test suite in Windows PR builds. We disable all the test suites
in Windows PR builds that are known to fail.

[System] Server property of HttpWebResponse should never be null

[btls] Fix i386 build on Linux by setting -arch on Darwin only

The -arch option is an Apple extension, it caused failures on i386 Linux: "unrecognized command line option '-arch'"

Merge pull request #3689 from alexrp/profiler-c9-fixes

Profiler fixes for Cycle 9 (#2)

[profiler] Fix GC finalize object end event not getting invoked.

Revert "[domain] Report domain unloads to the profiler when finalizing the domain."

This reverts commit c8b20290808563f89d29b22d2fe35eb906cfe725.

This caused ordering problems with app domain events: https://bugzilla.xamarin.com/show_bug.cgi?id=44552

Conflicts:

mono/metadata/gc.c

Merge pull request #3688 from alexrp/profiler-c9-fixes

Profiler fixes for Cycle 9

[profiler] Bump profiler version to v1.1.

[profiler] Don't emit enter/leave events in onlycoverage mode.

[profiler] Add a nocounters option.

[profiler] Address a couple of FIXMEs in the counters sampling code.

[profiler] Use eglib alloc/free functions everywhere.

[profiler] Warn if we can't detect the CPU count.

[profiler] Use the native thread wrapper functions to manage threads.

[utils/threads] Add and export a mono_native_thread_join () function.

[utils/threads] Export all native thread functions.

[profiler] Access in_shutdown atomically.

[profiler] Drop support for perf events on Linux.

[utils/lfa] Add a missing #include to fix the profiler build.

[profiler] Disable GC move events when allocation events are disabled.

They serve no purpose when allocation profiling is enabled, and just make log
files bigger than they have to be.

[profiler] Clean up and improve the helper thread code.

* Don't invoke undefined behavior if an fd is >= FD_SETSIZE.
* Allow multiple command sockets.
* Close the command socket(s) on shutdown.
* Properly run cleanup logic on shutdown.

[profiler] Process requests in method_jitted ().

[profiler] Remove the need to pass a MonoProfiler argument to some functions.

This is now available on the MonoProfilerThread structure.

[profiler] Fix the way the profiler frees hazardous data on shutdown.

[profiler] Replace a check with an assert in coverage_filter ().

[profiler] Add messages to most assertions.

[profiler] Remove the ign_res () hack.

[profiler] Clean up the #include mess in mono-profiler-log.c.

Half of these aren't even needed - get rid of those. Sort the rest.

[profiler] Remove mono_threads_attach_tools_thread () declaration.

We get this from mono-threads.h now.

[profiler] Remove some broken, commented-out code from register_method_local ().

[profiler] assert () -> g_assert ().

[profiler] Always enable the helper thread.

We need it in all cases now that we use it to periodically flush all buffers.
The only platforms it was disabled on was NaCl, which we haven't bothered to
support in a while anyway.

[profiler] Encapsulate some common logic in the ENTER_LOG/EXIT_LOG macros.

This also has the nice effect of reducing the amount of times we attempt to
take the reader lock. Now we take it and release it once for a single event.

[profiler] Add a missing deinit_thread () call to the helper thread.

[profiler] Fix several race conditions in accessing MonoProfilerThread.buffer.

Only in very few situations is it okay to access this field without first
taking the reader lock or exclusive lock.

Fixes some crashes observed in the profiler stress tests.

[profiler] Don't flush empty log buffers.

[profiler] Add an extra assert.

[sgen] Always report object moves on the GC thread.

This avoids exposing the SGen worker threads through the profiler APIs. Doing
so had two main problems:

* It's easy for a user of the profiler API to accidentally mess with the state
  of the worker thread in all sorts of ways, which could break the GC.
* It's been a long-standing guarantee that any profiler API callback will be
  invoked on a thread that is, at the very least, attached to the runtime via
  the mono-threads infrastructure. This is not the case for SGen workers.

This fixes a deadlock in the log profiler when using major=marksweep-conc.

[profiler] Rename decode.c to mprof-report.c.

[profiler] Rename proflog.{c,h} to mono-profiler-log.{c,h} for consistency with other profilers.

[profiler] Remove utils.c/utils.h, move the code to proflog.c/decode.c, and clean it up.

* Drop support for fast/null timers.
* Use Mono runtime functions for getting the thread ID and allocating buffers.

[acceptance-tests] Improve the profiler-stress test runner.

* Emit an NUnit test report file.
* Set a timeout of 6 hours per test.
* Do a thread dump for timed-out tests.
* Don't print stdout/stderr for passing tests.

[ci] Bump the timeout of the profiler stress test suite to 24 hours.

[sgen] Remove bogus mono_profiler_events assert.

This variable can be changed at any time by any thread. It makes no sense to
assert that it has a particular flag.

[utils] Move MONO_DEPRECATED to mono-publib.h and make it imply MONO_API + MONO_RT_EXTERNAL_ONLY.

Also make it work with MSVC.

[profiler] Remove the long-obsolete hsmode option.

Also refactor the heap walk code a bit.

[profiler] Document and fix the onlycoverage option.

[profiler] Remove the countersonly option.

It was never documented and is no longer functional.

[man] Update mprof-report.1 to indicate the correct default maxframes value.

[profiler] Use the signal shutdown logic for the sampling thread for more cases.

[profiler] Remove some indirection in setting up GC events.

* mono_gc_enable_events: This did nothing for SGen and we called it
  unconditionally for Boehm. So no point in having it around.
* mono_gc_enable_alloc_events: The performance difference between checking
  a static alloc_events variable versus checking mono_profiler_events is
  marginal at best and doesn't justify the indirection, especially now that
  we support managed allocators while profiling.

[profiler] Lift profiler limitation in mono_class_get_allocation_ftn ().

This has not actually been necessary since allocation reporting was moved into
the GC implementations.

[profiler] Don't leak counter data structures on shutdown.

[profiler] Simplify counters initialization.

[profiler] Don't emit a perf counters sample event if none are registered.

[profiler] Fix the code that synthesizes thread end events.

It could previously fail to process a newly allocated buffer.