[coop] Fix the state transition for a pending suspend in reset blocking.

The problematic scenario is the following: [1]

thread A) [DO_BLOCKING] runnable -> blocking (0)                      -- i.e, do a pinvoke
thread B) [ASYNC_SUSPEND_REQUESTED] blocking -> blocking (+1)         -- thread B tries to suspend thread A
thread A) [ABORT_BLOCKING] blocking -> async_suspend_requested (0)    -- i.e, that pinvoke called into a reverse delegate and needs to go back to runnable state
thread A) [STATE_POLL] async_suspend_requested -> self_suspended (0)  -- thread A self suspend due to the request from B

If polling witness async suspend, it must notify the initiator of a successful suspend.
If the initiator witness a thread in blocking, it bumps the suspend count, assumes it suspended and don't wait for a notification.

This means that thread A would post to the suspend semaphore but thread B (the suspend initiator) will not wait on it.

Given the initiator won't wait in this case, it makes no sense for abort blocking to put the thread in async_suspend_requested.
The solution is to put it on self_suspend_requested, which won't trigger state_poll to notify.

[1] Read it as: thread) [TRANSITION NAME] old state -> new state (suspend count delta) -- explanation

[coop] Improve logging and add a pair of suspend related due asserts.

Add asserts to both mono_threads_begin_global_suspend and mono_threads_end_global_suspend to ensure
balanced usage of the suspend semaphore.

The suspend semaphore must end a suspend operation at zero, meaning there must an equal number of
post and wait calls.

This ensure the initiator can't jump the gun when waiting for pending suspends by consuming a
post from a previous suspend/resume op.

[coop] Fix debug logging.

[runtime] Remove the usage of the GC aware hash table from the tasklets code.

[sgen] Properly synchronize `allocated_blocks`.

We were missing a memory barrier here.  This might fix the current
intermittent crashes on wrench.

Fix IL_SEQPOINT processing; Add names to trampolines

Merge pull request #2015 from alexrp/profiler-roots-v2

[runtime] Add root type and human-readable description for all GC roots.

[runtime] Throw an exception instead of asserting when trying to call remoting methods linked away.

