Merge pull request #1870 from saper/langinfo_h

[eglib] Prefer <langinfo.h> to <localcharset.h>

[mcs] Reduced statement expressions can still be used as statements. Fixes #33142

[jit] Avoid taking the domain lock in mono_icall_get_wrapper_full (), do most work outside the loader lock.

[runtime] Remove unused mono_delegate_section variable.

[runtime] Change a few g_assert (foo != NULL) to g_assert (foo).

[coop] Make type_initialization_lock () a blocking lock, since some critical sections protected by it can block. Reorganize mono_runtime_class_init_full (), avoid putting the majority of the function inside an if.

[coop] Make threadpool->active_threads_lock a blocking lock, since some critical sections protected by it can block.

Merge pull request #1981 from akoeplinger/ilasm

[ilasm] Add stubs for the "legacy library" and "auto" keywords in .assembly

[ilasm] Add stubs for the "auto" keyword in .assembly

Example IL: ".assembly extern mscorlib {auto}"

[ilasm] Add stubs for the "legacy library" keyword in .assembly

Example IL:

".assembly extern legacy library mscorlib {}"
".assembly legacy library test {}"

[bcl] Fix test failures in the System.ServiceModel tests after 9ffb681d06858ed5474320ca2e7a82ccf2ce8be3.

Merge pull request #1979 from kumpera/conc_fix

[utils] Change mono_conc_hashtable write operations to rely on external locking.

Merge pull request #1980 from akoeplinger/fix-monoevent

[corlib] Implement MonoEvent Module property

[corlib] Implement MonoEvent Module property

Add tests for Event-, Method- and ConstructorInfo to verify accessing .Module works (the latter two already worked before this change).

[utils] Change mono_conc_hashtable write operations to rely on external locking.

The original design was to pass a lock to the mono_conc_hashtable constructor and
then the write operations would do locking themselves.

That turned out to be a problem for two reasons:

First, it hidden the fact that locking was happening. Every other data structure in
the runtime rely on the caller to perform locking - thus that design ended up been
a lie.

Second, under coop, taking the lock became a problem as leak locks should be taken
in GC Safe mode and other locks should be taken under GC Unsafe mode. Taking a lock
in the wrong GC mode makes the suspend watchdog bark really loudly.

The runtime, funny enough has 2 usages of the conc hashtable, one with the image lock
and one with the domain lock - and each must be taken in a different GC mode. That
meant that either overcomplicating the design, or just removing locking from it.

This commit does that. It removes the mutex from mono_conc_hashtable and hoist locking
around the calls that used to do it.

[jit] Reorganize the code in mini.c to reduce the number of DISABLE_JIT blocks.

[jit] Remove some dead code.

[sdb] Document the locks protecting various data structures.

[sdb] Simplify the locking in set_breakpoint() a bit.

[sdb] Revert 6f41607cc3b9c34e54716efe35d22d31132545d9, simplify the code inside the domain lock instead.

[sdb] Protect MonoJitDomainInfo->seq_points by the loader lock instead of the domain dock, since accesses to it from sdb are not simple code paths.

[sdb] Make the locking simpler and finer grained.

[runtime] Initialize the class name cache using double checked locking.

[gsharedvt] Don't pass non-variable size gsharedvt types by ref. Fixes #32931.

[jit] Fix a crash by using mono_field_get_type () instead of accessing field->type directly. Fixes #32907.

[coop] Fix ves_icall_System_IO_MonoIO_GetFileSystemEntries.

We were calling mono_string_to_utf8 while in GC safe mode.

It's technically ok'ish to do it as it will probably be pinned by the called, but
we're better off by reducing the surface as much as we can.

[coop] Fix crasher during shutdown initialization.

Make it possible to suspend the main thread when starting the shutdown of the thread pool.

The problem is that cleanup can take an unbound amount of time.

[coop] Add a workaround in mono_marshal_get_thunk_invoke_wrapper.

This is a workaround until we figure out what to do with the embedding API.

We ensure we transition the executed managed code into unsafe mode.

Maybe the right call would be to assert if the caller is not already on it, but that's too soon of a decision to make.

[coop] Fix coop support for mono_method_get_unmanaged_thunk.

