Merge pull request #1326 from BrzVlad/master

[sgen] Fix pointer access.

[runtime] Remove pre WinXP SP2 windows support code.

[runtime] Use the functionality in mono-proclib.c to get the current pid, remove duplicate code from the io-layer.

[runtime] Use the functionality in mono-proclib.c to enumerate processes, remove duplicate code from the io-layer.

Fix oops in DataContractJsonSerializer fix

[bcl] Use List instead of ArrayList () in a few places.

[bcl] Free strings allocated using Marshal.SecureStringToBSTR () using Marshal.ZeroFreeBSTR ().

Merge pull request #1330 from esdrubal/blockingcollection

Improved BlockingCollection.AddAny.

[threaddpool] Fix deadlock in thread pool with monitor thread (2nd attempt)

Because the monitor thread was not marked as background thread, remove_and_abort_threads (threads.c:2807) would mark it as needed to be waited to shutdown the runtime, and wait_for_tids (threads.c:2652) would then wait indefinitely for it to finish. The issue was that the monitor was waiting on the semaphore monitor_sem, and thus would never exit.

Thread 7 (Thread 0x1c13 of process 99776):
#0  0x00007fff86590a56 in ?? () from /usr/lib/system/libsystem_kernel.dylib
#1  0x00000001003cc03a in mono_sem_wait (sem=0x1004f7f50, alertable=0) at mono-semaphore.c:103
#2  0x00000001002cef41 in monitor_thread (unused=0x0) at threadpool.c:898
#3  0x00000001002cb78f in start_wrapper_internal (data=0x10500abf0) at threads.c:657
#4  0x00000001002cb4a1 in start_wrapper (data=0x10500abf0) at threads.c:704
#5  0x00000001003d61d4 in inner_start_thread (arg=0x7fff5fbfe500) at mono-threads-posix.c:84
#6  0x00007fff8e7c9899 in _pthread_body () from /usr/lib/system/libsystem_pthread.dylib
#7  0x00007fff8e7c972a in _pthread_start () from /usr/lib/system/libsystem_pthread.dylib
#8  0x00007fff8e7cdfc9 in thread_start () from /usr/lib/system/libsystem_pthread.dylib
#9  0x0000000000000000 in ?? ()

Thread 1 (Thread 0x1503 of process 99776):
#0  0x00007fff86594716 in ?? () from /usr/lib/system/libsystem_kernel.dylib
#1  0x00007fff8e7cbc3b in _pthread_cond_wait () from /usr/lib/system/libsystem_pthread.dylib
#2  0x000000010039cfaa in _wapi_handle_timedwait_signal_handle (handle=0xa55, timeout=0x0, alertable=1, poll=0) at handles.c:1595
#3  0x000000010039d03d in _wapi_handle_wait_signal_handle (handle=0xa55, alertable=1) at handles.c:1540
#4  0x00000001003b7ba3 in WaitForSingleObjectEx (handle=0xa55, timeout=4294967295, alertable=1) at wait.c:194
#5  0x00000001003b865d in WaitForMultipleObjectsEx (numobjects=1, handles=0x7fff5fbff1f8, waitall=1, timeout=4294967295, alertable=1) at wait.c:516
#6  0x00000001002c75df in wait_for_tids (wait=0x7fff5fbff1f8, timeout=4294967295) at threads.c:2658
#7  0x00000001002c70e5 in mono_thread_manage () at threads.c:2951
#8  0x00000001000d551b in mono_main (argc=6, argv=0x7fff5fbff9e8) at driver.c:2021
#9  0x0000000100001bd1 in mono_main_with_options (argc=6, argv=0x7fff5fbff9e8) at ./main.c:91
#10 0x00000001000017d3 in main (argc=6, argv=0x7fff5fbff9e8) at ./main.c:122

Thanks @akoeplinger for the help debugging

Refactored AddToAny and TakeFromAny to reuse code.

Improved BlockingCollection.AddAny. Fixes #22775.
AddAny no longer consumes OperationCanceledException.
AddAny no longer blocks while adding an item to the first collection when its full.

Test BlockCollection.AddAny. Covers #22775.

BlockingCollection.Add was adding one more item than it should.

