[mono-api-html] Fix printing of Obsolete attributes.

[mono-api-info] Sort nested types.

So that info files can be compared textually to test for api equality.

[mono-api-info] Minor perf improvements.

[mono-api-info] Sort methods using full signature.

So that info files can be compared textually to test for api equality.

Merge pull request #2039 from simonhartmann/dns_refresh_timeout

[class libs] Implemented DnsRefreshTimeout for ServicePointManager

implemented DnsRefreshTimeout to ServicePointManager and adjusted behavior of ServicePoint.HostEntry property.

simplified some calls as requested

[mcs] Null operator on value type extension argument. Fixes #33754

Merge pull request #2032 from directhex/parallel_automake_is_hard

Parallel automake is hard

[mcs] Improve test from e97227a53dc1a598f4e38de0fbd401dbe9ca7ca5

[mcs] Lift result of binary operation on enum types in more cases. Fixes #33723

[mcs] Fixes a typo

[runtime] Another fix for fast TLS target checking to work with cross-compilation.

[mcs] Consider whole hierarchy of inherited type arguments inflated via base type. Fixes #33669

[mono-api-info] Order interfaces.

So that info files can be compared textually to test for api equality.

[Facades] Fix default profile parallelization

PROFILE_SUBDIRS is built serially, then PROFILE_PARALLEL_SUBDIRS in parallel. Since we want the latter for everything, only define the latter.

[Facades] Fix profile name error causing non-building

The parallel build infrastructure we're trying to leverage for Facades depends on the profile name when populating the parallel lists - the default profile is now named "net_4_x", so variables with "net_4_5" in them are not considered - causing the whole Facades/ directory to be skipped

[corlib] Make test reproducible

Merge pull request #2031 from esdrubal/uriworkaround

[System] UriKind.RelativeOrAbsolute workaround.

Fix a warning.

Merge pull request #2033 from tritao/zip-empty

[System.IO.Compression] Fixed handling of empty Zip archives in Update mode

Merge pull request #2035 from joelmartinez/mdoc-memberremoval-fix2

[mdoc] Follow up fix for member and type removal.

[mdoc] Follow up fix for member and type removal.
In supplement of the changes submitted in 09b97cba2e7c2d9c68830d55ff3d99c5f36eb439, there were
some additional edge cases that were not originally considered, and so this patch resolves these
additional issues. In particular, there was a bug in the `RemoveApiStyle` method that wasn't
checking the value of the `apistyle` attribute before removing it. This caused members to be
stripped of apistyle inadvertently.

There was an issue in the type removal code that caused
too many files to be removed. This patch adds additional checks to make sure that a file isn't
going to be removed if it contains members in the other style that shoulnd't be removed

Finally, there was an issue that arose after the aforementioned changes in the `OrderTypeAttributes`
method. It was somewhat counterintuitive, but it wasn't allowing the reordering of attributes
with the `.Remove` method (throwing an exception). Not entirely sure why this was happening,
but using the `.RemoveNamedItem` method worked around the issue.

[aot] Avoid an assert in mono_aot_jit_info_table_find () when the address is right after the end of the method. Fixes #32955.

[jit] Fix crashes in the dwarf writer if the backing field of an enum is not named 'value__'. Fixes #33591.

[threading] Fix race condition in the suspend account code.

Fix another broken test due to OSX oddities.

[runtime] Fixed fast TLS target checking to work with cross-compilation.

Fix the DefaultThreadCurrentCulture for real.

This test is very fragile as it depends on appdomain global state.
In this case the current thread culture.

The problem with CurrentCulture is it's behavior that once set, it can't
be unset, IOW, revert to follow DefaultThreadCurrentCulture.

This meant that test ordering played a huge deal here and it took a few
tries to learn what was going on.

The way to fix this is to run the test itself from a thread that we
can ensure that won't have CurrentCulture set. This includes TP threads
or new threads.

I opted to use a TP thread as there probably is one available already
when the test runs.

[Mono.Posix] Add ToString to struct Linger to help troubleshoot CI failure.

[System.IO.Compression] Fixed handling of empty Zip archives in Update mode. Fixes bug #32725.

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

[System] UriKind.RelativeOrAbsolute workaround.

In .NET an URI constructor from "/foo" and UriKind.RelativeOrAbsolute is
relative whereas in mono it is assumed as an absolute file path.

This adds another workaround that changes all Uri constructor calls to
behave like .NET when receiving UriKind.RelativeOrAbsolute.

The environment variable workaround consists in setting
MONO_URI_DOTNETRELATIVEORABSOLUTE=true before starting the application:

Reflection workaround:
```
if (Type.GetType ("Mono.Runtime") != null)
    typeof (Uri).GetField ("useDotNetRelativeOrAbsolute",
        BindingFlags.Static | BindingFlags.GetField | BindingFlags.NonPublic).
        SetValue (null, true);
```

WARNING: The workarounds change the behavior of all Uri constructor calls.
Unexpected problems can occur if an used library relies on
UriKind.RelativeOrAbsolute to parse absolute file paths.

Merge pull request #2030 from esdrubal/urispaces

Fixes Uri.UserInfo with spaces.

Fixes Uri.UserInfo with spaces.

Fixes #31172.

Tests Uri.UserInfo with spaces.

Covers #31172.

Merge pull request #2022 from esdrubal/fixes-GetDaylightChanges

Fixes TimeZoneInfo.GetDaylightChanges #32722.