Ensure we create the wrapper in GC Unsafe mode, which is the default one for runtime code.

[checked-build] Annotate object.c with GC state asserts.

[coop] Add performance counters for frequent transitions.

[checked-build] Initial implementation of checked build asserts.

To enable checked builds pass --with-checked-build=yes to autogen/configure.

The current set of checked build macros verify the GC state on function entry.

[threads] Add function to stringify thread state names.

[coop] Make state-machine asserts print the tid as it's the same used by other parts of the runtime.

[arm] Add support for split reg-stack argument passing to the dyn call code. Fixes #33080.

Fix configure.ac to detect WatchOS and define TARGET_WATCHOS.

Merge pull request #1972 from esdrubal/proc_pidpath

[runtime] GetModuleFileNameEx

Merge pull request #1975 from alexrp/profiler-gch-bt

[profiler] Add GC handle events with backtraces.

[tests] Remove CWL from test catch clause as it can be confused with a real error [#32593]

[referencesource] Import System.Math

Add specific configure option to enable the WatchOS BCL libraries.

Build an initial version of System.Net.Http.dll for monotouch as part of the regular BCL build.

We have Facade assemblies that depend on System.Net.Http.dll to exist, so
we can't delay building it (at least one version of it).

Added support for WatchOS's armv7k to configure.ac.

Posix RTS test fixes and Android compatibility

Android's libc (bionic) defines the _GNU_SOURCE macro but fails to adhere
to the GNU sematics in (at least) one case - the strerror_r function.
The function follows XSI semantics instead of the GNU one which caused our
code to segfault on Android. This patch special-cases Android to use the XSI
semantics for strerror_r. This un-breaks some POSX real-time singals tests.

It also makes provisions for detecting whether the real-time signals are safe
to use on the current platform. They are safe for all platforms desktop Mono
runs on, but they're not safe on most Android platforms.

[corlib] Remove unused code

[profiler] Add GC handle events with backtraces.

[profiler] Remove CodeAnalyst profiler module.

This product has been discontinued by AMD and we don't even have support for
building this profiler module in the build system anyway; chances are pretty
good that bitrot has set in.

[jit] Add another suspend check to the JIT.

[jit] Avoid asserts during stack walks if the thread is not attached.

[runtime] Make sure Monitor.Enter () is always inlined since it consists only of a call to an icall.

[aot] Fix the calculate of the llvm code range for aot image with only 1 llvm method.

Un-revoke fix in BNC#144655 - mono-find-provides should ONLY work on GAC libraries, and any packages which break on this behaviour should be fixed - rather than generating utterly broken Provides.

(cherry picked from commit 1f4133a06f252ec1b78637dd91e49f50836cf570)

[runtime] Move the kevent() calls from KeventWatcher to the runtime, add support for interrupts.

Merge pull request #1973 from rolfbjarne/bug32947

[threadpool-ms-io] Remove unnecessary asserts. Fixes #32947.

[threadpool-ms-io] Remove unnecessary asserts. Fixes #32947.

Neither is_socket_async_callback nor is_async_read_handler need to
actually successfully load the classes they're comparing against;
if the classes can't be loaded, then the input class is obviously
not of that class.

This occurs when mscorlib is linked, the classes to compare against
might be linked away.

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

[runtime] GetModuleFileNameEx

GetModuleFileNameEx now returns a full path instead of the process name.
Fixes #32579

Merge pull request #1952 from esdrubal/proc_name

Fixes process names trimmed to 15 chars.

[coop] Handle blocking in parked threads correctly.

[runtime] Add a missing MONO_FINISH_BLOCKING to ves_icall_System_Net_Dns_GetHostByName_internal.

[jit] Emit the seq point for filter clauses after the op_get_exc instruction.

Revert "[runtime] Move the kevent() calls from KeventWatcher to the runtime, add support for interrupts."

This reverts commit 7f4428165809bed674697d482f76c7e9446bae4e.

Revert this as icalls don't marshal kevent structures correctly.

[runtime] Remove some debug printfs.

[coop] Fix a crash introduced by edb4d8f3faef5f5d0ad3fc639d58b4d490c82c94, some wrappers don't have wrapper infos.

[runtime] Move the kevent() calls from KeventWatcher to the runtime, add support for interrupts.

Fixes process names trimmed to 15 chars.

Process names were trimmed to 15 characters. proc_name was used to
retrieve the process name, by looking at its implementation we can see
that it returns a buffer which the size is MAXCOMLEN defined as 16.

When the proc_name result length is equal to 15 (MAXCOMLEN - 1) we now check if
proc_pidpath is able to return a longer process name.

Fixes #32539.

[coop] Fix MONO_SUSPEND_CHECK to use the right variable name.

[coop] Cue card now mentions that blocking has a suspend count.

[coop] Don't emit safepoints in the wrapper of the polling function.

Otherwise during a collection the wrapper would call itself resulting
in a stack overflow.

[coop] Switch from thread_state_init_from_sigctx to thread_state_init.

Suspend needs to work in the intermediate state where the mono-thread
is attached but not the managed thread. This happens in the startup
and cleanup paths and leads to crashes there.

The suspend initiator is now responsible to check if the thread is
attached or not. Which is easy, just check lmf/jit_tls.

[coop] Add thread_state_init that initializes the thread from the current stack state.

This new function works differently that others in that it always succeed, but might
provide incomplete data in some cases - like when the thread is not attached.

[coop] Attach mono-thread as early as possible.

This makes early suspend checks work. This makes it easier
to debug suspend errors by asserting on suspend fails.

[configure] Add checked build configure option.

[corlib] Handle Module in RuntimePropertyInfo. Fixes #32815

[mcs] Fix string interpolation parsing where inner expression contains { } tokens. Fixes #32832

[runtime] Remove an unused functions.

[runtime] Mark mono_monitor_try_enter_internal () as inline.

[runtime] Make synchronized methods use the Monitor.Enter(object,bool&) overload.

[runtime] Fix the coop build on posix platforms.

[arm] Implement support for gc safe points.

[runtime] Remove mono_g_hash_table_new (), use mono_g_hash_table_new_type () instead.

[runtime] Use a GHashTable instead of a MonoGHashTable in mono_class_from_name (), the hash table doesn't need to be GC aware.

[runtime] Remove the sgen_supported conditional from configure, we don't support architectures which don't have SGEN.

Merge pull request #1968 from LogosBible/uribuilder-no-hostname

Fix UriBuilder.ToString() when the host is empty.

Fix the build.

[jit] Add a suspect check into the middle of the JIT pipeline.

[runtime] Revert part of the previous commit to avoid the profiler crashing.

[runtime] Avoid passing gshared methods to clients of the embedding api.

[runtime] Fix missing method at link time

Add a conditional for fork and exec* functions.

They're not available on WatchOS.

[runtime] Avoid a memory overflow in Buffer.InternalBlockCopy () if the caller passes in a negative value by mistake.

[runtime] Remove some dead code.

[runtime] Fix memory corruption due to incorrect use of MONO_OBJECT_SETREF

MONO_OBJECT_SETREF would use mono_gc_wbarrier_set_field which would write over 8 bytes ( *(void**)field = value ), while we want to write over 1 byte here.

Add UriBuilder test for no hostname.

Uri.ShemeDelimiter should not be used when the host is empty.

Merge pull request #1966 from angeloc/master

Better handling of vasprintf checking

Better handling of vasprintf checking

This patch improves the handling of vasprint checking.
It switches the unneeded have_vasprintf variable with the
AC_CHECK_FUNCS defined ac_cv_func_vasprintf one.

Merge pull request #1965 from akoeplinger/fix-flaky-test

[System.ServiceModel] Fix flaky Bug652331_2 test

[System.ServiceModel] Fix flaky Bug652331_2 test

The test sometimes failed on Jenkins with the timeout assert. Running only this single test locally
with `make check TESTNAME=System.ServiceModel.Dispatcher.Bug652331Test.Bug652331_2` however reproduced
this almost every single time.

It seems that WCF takes a bit to initialize until it accepts connections, so putting a little sleep
fixed this for me locally and running it in a loop for half an hour didn't assert even once.

[runtime] Re-enable the soft debugger on x86 without presence of signals.

As of b76665b8485e45bea13e2694107117877f96c98b the debugger does not rely on signals anymore for stepping.

[sgen] Do not use Mach OS initialisation if using coop GC.