Test that BlockingCollection.Add blocks when there is no more space.

[jit] Extract the ctor calling code from the NEWOBJ implementation into a separate function.

[jit] Clean up the jit failure handling code a bit, move the the bulk of the eror handling macros into functions, unify the cleanup code at the end of mono_method_to_ir ().

[bcl] Implement KnownTypes for DataContractJsonSerializer. Fixes #23058 and polymorphism support.
Also discarded old __type hint behavior in JsonSerializationReader that potentially allowed any loaded type to be instantiated.

[tests] Add tests for Bug #23058 and polymorphism/known types in DataContractJsonSerializer.

[threaddpool] Fix deadlock in thread pool with monitor thread

Because the monitor thread was not marked as background thread, remove_and_abort_threads (threads.c:2807) would mark it as needed to be waited to shutdown the runtime, and wait_for_tids (threads.c:2652) would then wait indefinitely for it to finish. The issue was that the monitor was waiting on the semaphore monitor_sem, and thus would never exit.

Thread 7 (Thread 0x1c13 of process 99776):
#0  0x00007fff86590a56 in ?? () from /usr/lib/system/libsystem_kernel.dylib
#1  0x00000001003cc03a in mono_sem_wait (sem=0x1004f7f50, alertable=0) at mono-semaphore.c:103
#2  0x00000001002cef41 in monitor_thread (unused=0x0) at threadpool.c:898
#3  0x00000001002cb78f in start_wrapper_internal (data=0x10500abf0) at threads.c:657
#4  0x00000001002cb4a1 in start_wrapper (data=0x10500abf0) at threads.c:704
#5  0x00000001003d61d4 in inner_start_thread (arg=0x7fff5fbfe500) at mono-threads-posix.c:84
#6  0x00007fff8e7c9899 in _pthread_body () from /usr/lib/system/libsystem_pthread.dylib
#7  0x00007fff8e7c972a in _pthread_start () from /usr/lib/system/libsystem_pthread.dylib
#8  0x00007fff8e7cdfc9 in thread_start () from /usr/lib/system/libsystem_pthread.dylib
#9  0x0000000000000000 in ?? ()

Thread 1 (Thread 0x1503 of process 99776):
#0  0x00007fff86594716 in ?? () from /usr/lib/system/libsystem_kernel.dylib
#1  0x00007fff8e7cbc3b in _pthread_cond_wait () from /usr/lib/system/libsystem_pthread.dylib
#2  0x000000010039cfaa in _wapi_handle_timedwait_signal_handle (handle=0xa55, timeout=0x0, alertable=1, poll=0) at handles.c:1595
#3  0x000000010039d03d in _wapi_handle_wait_signal_handle (handle=0xa55, alertable=1) at handles.c:1540
#4  0x00000001003b7ba3 in WaitForSingleObjectEx (handle=0xa55, timeout=4294967295, alertable=1) at wait.c:194
#5  0x00000001003b865d in WaitForMultipleObjectsEx (numobjects=1, handles=0x7fff5fbff1f8, waitall=1, timeout=4294967295, alertable=1) at wait.c:516
#6  0x00000001002c75df in wait_for_tids (wait=0x7fff5fbff1f8, timeout=4294967295) at threads.c:2658
#7  0x00000001002c70e5 in mono_thread_manage () at threads.c:2951
#8  0x00000001000d551b in mono_main (argc=6, argv=0x7fff5fbff9e8) at driver.c:2021
#9  0x0000000100001bd1 in mono_main_with_options (argc=6, argv=0x7fff5fbff9e8) at ./main.c:91
#10 0x00000001000017d3 in main (argc=6, argv=0x7fff5fbff9e8) at ./main.c:122

[jit] Reduce code duplication in the inlining code.

Merge pull request #1329 from esdrubal/httputility

HttpQSCollection.ToString now url encodes values.

Merge pull request #1327 from ermshiperete/Xamarin-23168

[MWF] Pass graphics when calculating button text and image

[System.Core/Android] Fix standalone unit tests.

