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>
```

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

Method field count (#3907)

* [runtime] Add accessors for method.count and field.count.

* [runtime] Move method/field count to MonoClassDef.

[sdb] Avoid asserting when trying to single step on a thread with no managed frames. Fixes #34802.

[runtime] Fix another race in mono_class_setup_supertypes ().

[threads] Always use the `suspended` MonoOSEvent for self-suspend (#3915)

* [stw] Refactor to treat error/success cases first

* [threads] Always use the `suspended` MonoOSEvent for self-suspend

The previous self-suspend mechanism was inherently racy:
 - T1 self suspend:
  - T1 calls mono_thread_info_begin_self_suspend, but doesn't call mono_thread_info_end_self_suspend yet
 - T2 suspends T1 <-- the suspend_count is > 0, trigerring the assertion at mono-threads.c:184

[threads] Destroy suspend MonoOSEvent when detaching (#3923)

* [threads] Destroy suspend MonoOSEvent when detaching

It could happen that we destroy the managed InternalThread object, before we actually detach the thread (in mono/tests/threadpool-exceptions1.exe for example).

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

* [threads] Close thread handle once done with it

[runtime] Simplify the wait_for_tids () code. (#3920)

[os-event] Make them manual only (#3921)

We only use them as manual reset event in the runtime, and that greatly simplifies the code as well as the way they should be used.

[sdb] Avoid asserting when trying to single step on a thread with no managed frames. Fixes #34802.

[io-layer] Remove dead code

[System] Fix a few type members on WatchOS

The .platformnotsupported.cs versions of some classes were
inconsistent with their "real" counterparts.

[process] Allocate a handle even for non-child processes (#3905)

* [w32handle] Inline only use of mono_w32handle_search

* [w32handle] Make special_wait return MonoW32HandleWaitRet

This is to avoid a unnecessary translation from WAIT_* to MonoW32HandleWaitRet

* [process] Use some more referencesource

* [process] Rename mono_processes to processes

* [process] Allocate a handle even for non-child processes

* [process] Create the handle only once the process has started

[os-event] Make unix version alertable (#3909)

This fixes `appdomain_unload.exe` test timeout. The following would happen:
 - T1 unload the current domain
  - T1 launch T2 to unload domain
  - T1 waits on T2 to finish
 - T2 tries to abort T1 <-- because the wait on T2 is not alertable, T1 is not going to be aborted

[runtime] Add locking to mono_class_setup_supertypes () since the 16 bit idepth field cannot be set atomically. Fix a race if mono_field_resolve_type () was called without locking. (#3919)

[x86] Fix max code size of throw trampoline.

Bump nunitlite for display fixes

Bump bockbuild

[io-layer] Include sys/types.h header file in wapi.h

Fixes "error: unknown type name 'pid_t'" on Alpine/musl libc.

[btls] Include stdint.h header file in btls-time64.c

Fixes "error: unknown type name 'int64_t'" on Alpine/musl libc.

Merge pull request #3914 from ntherning/mono_context_get_current-on-windows-does-not-save-rsp-properly

mono_context_get_current() on Windows stores registers at incorrect indexes

Merge pull request #3773 from mono/bockbuild-integration

Integration of Bockbuild's Mac SDK profile

Bump bockbuild

Move some more files back to bockbuild repo

mono_context_get_current() on Windows stores registers at incorrect indexes

The indexes must correspond to the values in the AMD64_Reg_No enum in
amd64-codegen.h.

Merge pull request #3913 from omwok/master

column name and ordinal fix.

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

In order to reduce dependencies on external toolchains when doing full AOT on Windows
targeting x64 Windows platforms we would like to use what's available in
VS 2015 MSVC toolchain. Mono's full AOT compiler need an assembler and a linker from
available toolchain in order to do different full AOT scenarios, emitting assembler, compile
objects or link complete shared library.

Since the current emitter emits GNU assembler it is beneficial to reuse that assembler
syntax without the need to adopt to a different syntax. In VS2015 Update 1 a new toolchain
option has been released utilizing the Clang parser combined with MSVC code gen. This will
produce object files fully compatible with MSVC linker. It turns out that VS2015 Clang version
also supports GNU assembler and since Mono just uses simple constructs in the assembler
(all code is already compiled and emitted as byte streams in assembler) it is possible to use Clang
for VS to compile our emitted assembler files.

This PR switches to Clang for Microsoft VS2015 and MSVC linker when doing full AOT targeting Windows x64
platforms hosted on a Windows platform. It requires an installation of at least Microsoft VS2015 Update 1
with Clang component selected. NOTE all other scenarios is currently left as is.

This PR also makes adjustments to AOT compiler in order for above scenario to work, supporting asm only output, static output
and regular shared libraries using MSVC linker.

Merge pull request #3911 from akoeplinger/marshal-cleanuprcw

[corlib] Add Marshal.CleanupUnusedObjectsInCurrentContext() stub

Additional none desktop API families triggered by changes.

* Changes to process.c triggered errors on none desktop API builds.
* Implementation of memory mapped files caused build errors on none desktop API builds.
* Adjustments to disable JIT caused build errors on none JIT builds.

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

Continuation work on previous none desktop Windows API family PR's adjusting all
changes to similar patterns used over previous PR's. This PR also move the logic
to silence a MS VS linker warning on empty source files into a macro. It also includes
the use of the macro in a couple of more source files causing linker warnings when build
on none desktop API families excluding JIT support.

Since other Windows API family work is still in progress it also disables build capabilities
for none desktop API families.

Merge pull request #3912 from akoeplinger/marshal-ptrtostrbstr

[corlib] Expose Marshal.PtrToStringBSTR on mobile

Merge pull request #3902 from henricm/fix-rename-lpstr-utf8

Rename fix for lpstr to utf8str on Windows

[corlib] Expose Marshal.PtrToStringBSTR on mobile

The SecureStringHelper from referencesource I added in https://github.com/mono/mono/pull/3887
relies on this to convert from SecureString to plaintext.

It failed on mobile because we always throw a NotImplementedException there,
but we actually have an implementation on the runtime side (even in the DISABLE_COM case).

[corlib] Add Marshal.CleanupUnusedObjectsInCurrentContext() stub

https://github.com/dotnet/standard/issues/94

[runtime] Fix the ios build.

[jit] Really fix the DISABLE_JIT build.

Merge pull request #3896 from lambdageek/bug-46250

[runtime] Ensure generic type definition arity matches generic context arity

[threadpool] Fix hang on domain unloading (#3904)

Because we wouldn't return the correct ThreadPoolDomain for the requested domain, we would simply wait on the last inserted MonoDomain. This would lead to a hang when unloading the domain, as we wouldn't wait on the correct domain.

Fix tarball build

[BTLS]: Use `int64_t` instead of `long` in the native API. (#3857)

* [BTLS]: Use `int64_t` instead of `long` in the native API.

* Copy time64.c from Android's source code to get a 64-bit timegm64().

* Rename into btls_timegm64() and remove everything we don't need.

(cherry picked from commit 2365067bdae39f75979df30b65ae08f095cbd176)

Merge pull request #3894 from akoeplinger/fix-actor

[System.ServiceModel] Recognize 'faultactor' in SOAP fault message

[jit] Fix the DISABLE_JIT build.

Merge pull request #2408 from tastywheattasteslikechicken/MoreInterfaceSupport

Enhance maximum number of supported interfaces from 2^16.

Merge pull request #3903 from ntherning/fix-NRE-in-Win32-IpcChannel.StopListening-when-no-server-started

Fix NRE in Win32 IpcChannel.StopListening() when no server started

Merge pull request #3901 from ntherning/Process_GetModules_internal-crash-on-windows

Fix crash in Process.GetModules_internal() on Windows after refactorings

Fix NRE in Win32 IpcChannel.StopListening() when no server started

This makes the System.Runtime.Remoting test suite fail in various places on
Windows due to the TestFixtureTearDown method in BaseCalls throwing an NRE.

Rename fix for lpstr to utf8str on Windows

PR https://github.com/mono/mono/pull/3776 renamed `mono_string_to_lpstr`
to `mono_string_to_utf8str` without updating the Windows version.
This fix renames `mono_string_to_lpstr` to `mono_string_to_utf8str` to
make sure the string is allocated using `CoTaskMemAlloc` on Windows
since it will be freed using `CoTaskMemFree`.

Fix crash in Process.GetModules_internal() on Windows after refactorings

EnumProcessModules() is called incorrectly. Also fixes a missing curly braces
pair.

Merge pull request #3900 from monojenkins/update-csprojs

[msvc] Update csproj files

[runtime] Move field.first and method.first to MonoClassDef. Add accessor functions for them. (#3898)

[msvc] Update csproj files

[arm64] Set a flag in MonoContext if the fp regs are saved, and only restore them if they are. These registers are not saved when the context is created from a ucontext. (#3890)

[mkbundle] Don't free bundled dylibrary directory (#3897)

mkdtemp modifies the template in-place, don't free it.

Merge pull request #3796 from ntherning/windows-backend-for-MemoryMappedFile

MemoryMappedFile support on Windows

[mcs] Consider method candidates with misplaced named arguments not applicable. Fixes #46190

[mcs] Remove debug check

Properly configure the bockbuild submodule

[runtime] Add a few mono mono_class_is_... accessors. (#3891)

Fix setting the min. macOS version in the Xamarin profile

Rename 'bockbuild' to 'packaging' so that it can be home to  other packaging code

Move the maintained common-use packages back to the bockbuild repo

[System.ServiceModel] Recognize 'faultactor' in SOAP fault message

It's an optional element according to [1] and we were simply
throwing a NotImplementedException when encountering it.

Fixed by reading the element value instead. I noticed that we
had a 'details' variable that was never assigned, so the check
for `details == null` was always true and can be removed.
This makes inlining the body of the CreateFault() call easier
since we no longer need to choose different overloads and makes
the code consistent with the CreateFault12 method.
Also added a better exception in case of an unrecognized element.

The CreateFault12 method (handles SOAP 1.2) does not need a similar
fix since actor is not an element in that version [2].

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

[1] https://www.w3.org/TR/soap11/#_Toc478383507
[2] https://www.w3.org/TR/soap12/#soapfault

[runtime] Ensure generic type definition arity matches generic context arity

Don't instantiate generic type definitions (like Dictionary`2) with too few or too many
type arguments.

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