[bcl] Disable some tests which rely on fp behavior on mobile. (#33510).

[Facades] Fix parallel builds for non-general profiles

Joao reported that the facades build broke with 9022df44211c434c95c5e2d1f4e5f5e3979f3dd0
as it tried to compile monodroid assemblies and this attempt to compile System.IO.Pipes
which in turn needs a type not presented on Android: SafePipeHandle

This should fix that

[ppdb] Fix support for Assembly.LoadFrom(). Fixes #33550.

Merge pull request #2012 from joelmartinez/mdoc-memberremoval-fix

[mdoc] Fixes an issue with member deletion.

[mdoc] Fixes an issue with member and type deletion.

Specifically, when updating classic and unified assemblies, members
that were completely removed from one or both assemblies were not
being removed from the XML. This patch updates mdoc's `DeleteMember`
method to properly handle deletion in all scenarios (normal, classic, unified).

Of course, deletion will still not happen if one of two things is true:

- `--delete` is *not* passed into the `mdoc update` call and the member node
  contains existing documentation.
- `--preserve` *is* passed into the `mdoc update` invocation.

In both of those cases, the member will not be deleted, and a message stating
as such will be written as a warning to the output.

Merge pull request #2019 from directhex/powerpc-fixes

Add move_f_to_i4 and move_i4_to_f to PowerPC

[Facades] Use Zoltan's parallel build infrastructure

[runtime] Add root type and human-readable description for all GC roots.

Reverted too long to run test.

1abf5baf16a8051091a3c30e11118651c334aa6c added a tests that was making
jenkins to timeout.

[threadpool-io] Reduce the value of poll_fds_size on socket removal

Add move_f_to_i4 and move_i4_to_f to PowerPC

[mcs] Remove not implemented from ambiguous static type lookup. Fixes #33527

Merge pull request #2016 from tpokorra/patch-1

fixing typo appropiate in WebHeaderCollection.cs

fixing typo appropiate in WebHeaderCollection.cs

as mentioned by FastCode in IRC

Revert "[llvm] Disable support for nested clauses again, it still causes test failures like #32583."

This reverts commit d48d8e93e033f5850d2f6fdb86b1f6c35be789d7.

Revert this as it breaks the build.

[llvm] Disable support for nested clauses again, it still causes test failures like #32583.

[threadpool-io] Remove sparse-array for poll backend

The sparse-array approach led to better insert/remove performance, by allowing a O(1) access, but led to memory consumption issue. The kernel would, for example, have to parse the whole array, even if it's mostly empty, and that can be an issue for large arrays.

Remove incorrect warnings

[mcs] Don't report error for warnings which are not intended to be compiler warnings. Fixes #33487

[system] WebUtility from reference sources

[runtime] Removed duplicated code

[mcs] 'future' option is now 'experimental' - clarify in man page and help text.

Revert "Merge pull request #2008 from dlech/CustomAttributeData"

This reverts commit 2775839f66faf67803efd96af7aa075888a1e09b, reversing
changes made to 9cf3500204c70d0cf69584a2c47c2c1591f198b0.

Merge pull request #1936 from esdrubal/DotNetRelativeOrAbsolute

[System] UriKind.RelativeOrAbsolute workaround.

Merge pull request #2008 from dlech/CustomAttributeData

Make CustomAttributeData.AttributeType virtual

Updated list of available MONO_DEBUG options.

Make CustomAttributeData.AttributeType virtual

According to https://msdn.microsoft.com/en-us/library/system.reflection.customattributedata.attributetype%28v=vs.110%29.aspx, System.Reflection.CustomAttributeData.AttributeType is a virtual property.

Merge pull request #2005 from BrzVlad/feature-concurrent-work

Concurrent sgen improvements

[sgen] Add missing write barriers for the concurrent collector

[sgen] Major allocation should not happen while we sweep

[sgen] Enable cementing for concurrent collector

[sgen] Remove the finishing nursery collection within the concurrent collector

[sgen] Clear nursery at gc if we are doing concurrent checks

[sgen] Fix scanning of invalid entries within arrays

Merge pull request #2007 from madewokherd/imagelock

Fix an unlock without a corresponding lock.

Fix an unlock without a corresponding lock.

8eeb2cef moved the lock into an if statement.

This commit is licensed as MIT/X11.

[runtime] Remove some dead code.

[sgen] Flush after writing the protocol entry

Merge pull request #2004 from esdrubal/singlestacktrace

[corlib] Changes StackTrace.ToString to use Exception format.

[corlib] Changes StackTrace.ToString to use Exception format.

StackTrace.ToString format was different from Exception.ToString.
StackTrace.ToString had an identation of 3 spaces instead of 2, outputed
"line " before the line number which Exception.ToString does not, and it
did not print offsets.

This change allows StackTrace.ToString output to be used by the mono-symbolicate tool.

Test mono-symbolicate with StackTrace.ToString.

We were testing mono-symbolicate tool with Exception.Tostring stack
traces but now we also test that the tool works with StackTrace.ToString.

Merge pull request #2000 from BogdanovKirill/master

Bug fix: reading of infinite web response can lead to connection loss

Remove stray ifdef

Merge pull request #2003 from esdrubal/seq_test_fix2

[runtime] Fix test_op_il_seq_point in amd64.

Target test-local in mono-symbolicate Makefile.

mono-symbolicate tests were not called by recursive test due to not having the
target test-local in its Makefile.

[runtime] Fix test_op_il_seq_point in amd64.

In amd64 when possible 32bit instructions and registers are usedi
instead of 64bit ones.

Using MONO_DEBUG=single-imm-size avoids 32bit optimizations thus
maintaining the native code size between compilations.

[runtime] Added MONO_DEBUG=single-imm-size.

In amd64 when i8const are use with values within the first 32 bits
instructions and registers used can be optimized to their 32 bit
versions.

Setting MONO_DEBUG=single-imm-size variable disables such optimizations
and guarantee that the the generated native code size is independent
of the i8const values.

[mcs] Bump ikvm to support up to 4GB output files

[gsharedvt] Disable the box+brtrue optimization for gsharedvt types. Fixes #33324.

[MonoPosixHelper] Fix compilation for Android.

The Android NDK decalres the `struct timeval` struct in <sys/time.h>.
Update sys-socket.c to #include <sys/time.h> to fix compilation
with the Android NDK.

Fix the build.

[x86] Fix fullaot support.

[coop] Fix the public coop api.

[coop] Blacklist two more wrappers when generating safepoints.

A GC Safepoint require the thread to be in GC unsafe mode to run properly.

Both mono_threads_finish_blocking and mono_threads_reset_blocking_start are called
in GC Safe mode, which would cause them to assert in checked-build.

mono_threads_finish_blocking is called after a pinvoke finished, which means the thread will
still be in blocking mode.

mono_threads_reset_blocking_start is called by reverse pinvoke wrappers before calling managed
code, so it will be in blocking mode too.

This looks like something checked-build should catch by setting the polling variable more frequently.

[JIT] Prefer gdb over lldb on OSX as the later still produces useless output on my system.

Merge pull request #1993 from steffen-kiess/posix-sockets-1

Add socket-related functions and data structures to Mono.Posix

Merge pull request #2002 from mono/revert-1991-seq_test_fix

Revert "[runtime] Fix test_op_il_seq_point in amd64."

Revert "[runtime] Fix test_op_il_seq_point in amd64."

Merge pull request #2001 from esdrubal/get_module_filename

Fixes get_module_filename.

Fixes get_module_filename.

Merge pull request #1991 from esdrubal/seq_test_fix

[runtime] Fix test_op_il_seq_point in amd64.

[coop] Initial drop of the supporting public API.

[corlib] Fix the mobile test suite.

[threadpool-io] Fixed the Windows backend to call the correct polling function.

[sgen] Don't access LOS object size directly because it has to be masked.

We use the lowest bit in the LOS object size to denote whether the object is
marked/pinned, so we can't directly use the size.  Use the getter function
instead.

[sdb] Disallow nested invokes.

Merge pull request #1999 from rodrmoya/master

Fix format description for TYPE_CONTEXT

Fix format description for TYPE_CONTEXT

[llvm] Implement support for GC_SAFE_POINT.

[corlib] Fixes test not compiling with older NUnit.

[WebConnectionStream] Fix connection loss for >2GB reads

When HttpWebRequest is used to download infinite (ContentLength not set) response from web server, we can see that after reading about 2 GB (Int32.MaxValue) of data, totalRead variable can be equal to contentLength (which is int32.MaxValue). Then WebConnectionStream thinks that all data is received (see soft condition totalRead >= contentLength), but this is not correct.

This change is released under the MIT license.

Merge pull request #1998 from BillSeurer/master

Fix coding error with ! operator in two places in libtest.c

Fix coding error with ! operator in two places.

Merge pull request #1992 from BogdanovKirill/ChunkedReading

[ChunkStream] CPU and memory leak fix when writing and reading infinitely

Merge pull request #1996 from BillSeurer/master

[ppc] Add MonoContext and asynch callback support

[ppc] Add MonoContext and asynch callback support

[runtime] Remove the is_mb_open field from MonoMethod, it doesn't seem to be needed, inflated MethodBuilders are represented by MethodOnTypeBuilderInst objects.

[sdb] Remove the code which stopped single stepping when exiting a runtime invoke, it was only enabled on ios/android, and it doesn't seem to be needed anymore.

[threadpool-io] Remove unecessary operation change

These 2 operations are going to be treated the same in managed, so the change is useless

[threadpool-io] Add log tracing

[threadpool-io] Rework backend API

The main change is the fd will not be removed from the backend when there is no event to poll, but only when we close the socket. This also reduce the number of functions of the backend.

[threadpool-io] Move the states table to the selector thread exclusively

This greatly simplify the model of the states as we do not need to worry about modifications in different threads.

[threadpool-io] Disable AIO by default

[coop] Add some TODOs to the stress test suite.

Add socket-related functions and data structures to Mono.Posix

This commit adds those socket-related functions which do not require the
sockaddr_* structures or the msghdr structure.

Merge pull request #1989 from esdrubal/local_tz_from_link

Local tz id from symbolic link

[runtime] Fix test_op_il_seq_point in amd64.

In amd64 when possible 32bit instructions and registers are usedi
instead of 64bit ones.

Using MONO_DEBUG=single-imm-size avoids 32bit optimizations thus
maintaining the native code size between compilations.

[runtime] Added MONO_DEBUG=single-imm-size.

In amd64 when i8const are use with values within the first 32 bits
instructions and registers used can be optimized to their 32 bit
versions.

Setting MONO_DEBUG=single-imm-size variable disables such optimizations
and guarantee that the the generated native code size is independent
of the i8const values.

[runtime] Fix native backtraces with lldb by closing stdin.

Fix the build.

[runtime] Implement support for dynamic methods in stack traces in a different way: instead of constructing the stack trace eagerly when it contains a dynamic method, save the list of dynamic methods into the Exception object so they are kept alive.

[runtime] Remove eager exception stack construction when it contains dynamic method. Fixes #33064

We used to construct exception stack traces as string at throw site (for dynamic methods) but
that does not work correctly because exception stack trace can be decomposed via
new StackTrace (exception) or merged via ExceptionDispatchInfo both of them need the exception
stack in raw form and not as a string.

Further the format used by local stack builder was not same as format used by
System.Diagnostics.StackTrace.

The orignal reason for the specialization was that dynamic method can be collected
before stack is rendered which leads to stack frame to be reported as

at <unknown method> instead of
at (wrapper dynamic-method)

That should be addressed by gc-link between dynamic method and exception or
stack-frame.