Commit 5564dad1 broke the self-test (for testing on the developer
machine, not an Android target). Fixup the comment so that appropriate
data is grabbed and used for testing, and improve
AndroidTimeZones.GetDefaultTimeZoneName() so that the default timezone
ID can be specified via the __XA_OVERRIDE_TIMEZONE_ID__ environment
variable for testing.

Finally, demonstrate TimeZoneInfo.GetUtcOffset(DateTime) use so that I
can get better insight into the timezone data stored within the
database:

$ __XA_OVERRIDE_TIMEZONE_ID__=America/New_York ANDROID_ROOT=`pwd` ANDROID_DATA=`pwd` mono --debug tzi.exe --offset=1969-01-01
Using DateTime Offset: 1/1/1969 12:00:00 AM
DB type: System.AndroidTzData
name=Africa/Abidjan                           Africa/Abidjan                          From Offset: 00:00:00
name=Africa/Accra                             Africa/Accra                            From Offset: 00:00:00
name=Africa/Addis_Ababa                       Africa/Addis_Ababa                      From Offset: 03:00:00
...

HttpQSCollection.ToString now url encodes values. Fixes 22557.

Test HttpQSCollection.ToString url encoding. Covers #22557.

[runtime] Move the MonoJit/MonoJitInfoTable code to a separate file from domain.c.

[runtime] Factor out some duplicate code from the MonoJitInfo creation code.

Merge pull request #1328 from vkargov/master

[runtime] Take correct domain information from StackFrameInfo for each stack frame.

[MWF] Pass graphics when calculating button text and image

This fixes a problem with calculating the text width of a button
which caused the text to get truncated under certain circumstances
instead of being wrapped. This fixes Xamarin-23168.

Change-Id: Iac0d8a2720a99cf5433e05d5123a04c4b019e493

[runtime] Take correct domain information from StackFrameInfo for each stack frame.

Merge pull request #1179 from ludovic-henry/pr25-threadpool

[threadpool] Improve heuristic to approach optimal number of threads based on workload

Added missing files to projects csproj files.

[runtime] Fix another lost canary change

Kill timed-out tests when using the managed test runner.

[sgen] Fix pointer access.

Thanks to @martinpotter for finding the bug.

[bcl] Remove some verbose test output.

[runtime] Silence a warning that has not helped us fix anything.

[runtime] Fix canary oops

[runtime] Implement 'nursery-canaries' option for SGen.

[runtime] Convert String.InternalSetLength to an icall. Needed for canary support.
The change does not have a negative impact on StringBuilder performance; under variable-string-length microbenchmarks it seems to perform ~15-20% faster than the managed implementation.

[runtime] Implement nursery canaries.
Use the new MONO_GC_DEBUG option "nursery-canaries"  to suffix each nursery object with a canary (guard) word, which is checked on each minor collection. Can be used to detect/debug
heap corruption issues.

Adding canaries to LOS is also easy if wanted - not sure about major heap.

[system.xml] Make XmlSerializer work even when XmlTextReader has explicitly set its resolver

[system.xml] Add missing 4.0 property

[corlib] Notify parent task when child task uses NotOnFaulted. Fixes #23594

[arm] Increase size of tls_get opcode to 24.

[runtime] Fix allocation size calculation for strings in 64-bit builds.
Depending on alignment and packing, part of the string might be inside the MonoString struct boundaries. This e.g. shows up on 64-bit builds of mono where we were allocating 4 extra bytes per string.

[bcl] Use random ports for the HttpListener tests.

[jit] Implement 5dbf97e66057f8626c53fcea30f1988d6627e5a2 for calls made from NEWOBJ too. Fixes #23557.

[jit] Get rid of unused MONO_ARCH_THIS_AS_FIRST_ARG define.

[arm] Implement handler block guards.

[amd64] Simplify mono_arch_install_handler_block_guard ().

[arm] Increase the length of the tls_get opcode.

[mono-api-html] Add filter for new namespaces/types

- Reviewed by Sebastien

Fix a warning.

Fix the arm build.

[sgen] Reorganize sgen-archdep.h to remove unused defines.

[jit] Remove unused defines.

[amd64] Define MONO_ARCH_NOMAP32BIT on all non-linux platforms instead of defining it once for each platform.

Revert "Finish test changes started in 7ece665d16714db0979bde5df254231c5be24f10"