[runtime] Fix the thunks.exe tests when not running with the coop gc.

Fix NREX in Linux network change notification code

Linux code uses a netlink socket to be notified about network interface
changes. The code is a bit racy and for that reson it can happen that
the LinuxNetworkChange.OnDataAvailable handler is called after the
socket is destroyed (from a timer event) and socket stored in an
instance variable is already null. This is a race most probably
introduced by the new threadpool or TPL code as it started to happen in
several places (mostly BCL tests on Android, but it's *not* specific to
Android).
The workaround here serves as a quick band-aid to be able to move on,
but a more thorough review of the socket code will be required.

[Mono.Posix] Fix a test to accomodate for OSX's getsockopt implementation.

[corlib] Fix a test. It was broken in so many ways, but hidden behind a check that disabled it everywhere. :(

[coop] Add coop support to libtest.

[mcs] Use auto-property backing field for setting only when setter is not available. Fixes #33600

[windows] Check for WINVER define before re-defining.

[windows] Fixed some warnings when calling mono_cond_init.

Conflicts:
mono/metadata/threadpool-ms-io.c

[windows] Include missing build headers when defining WIN32_LEAN_AND_MEAN.

DIST_SUBDIRS is a thing, fixes parallelism broken by previous change

See http://www.gnu.org/software/automake/manual/html_node/SUBDIRS-vs-DIST_005fSUBDIRS.html

Merge pull request #2024 from BogdanovKirill/webrequesttest

[HttpWebRequestTest] New test to check that after reading 2 GB of data from web server HttpWebRequest still works

Merge pull request #2026 from directhex/fix-missing-tarball-facades

[Facades] fix exclusion from "make dist"

[Facades] fix exclusion from "make dist"

If SUBDIRS is not set, then no files which aren't explicit Makefile targets will get included - which in this case means the only file in Facades/ in tarballs is the Makefile itself.

[mcs] Cache fixed array type expression. Fixes #33573

[HttpWebRequestTest] New test to check that after reading 2 GB of data from web server HttpWebRequest still works

When HttpWebRequest is used to download infinite (ContentLength is not set) response from web server, we must be sure that after reading 2 GB (Int32.MaxValue) of data our internal classes are still working correctly

This change is released under the MIT license.

[corlib] Fixes TimeZoneInfo.GetDaylightChanges.

Fixes #32722.

[corlib] Test GetSystemTimeZones in all timezones.

Covers #32722.

[mcs] Implements null operator used from both outer and inner expression of the statement. Fixes #33341

Merge pull request #2017 from ludovic-henry/master

[math] Fix Math.Pow (1, Double.MaxValue) != 1 on Linux 32bits

[runtime] Remove some dead code.

[sdb] Fix locals info when two c# level locals are allocated to the same IL local.

Merge pull request #2018 from alexrp/profiler-jit-info-removal

[profiler] Emit dummy code start/size info in cases where we can't retrieve it.

[wcf] Fix a regression introduced by 577d50e2e149143b7c3030dbcf3bf03f8d71d06d.

[runtime] Fix a regression introduced by f5222fe4f826afae1e8a628ffdba1d355d8e772f, arguments were not properly passed to threads started from runtime code.

[Facades] Fix accidental removal of a line

[runtime] Make mono_type_is_hfa () return TRUE for structs with more than 4 fields.

[math] Fix Linux 32bits tests

corlib/System.MathTest.Pow would fail at test #45 because the native pow implementation would return NaN (instead of 1) for pow (-1, Double.MaxValue).

The previous implementation treated this case for exponents bigger than 9007199254740991.0 and smaller than -9007199254740991.0

Bump reference sources

[corlib] Construct specific culture using most specific culture name region. Fixes #33400

Merge pull request #2010 from buraktamturk/master

Implement SqlDataReader.GetFieldValue<T> in order to fix EF's UpdateException

Merge pull request #2021 from delme-imgtec/patch-1

Update SynchronizedCollection.cs

Remove an obsolete comment.

Update SynchronizedCollection.cs

Change is to fix following exception. Which suspect is cause by two threads both call remove at same time.

System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
  at System.ThrowHelper.ThrowArgumentOutOfRangeException () [0x00000] in <filename unknown>:0
  at System.Collections.Generic.List`1[System.ServiceModel.Channels.IDuplexSessionChannel].RemoveAt (Int32 index) [0x00000] in <filename unknown>:0
  at System.Collections.Generic.SynchronizedCollection`1[System.ServiceModel.Channels.IDuplexSessionChannel].RemoveItem (Int32 index) [0x00000] in <filename unknown>:0
  at System.Collections.Generic.SynchronizedCollection`1[System.ServiceModel.Channels.IDuplexSessionChannel].RemoveAt (Int32 index) [0x00000] in <filename unknown>:0
  at System.Collections.Generic.SynchronizedCollection`1[System.ServiceModel.Channels.IDuplexSessionChannel].Remove (IDuplexSessionChannel item) [0x00000] in <filename unknown>:0

Revert "[sdb] Disallow nested invokes."

This reverts commit c4db814a44119462bc9029170790e1c445a93426.

Revert this, its supported functionality.

[runtime] Remove MONO_HASH_CONSERVATIVE_GC, its not used anymore.

Fix a warning.

[runtime] Obtain the thread argument directly from MonoThread, avoid storing it in a hash table.

[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.

[profiler] Emit dummy code start/size info in cases where we can't retrieve it.

[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