[tests] Test System.Type.GetType() on generics with wrong arity.

[mcs] More work on debug scopes declared out of order. Fixes #45774

[os-event] Fix set operation

This is a logic bug where we would only wake up one of the threads that are waiting for the manual MonoOSEvent to be set. This would be a problem only if more than one thread is waiting.

[threads] Use MonoOSEvent for Thread.Suspend/Resume (#3881)

* [threads] Refactor how we create Thread and InternalThread in native

* [threads] Add coop support for async call

* [threads] Use MonoOSEvent for Thread.Suspend/Resume

By using a MonoOSEvent we avoid relying on the utils/threads suspend mechanism to suspend a metadata/threads. This reduce coupling between both mechanism, and that would allow us to remove a bunch of code from utils/threads when switching to coop.

This only works with coop suspend, as we need the ability to return from an async call, and that's not possible with preemptive suspend.

[tests] Remove # to enable tests

Make would consider the files following the first # to be comments, leading to not passing it to the test-runner.

Merge pull request #3893 from henricm/fix-class-accessor-vsproj

Adding class-accessors.c to VS project

Merge pull request #3887 from akoeplinger/networkcredential-securestring

[System] Replace NetworkCredential with referencesource

Adding class-accessors.c to VS project

Merge pull request #3892 from monojenkins/update-csprojs

[msvc] Update csproj files

[msvc] Update csproj files

column name and ordinal fix...tested on 10.1

[jit] Cleanup the usage of DISABLE_JIT a bit so it encompasses the whole mini-codegen.c file, add DISABLE_JIT to mini-llvm.c. (#3888)

[runtime] Revert parts of 016709fe52e921416322982f8185925f83de5866 which seems to cause races until they can be fixed. (#3889)

[System] Replace NetworkCredential with referencesource

Fixes an issue where the Mono implementation wasn't converting
between the SecureString and the plaintext version of the password.

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

Added 'mac-sdk-package' Make target

Fix a warning.

[arm64] Save+restore fp regs from the context in the call filter trampoline, the JITted code doesn't use them, but if the called code throws an exception, the throw context for the inner throw needs to contain the same values as the original throw context. (#3877)

Merge pull request #3871 from akoeplinger/mono.posix-windows

[Mono.Posix] Disable Unix-specific tests on Windows

[corcompare] Update custom XI project file to embed cecil sources