This reverts commit d171cdf5e281973c06f2751f4e33098f1a624a95.

Revert "Refresh HashAlgorithmTest to match changed NullStream behaviour in 8b9c60075f7b97fe88c3c588968977ff9bcf8c28"

This reverts commit 7ece665d16714db0979bde5df254231c5be24f10.

Revert "Missing Args checks."

This reverts commit 8b9c60075f7b97fe88c3c588968977ff9bcf8c28.

[jit] Define NOMAP32BIT on winx64.

[sgen] Fix concurrent mark with split nursery.

[jit] Remove the unused MONO_DISABLE_PENDING_EXCEPTIONS env var, the functionality it disable is not experimental anymore.

[x86] Remove the now unused push based parameter passing code.

Merge pull request #1306 from joelmartinez/docs-privateinterface

mdoc no longer documents private interfaces

[corlib] Fix UnicodeEncoding GetMaxByteCount and GetMaxCharCount to return the same values as .net.

[runtime] Add a missing call to mono_thread_notify_pending_exc_fn () to the thread abort code.

Fix some warnings by not including utils/mono-threads.h into threads-types.h.

[jit] Make each CASTCLASS_CACHE patch unique by generating a unique identifier from the method index and an per-method counter. Fixes #23478.

Finish test changes started in 7ece665d16714db0979bde5df254231c5be24f10

Add new cert-sync command to scripts/.gitignore

Refresh HashAlgorithmTest to match changed NullStream behaviour in 8b9c60075f7b97fe88c3c588968977ff9bcf8c28

[mcs] Undo bool loads normalization from byte arrays

[mcs] Correctly identify constant switch section when value match is not section start. Fixes #23475

[mcs] struct initializers are still experimental

Merge pull request #1318 from AlexanderBekrenev/baa-membershipuser

MembershipUser::UpdateUser method fix

MembershipUser::UpdateUser method fix

Fix build on pre-C99 compilers.

[sgen] Remove useless internal allocation

[runtime] Enable the new interrupt code on ios.

[sgen] Don't use callback for pinning cemented objects.

[sgen] Tag block pointers with whether the blocks contain references.

[sgen] Macro abstractions for pointer tagging.

[sgen] Shift, don't divide, just to make sure.

[sgen] Move SGen statistics out of gc-internal.h.

This introduced a dependency from sgen-conf.h to a lot of compilation units
in metadata, leading to long build times when experimenting with SGen.

[sgen] Don't unroll run-length scanning loop.

[sgen] Keep allocated blocks in array instead of linked list.

We were spending a lot of time in major_iterate_live_block_ranges() on
the lcscbench benchmark, mainly from following the linked block list.
This change makes it much faster.

[sgen] Move GRAY_OBJECT_ENQUEUE/DEQUEUE to sgen-gray.h.

[sgen] Move HEAVY_STAT macro to sgen-conf.h.

That's where we define HEAVY_STATISTICS, too.

[sgen] Scoping fix in a macro.

[sgen] Heavy statistics counters for scanned object descriptors.

[sgen] do-while loop instead of 1-unrolled while loop in bitmap scan.

[sgen] Added some comments.

[sgen] Simplify sgen_drain_gray_stack().

This seems to have accumulated code duplication by accident.

[sgen] Free gray queue sections after GC, not at start of next.

We trim the free list of the gray queue to a maximum number of
sections, but we did it at each collection start, so the extraneous
sections were kept around between collections.

[sgen] Enqueue objects with their descriptors.

At the point where we enqueue objects we have already fetched their
descriptors, and when we later scan them (after dequeuing) we need the
descriptor again, so we avoid cache misses by enqueuing them together.

[sgen] Pass descriptor, not vtable, to OBJ_COMPLEX_ARR_FOREACH_PTR.

Later on, when we enqueue objects with their GC descriptors, we won't
load the vtable before scanning (because we'll already have the
descriptor), so the scanning macros shouldn't take them as arguments.

[sgen] GrayQueueEntry struct instead of just char*.

This makes it easier to later enqueue the GC descriptor with the
object.

[sgen] Remove obsolete check in #ifdef.

That macro doesn't exist anymore.