Merge pull request #3968 from BrzVlad/fix-monitor-exception

[runtime] Fix monitor exception throwing

Merge pull request #3979 from radical/fix-xbuild-tgt

[xbuild] Handle a repeat build of a target, when building through ..

[runtime] Add some printfs to help debug the g_assert (klass->instance_size == instance_size); assert failure.

[mcs] Fixes user operator extraction from shared list. Fixes #46806

Merge pull request #3965 from akoeplinger/ppc-disable-tests

[ppc64] Disable a few runtime/bcl tests that currently fail

[runtime] Fix the waiting for background threads which got broken by 4a1161e2d691672e8544ce5a3e20e5c08daa4f45. (#3983)

Merge pull request #3953 from rolfbjarne/offsets-tool-autoreclone

[offsets-tool] Make sure to re-clone when the hash for the cpp binaries changes.

[ppc64] Disable a few runtime/bcl tests that currently fail

Until someone can investigate those we want to have a clean CI.

Merge pull request #3966 from ntherning/prevent-multiple-c-runtimes-on-windows

Prevent Mono.Posix from using multiple C runtimes on Windows

[jit] Make mono_print_method_from_ip () work when invoked from threads which are not attached to the runtime.

[corlib] Remove public ctor from IncrementalHash

It doesn't have a public constructor in netstandard/corefx.

[xbuild] Handle a repeat build of a target, when building through ..

.. `Project.Build(..)`. For example, in:

```
<MSBuild Projects="bar.proj" Targets="foo;foo" />
```

The second invocation of `foo`, in the *same* `Project.Build(..)` call,
would get skipped, but BuildTarget assumed that no previous `outputs`
entry existed in the `targetOutputs` dictionary.

System.Exception: MSBuild operation failed ---> System.ArgumentException: Item has already been added. Key in dictionary: 'foo'  Key being added: 'foo'
  at System.Collections.Hashtable.Insert (System.Object key, System.Object nvalue, System.Boolean add) [0x00200] in <94fd79a3b7144c54b4cb162b50fc7761>:0
  at System.Collections.Hashtable.Add (System.Object key, System.Object value) [0x00000] in <94fd79a3b7144c54b4cb162b50fc7761>:0
  at Microsoft.Build.BuildEngine.Project.BuildTarget (System.String target_name, System.Collections.IDictionary targetOutputs) [0x0007e] in <b1524340ac7e4c06b22fac1b2db62c08>:0
  at Microsoft.Build.BuildEngine.Project.BuildInternal (System.String[] targetNames, System.Collections.IDictionary targetOutputs, Microsoft.Build.BuildEngine.BuildSettings buildFlags) [0x00134] in <b1524340ac7e4c06b22fac1b2db62c08>:0
  at Microsoft.Build.BuildEngine.Project.Build (System.String[] targetNames, System.Collections.IDictionary targetOutputs, Microsoft.Build.BuildEngine.BuildSettings buildFlags) [0x0004b] in <b1524340ac7e4c06b22fac1b2db62c08>:0

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

[btls] Download & extract BTLS certs, on Linux builds.

Actually use the correct pathname.  Martin.

Merge pull request #3956 from directhex/install-btls-certs-in-jenkins

[btls] Download & extract BTLS certs, on Linux builds.

Merge pull request #3967 from atsushieno/fix-linux-btls

Fix btls build for Linux.

Merge pull request #3950 from mono/work-btls-sni-2

[BTLS]: Add MonoBtlsSsl.GetServerName() and set MonoTlsConnectionInfo.PeerDomainName
(cherry picked from commit e6aa904d9d47186f01e92afc1d1f6d334aa46d1e)

Merge pull request #3678 from mono/seq-read

[IO] Implement FileOptions.SequentialScan for Mac/BSD

[mcs/build] Use make functions instead of calling a shell

[mcs/build] Factor Q_AOT for future use

[acceptance-tests] Add dependency on nunitlite.dll

nunitlite.dll is not built anymore with a normal Mono build
so we need to depend on it explicitly.

[acceptance-tests] Specify nunitlite path

Merge pull request #3971 from directhex/longer-longer-longer-longer-longer-lasting-tailcalls

Increase tailcall instruction length on ppc64

Prevent Mono.Posix from using multiple C runtimes on Windows

Since libMonoPosixHelper.dll is linked against ucrtbase.dll the C runtime
wrapper functions in it will call into ucrtbase.dll. Some pinvokes in
Mono.Unix.Native.Stdlib, however, bind directly to C runtime functions in
msvcrt. This is problematic if a resource (heap memory, file pointer, etc) is
allocated using one C runtime and then operated upon using funcitons form the
other C runtime. E.g., Stdlib.malloc() calls into a wrapper in
libMonoPosixHelper so it will use the ucrtbase heap while Stdlib.free() calls
directly into msvcrt's free() which will abort the process since it's called
with a pointer in an unknown heap.

This commit adds libMonoPosixHelper wrappers for all memory and file related
functions in Stdlib to ensure that all such functions use the same C runtime.

Some of the Mono.Posix tests still fail but at least the test suite doesn't
crash after this change. The test failures will be addressed in a future PR.

Increase tailcall instruction length on ppc64

wrong maximal instruction length of instruction tailcall (expected 120, got 124)
* Assertion: should not be reached at mini-ppc.c:4694

[runtime] Disable some failing tests until they are fixed.

[runtime] Change the thread wait code to use cond variables instead o… (#3946)

* [runtime] Change the thread wait code to use cond variables instead of events.

Instead of waiting for the thread handles to be signalled, wait for a cond variable signalled which is signalled by the code which calls pthread_join ().

* [runtime] Synchronize access to njoined_threads. Only use the background_change_event on windows.

* Fix a warning.

Bump bockbuild for helpful CI cleaning fix

[jit] Optimize out the early pinvoke address loading done by emit_native_wrapper () when coop+direct pinvokes is enabled, since its not needed, and it generates MONO_PATCH_INFO_ICALL_ADDR relocations which cause pinvoke lookups at runtime on platforms where its not supported like watchos. Fixes #45369. (#3960)

[runtime] Fix monitor exception throwing

mono_monitor_ensure_owned used to throw exception directly and not return. After switching the runtime to use pending exceptions, monitor code would continue running assuming we owned to monitor.

Fixes 46994

Fix btls build for Linux.

As @baulig suggested us elsewhere.

Remove global ipv6 disabled setting. Fixes #35536

Merge pull request #3963 from ntherning/fix-missing-property-bag-c-file-in-vs-project

Fix missing property-bag.c file in VS project

Merge pull request #3964 from ntherning/fix-vs-linking-error-missing-mono_dl_open

Fix VS error due to missing mono_dl_open()/mono_loader_register_module()

Fix VS error due to missing mono_dl_open()/mono_loader_register_module()

PR #3955 removes mono_dl_open() and mono_loader_register_module() from
msvc/mono.def and msvc/monosgen.def which causes VS linking error. This change
readds the symbols to the def files as a temporary solution to get the VS
builds to build successfully again.

Fix missing property-bag.c file in VS project

PR #3936 adds the mono/metadata/property-bag.c file but doesn't add it to
msvc/libmonoruntime.vcxproj, resulting in linking errors when building the VS
solution.

Merge pull request #3961 from akoeplinger/ppc-parallelism

[ci] Reduce make parallelism on ppc64

Merge pull request #3952 from akoeplinger/add-tests

[ci] Add missing tests from Wrench

[ci] Reduce make parallelism on ppc64

-j4 seems to fail the build.

Merge pull request #3936 from kumpera/monoclass_reorg2

More MonoClass reorg, replace 3 fields with a property bag

[ci] Add missing tests from Wrench

While looking at the Wrench config I noticed there are a few
tests we don't run on Jenkins. This change adds those tests.

Merge pull request #3958 from akoeplinger/disable-tests

[runtime] Disable certain tests on PRs

[runtime] Disable certain tests on PRs

This allows us to get back to green builds for PRs
while still keeping track of failing tests in master.

[runtime] Cache marshal info in locals whenever possible.

Merge pull request #3945 from kumpera/fix-cross-domain-abort

[corlib] Fix cross-domain exception serialization under thread abortion.

[bcl] Add comment on why the empty try block.

Merge pull request #3951 from akoeplinger/add-csproj-warning

[msvc] Add warning about autogenerated csproj to genproj

[btls] Download & extract BTLS certs, on Linux builds.

Merge pull request #3955 from kumpera/fix-embedding-api

[runtime] Don't add mono-dl.h to loader.h. Move the internal function to an internal header.

[runtime] Don't add mono-dl.h to loader.h. Move the internal function to an internal header.

[offsets-tool] Make sure to re-clone when the hash for the cpp binaries changes.

[w32handle] Make mono_w32handle_ops_* functions private

[w32handle] Compile them in on windows

[w32handle] Remove unused header includes

[os-event] Replace global cond by local cond for each waiting thread (#3941)

[msvc] Update .csproj files

[msvc] Add warning about autogenerated csproj to genproj

Hopefully reduces the confusion about the csproj files.

[BTLS]: Use host name without port for the Server Name Indication (SNI).  Bug #46549.

(cherry picked from commit 20e0914e0319078e02a53be2fb4700e39d85347d)

Merge pull request #3940 from akoeplinger/deprecate-nunit

[bcl] Deprecate old NUnit 2.4 framework shipped with Mono

Merge pull request #3937 from akoeplinger/mph

[msvc] Add Mono_Unix_VersionString to monoposixhelper.def

Revert "Install utils/{mono-dl.h,mono-compiler.h}"

This reverts commit 397430c7f9fe4fae487342db36860b0a08cd56c0.

[msvc] Add Mono_Unix_VersionString to monoposixhelper.def

Fixes the EntryPointNotFoundException when running the Mono.Posix tests on Windows.
This check was introduced in 249ee0686bf87f33acf2efde76f3645ef06bb886 but the .def file wasn't updated.

With this the tests still crash so switching them off on PRs again for now.

[bcl] Deprecate old NUnit 2.4 framework shipped with Mono

Print a message when running nunit-console about the deprecation.
Additionally, throw a compilation error if compiling against the
TestFixtureAttribute from old NUnit.

Needed changes in ms-test-suite as well since it was still using NUnit.

Install utils/{mono-dl.h,mono-compiler.h}

These two headers are required by loader.h which is installed in
$prefix/include/mono-2.0/mono/metadata and their lack breaks compilation
of any software that includes loader.h

Merge pull request #3938 from ntherning/UnicastIPAddressInformation.IPv4Mask-from-refsource

Change UnicastIPAddressInformation.IPv4Mask to work as in refsource on Windows

[corlib] Fix cross-domain exception serialization under thread abortion.

RemotingServices::SerializeExceptionData started to fail as TAE was being raised
from within the serialization code and this triggered the completely wrong abort protection code.

This happened because a recent change that made .cctors raise TAE at their end and the stack trace
code would hit System.Reflection.Module's cctor to fetch its GUID for the purpose of symbolification.

The fix is to use the correct protection against aborts, which is to run code from finally blocks.

This fixes multiple failures in the runtime test suite.

[runtime] Add back the locking to setup_interface_offsets () to work around crashes in roslyn.

[os-event] Replace global cond by local cond for each waiting thread

Merge pull request #3942 from monojenkins/update-csprojs

[msvc] Update csproj files

Fix git submodule url

[msvc] Update csproj files

Remove extracted linker sources and reuse them via submodule

[w32handle] Stop returning 0 in every cases for locking/unlocking (#3926)

* [w32handle] Stop returning 0 in every cases for locking/unlocking

* [w32handle] Ensure we lock the signal handle in SignalAndWait

[mcs] Update better conversion tie breaking rules to cover candidate with longer default parameters. Fixes #45950

Change UnicastIPAddressInformation.IPv4Mask to work as in refsource on Windows

Mono returns null for disconnected interfaces with 169.254.X.X address while
refsources returns 255.255.0.0. This patch copies the relevant pieces from the
refsources to make Mono behave the same.

[offsets-tool] Fixed the Makefile to clone into the base directory instead.

Merge pull request #3809 from lateralusX/jlorenss/win-api-family-support-cleanup

Build mono runtime under none desktop Windows API family, adjustments and cleanup.

Merge pull request #3930 from ntherning/fix-System.ServiceProcess

Fixes bugs in System.ServiceProcess on Windows

Fixes bugs in System.ServiceProcess on Windows

There was a crash on Windows x64 in Win32ServiceController when enumerating
devices and services. Also fixed the ServiceControllerTest to succeed on .NET
as well Mono on Windows. Had to replace the services used by the test since
most of them have been removed from recent Windows versions.

This PR also enables the System.ServiceProcess tests in Windows PR builds.

[eglib/mono] Aleksey caught a discrepancy between the semantics of Unix and Windows, our eglib version should always strdup (#3899)

[xbuild] Don't reevaluate project when setting metadata in a dynamic ..

.. item.

If we have a project, that sets some properties or items at runtime,
like in a target:

```
        <Target Name="Build">
                <PropertyGroup>
                        <Bar>Bar01</Bar>
                </PropertyGroup>
```

.. such a property can be accessed as `$(Bar)` subsequently.

Then at later point, a target tries to update metadata on an item group
with existing items, like:

```
<ItemGroup>
<FooItem Include="xyz" />
</ItemGroup>

<Target Name="Foo">
                <ItemGroup>
                        <FooItem>
                                <SomeMetadata>MetadataValue</SomeMetadata>
                        </FooItem>
                </ItemGroup>
```

.. then it is seen that the value for the earlier created `$(Bar)` disappears!
So, if we try to print the value of `$(Bar)` right after that item
group, it would appear as `''`!

The issue is that the metadata update in target `Foo` caused the project
to get re-evaluated, which meant that items/properites created after the
project load were lost! We should not be reevaluating the project when
setting metadata on the basis of dynamic items, like in the target
`Foo`.

These dynamic item groups in a target are represented as
`BuildItemTask`, and these create BuildItems which have their
`IsDynamic` property set. And this is used to avoid re-evaluating the
project in `BuildItem.SetMetadata`.

But when updating metadata (`BuildItem.UpdateMetadata`), we need to
update metadata on the *existing* items, which might not have been
created from such dynamic item groups! So, their `IsDynamic==false`.
Hence, trying to `SetMetadata` on such items would cause the project to
be reevaluated, and thus properties/items like `$(Bar)` would be lost!

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

In this particular case, `Bar` was the `DesignTimeBuild` property, which
was losing it's value when `ProjectReference`'s `AdditionalProperties`
metadata was set in:

```
 <Target Name="BclBuildAddProjectReferenceProperties"
    <ItemGroup>
      <ProjectReference>
         <AdditionalProperties>$(_BclBuildProjectReferenceProperties);%(ProjectReference.AdditionalProperties)</AdditionalProperties>
     </ProjectReference>
    </ItemGroup>
```

[runtime] Move MonoClass::ref_info_handle into a property bag.

[runtime] Move MonoClass::marshal_info and MonoClass::ext to use a property bag.

[runtime] Wrap MonoClass::ext with accessors.

[runtime] Wrap MonoClass::marshal_info with accessors.

[runtime] Add a simple linearizable property bag implementation.

[arm64] always assume a cache line size of 4

otherwise we still have the theoretical chance of a race.

Merge pull request #3931 from ntherning/enable-System.Core-tests-on-windows-pr-builds

Enable System.Core tests on Windows PR builds

Merge pull request #3927 from tritao/updated_offsets_tool

Updated offsets tool to latest CppSharp.

[System.Xml] Add XElement array handling to serialization interpreter. Fixes #12571

[system.xml] Enable more tests

[offsets-tool] Updated build scripts to handle multiple platforms.

[offsets-tool] Fixed whitespace inconsistency.

Updated offsets tool to latest CppSharp.

Enable System.Core tests on Windows PR builds

System.Core is expected to be green now on Windows.

[ppc] clobber r0 in MONO_CONTEXT_GET_CURRENT

r0 can't be the base for a memory acces.

[ppc] some 32bit CPUs trap on std, use stw instead

[ppc] enable MONO_ARCH_HAVE_SIGCTX_TO_MONOCTX on every PowerPC configuration on Linux

[ppc] use proper calling convention when coming from a signal handler

ELF v2 ABI (aka. ppc64le) defines two entry points per function.
The global entry point (offset 0) sets up the TOC (r2) itself, while the
local entry point (offset 8) is a fast entry point if the TOC is
already set up properly.

When the global entry point is used, the address of the global entry
point is required in r12 according to the ABI.

[ppc] disable MONO_ARCH_HAVE_TLS_GET

`--with-tls=__thread` is broken on PowerPC.
`--with-tls=pthread` didn't work on PowerPC, but disabling
MONO_ARCH_HAVE_TLS_GET fixes it.

Instead of guarding the define with `HAVE_KW_THREAD`, let's just
disable it.  We want to get rid of `--with-tls=__thread` anyway.

Merge pull request #3854 from lateralusX/jlorenss/win-clang-toolchain

Switch to Microsoft VS2015 MSVC toolchain for Windows x64 AOT compiler.