Merge pull request #3540 from kumpera/abort-on-gerror

[logging] Fix g_error not aborting.

Once we enabled g_log redirection, we stopped aborting as the new handlers didn't behave like eglib.

Merge pull request #3528 from BrzVlad/fix-sgen-check-before-collections

[sgen] Fix sgen debug flags

[logging] Fix g_error not aborting.

[threads] Factor common attach code in mono_thread_attach_internal (#3530)

* [threads] Allocate StartInfo in create_thread

* [threads] Put start_notify semaphore in StartInfo

* [threads] Factor common attach code in mono_thread_attach_internal

* [threads] Use same registered mechanism as mono-threads

Merge pull request #3537 from ntherning/fix-failing-ThreadTest-tests-on-windows

Remove Assert.Fail() calls in ThreadTest when running on Windows

Remove Assert.Fail() calls in ThreadTest when running on Windows

The tests that fail like this actually work just fine on Mono on Windows.
Added the NotDotNet category to the CurrentThread_Domains() test since it
fails on .NET.

Avoid race condition between domain unload and threadpool. (#3491)

* Avoid race condition between domain unload and threadpool.

Previous logic only ensured that domain->threadpool_jobs was 0.
There was a race condition as follows:
* worker jobs are queued into ThreadPoolDomain outstanding_request's
* domain unload starts
* domain->threadpool_jobs is 0 so unload proceeds
* queued jobs are processed from the ThreadPoolDomain outstanding_request's
* the domain can be set/entered by worker threads since
  mono_domain_set only fails for domains with state of MONO_APPDOMAIN_UNLOADED,
  while our domain is in the MONO_APPDOMAIN_UNLOADING state
* jobs are now being unexpectedly run in domain that is being shutdown

* Fix a hang when unloading the domain.

This change applies the diff from ludovic-henry to correct the hang.

https://gist.github.com/ludovic-henry/6c8a4bc8951091c0ef36df2e52d5e32e

Merge pull request #3532 from alexrp/profiler-stress-tests

[acceptance-tests] Add profiler-stress suite.

[acceptance-tests] Add profiler-stress suite.

Merge pull request #3531 from alexrp/master

[profiler} Don't crash if a zero sampling frequency is given.

[profiler} Don't crash if a zero sampling frequency is given.

Moving WSAGetLastError calls closer to socket operation (#3450)

* When doing async connect on windows, connect returns `WSAEINPROGRESS`
which needs to be returned to managed code for correct handling. However
on Windows, last error gets reset before `WSAGetLastError` is called.
This fix moves all `WSAGetLastError` calls up closer to the socket
operation to make sure correct error code is returned to the managed code.

* Making sure `TcpNoDelay` is set to false for IPv4 by default and
that `DualMode` is default for IPv6.

* Making sure option value is set to 0 before calling getsockopt
because on Windows, a `BOOL` option value (eg TcpNoDelay) is returned
as a one byte value and the other bytes are left untouched which resulted
in TcpNoDelay always seemed to be true.

* Fixed so `TF_REUSE_SOCKET` is set only if `reuse` parameter is set
when disconnecting a socket.

Merge pull request #3511 from henricm/fix-no-socket-reuse

Not reusing sockets in dual mode tests

[threads] Remove CREATE_SUSPENDED (#3529)

* [threads] Remove use of CREATE_SUSPENDED for domain unload thread

* [threads] Remove use of CREATE_SUSPENDED for managed thread

* [threads] Remove CREATE_SUSPENDED

Merge pull request #3526 from alexrp/master

[sgen] Don't consider managed write barriers to be critical methods.

[sgen] Make sure we don't sweep a block if we're not supposed to

If the sweeping mechanism has been triggered by a major collection, then the iteration will wait for the general sweeping to finish and also for the sweeping of all the blocks. If this isn't the case (we are doing a major collection) we will iterate over the current state of the heap, some objects being bound to die during the collection, and the callback must account for this.

It doesn't make much sense to explicitly attempt to iterate without trying to sweep blocks that can be swept.

Rename local file into socket_fd (#3524)

This is to avoid hiding the static gint `file` and to better understand the code.

Merge pull request #3509 from ludovic-henry/iolayer-extract-threading

[mono-threads] Keep cleaning up mono-threads after iolayer extraction

[sgen] Don't consider managed write barriers to be critical methods.

When we emit calls to them, we also emit an OP_DUMMY_USE instruction to keep
the object alive, so it is OK for the GC to interrupt write barriers now.

[sgen] Remove unused and potentially wrong case from major object iteration

We never enter this case since we never iterate over the major objects without requiring to sweep, while a sweep is running. Even if we would be in this case, we directly access the mark bits which can be cleared at the same time by another thread sweeping the block, so we could miss live objects.

[sgen] Fix mod union consistency check

We don't need a mod union to mark an object with concurrent mark&sweep.

[sgen] Don't do remset consistency checks during majors

It doesn't make sense to check the remset during a major since majors dont use the cardtable and we clear it at the start. So we will obviously have missing remsets. For the debugging concurrent case, we should use the mod union consistency checks instead.

[sgen] Don't verify the heap after a major

We would need to deal with scanning forwarded objects and dead objects which is not worth it. Only verify the heap at the start of a collection, as indicated by the name of the flag.

[sgen] Fix missing remsets with verify before collections

When verifying the heap we need to have the set of pinned objects otherwise we can observe missing remsets if suspended between the store and the write barrier. If we don't have the set of pinned objects, allow missing remsets.

[mono-threads] Have error in case we cannot set thread priority

[mono-threads] Fix win32 to posix priority conversion

[mono-threads] Store handle related data in MonoThreadInfo

[mono-threads] Assert that we do not try to signal the handle multiple times

Merge pull request #3518 from akoeplinger/fix-fileiopermission

[corlib] Ensure FileIOPermission ctor is not called on mobile

[corlib] Ensure FileIOPermission ctor is not called on mobile

After https://github.com/mono/mono/pull/3452 was merged, the LinkSdkRegressionTest.SecurityDeclaration
test in XI started failing:

```
[FAIL] LinkSdkRegressionTest.SecurityDeclaration :   FileIOPermissionAccess
  Expected: null
  But was:  <System.Security.Permissions.FileIOPermissionAccess>

  at LinkSdk.LinkSdkRegressionTest.SecurityDeclaration () [0x00033] in /Users/builder/data/lanes/1411/2a96404f/source/xamarin-macios/tests/linker-ios/link sdk/LinkSdkRegressionTest.cs:990
  at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /Users/builder/data/lanes/1411/2a96404f/source/xamarin-macios/_ios-build/Library/Frameworks/Xamarin.iOS.framework/Versions/git/src/mono/mcs/class/corlib/System.Reflection/MonoMethod.cs:305
```

Due to the added call in the PR the linker preserved FileIOPermissionAccess enum,
which is apparently something we want to avoid.

Since according to https://github.com/mono/mono/pull/3452#discussion_r76384945 the only point of calling the
FileIOPermission is validating that the path doesn't contain invalid characters (mostly an issue on Windows),
I decided to just copy those checks as it's way cleaner.

Merge pull request #3525 from henricm/fix-access-level-get-token

Using max level allowed when getting current token

Using max level allowed when getting current token

Reference source of WindowsIdentity is using `MAXIMUM_ALLOWED`
as desired level by default when getting current token.
Changing to same behaviour to enable duplicate and impersonation
to be made on the token retrieved by GetCurrentToken.

[mcs/Makefile] make the 'btest' (bootstrap test) target work again

[jit] Unify the handling of 'error' and 'exc' in the llvm-only and non llvm-only runtime invoke code. Fixes #41961.

Better handle marshaling of packed types (#3508)

* This improves the marshaling for packed classes and class involved in an inheritance hierarchy.
* Similar changes were made in the Unity fork of mono across a number of commits.
* The tests here have been taken from the repo and converted to work as NUnit tests in Mono.

[jit] Emit a dummy call after calls to Type.GetType () in llvm-only mode to prevent clang from transforming the call to a tail call which would cause the stack walk done by GetType () to find the wrong frame. Fixes #41956.

Merge pull request #3521 from ntherning/fix-failures-in-RegistryKeyTest-on-windows

Fix for failures in RegistryKeyTest on Windows

Merge pull request #3520 from ntherning/fix-ModuleTest-too-long-temp-folder-path-on-windows

Prevent ModuleTest failures due to too long temp folder path on Windows

Fix for failures in RegistryKeyTest on Windows

Refcatored Win32RegistryApi.SetValue() to accept uint/byte/etc values for
DWord registry values.

Added handling of ERROR_CHILD_MUST_BE_VOLATILE errors to GenerateException().

Centralized MarkedForDeletion error handling to GenerateException().

Fixed tests which create volatile subkeys in RegistryKeyTest which used the
same key names but didn't clean up properly.

Merge pull request #3505 from ntherning/fix-resource-lookup-in-ResourceReaderTest-on-mono-on-windows

Lookup of test resource files in ResourceReaderTest fails on Windows

Prevent ModuleTest failures due to too long temp folder path on Windows

On Windows the TearDown() cannot delete the temp folder since it contains
loaded assemblies. The next time the test is run it will create a temp folder
inside the previous folder. After a number of runs the temp folder path will
approach the ~260 max path supported on Windows and some of the methods will
start failing.

This patch changes the way the temp folder is created and reduces the depth in
the same way as AssemblyTest was patched to fix a similar issue in
b90617e7b7a9ecd501abc65d171d97163ce4ee3d.

[sgen] Remove unused parameter

We never do nursery collections as part of the concurrent finishing pause

Lookup of test resource files in ResourceReaderTest fails on Windows

Removed the differing code paths in FixtureSetUp() when running on non-Windows
platforms vs when running Mono on Windows or on .NET. The path previously
taken only on non-Windows platforms has been verified to work as expected on
both Mono on Windows and on .NET.

[Dataflow] Bump timeouts in TransformBlockTest.DeferredUsage

It failed a couple of times according to the babysitter logs with the following error:

```
MESSAGE:
Expected and actual are both <System.Int32[10]>
  Values differ at index [1]
  Expected: -1
  But was:  0
```

Bump timeouts a little bit to see if this helps.

Merge pull request #3514 from ntherning/more-reliable-g_get_home_dir-on-windows

Prefer SHGetKnownFolderPath() in g_get_home_dir() over HOMEDRIVE/HOMEPATH environment variables

Merge pull request #3517 from alexrp/master

[man] Update mprof-report.1 to cover new profiler options.

Prefer SHGetKnownFolderPath() in g_get_home_dir() over HOMEDRIVE/HOMEPATH
environment variables

Relying on the HOMEDRIVE/HOMEPATH environment variables is fragile as those
variables don't always have to be set. One such scenario is when running on a
Jenkins slave. When not set g_get_home_dir() in eglib would return NULL which
led to a crash in ves_icall_System_Environment_InternalGetHome().

This patch changes g_get_home_dir() to first try to call
SHGetKnownFolderPath() to get the current user's profile directory (i.e. home
dir). If that fails it falls back to the USERPROFILE environment variable. If
that isn't available the function will fallback to the original behavior and
try with HOMEDRIVE/HOMEPATH.

[man] Update mprof-report.1 to cover new profiler options.

Also reformat some sections a bit.

[SRE] Move the resolving of user types to managed code, the runtime was not using write barriers. (#3515)

[runtime] Fix a C&P problem introduced by d4052c556399c2ef72c79088c05a8ad6913647a6.

[SRE] Remove empty TypeBuilder:setup_generic_class () icall.

[runtime] Remove the stub of ves_icall_TypeBuilder_create_internal_class ().

[corlib] Fix the signature of a few SRE icalls, they already receive a TypeBuilder as the 'this' argument.

Merge pull request #3446 from lambdageek/bug-42584

[loader] Check for ReferenceAssemblyAttribute when loading

[mcs] Adjust correctly newline counter inside pragma warning

[corlib] Add timeout to failing test

Merge pull request #3506 from ntherning/make-ZoneTest-succeed-on-mono-on-windows

Make Zone.CreateFromUrl() work with URLs like file://C:\foo\bar

Merge pull request #3452 from ntherning/fix-AppDomainSetupTest-on-windows

Validate AppDomainSetup.ApplicationBase path more like .NET does

Not reusing sockets in dual mode tests

Creating new sockets between each connect instead of
trying to reuse the client socket. The original tests
was not working in the MS .NET runtime and socket reuse
is not recommended to use unless in very special cases.

[System.Core] Allow TryConvert to return null for any reference types (not only object). Fixes #41509

[runtime] Check for ReferenceAssemblyAttribute when loading for execution.

[loader] Tests for RefereceAssemblyAttribute support

[runtime] Add mono_assembly_get_reference_assembly_attribute

Checks whether the given assembly has the
System.Runtime.CompilerServices.ReferenceAssemblyAttribute applied.

Merge pull request #3455 from kumpera/abort_cctor

[runtime] Don't abort .cctors. Fixes #43320

[mono-threads] Factor common platform create thread code (#3484)

Bundle other options (#3501)

* [mkbundle] add support for baking environment variables, make endian-independent

* [runtime] Bundle support, implement missing stream processing

Merge pull request #3498 from mattleibow/bug-41723

[Mono.Data.Sqlite] Added checks for entry point existence

Make Zone.CreateFromUrl() work with URLs like file://C:\foo\bar

On Windows the test used URLs like `file://C:\tmp\foo` which the
Mono.Security.Uri class was incapable of parsing. System.Uri from
referencesource parses such URLs just fine. This patch fixes the Uri class to
be able to parse such URLs. It also changes the Zone class to not throw an
exception if the Zone.CreateFromUrl() method is passed an invalid URL. Instead
the zone is set to NoZone. This seems to be how Zone in .NET behaves.

Merge pull request #3503 from ntherning/fix-MarshalTest.GetHINSTANCE-on-mono-windows

Ignore the MarshalTest.GetHINSTANCE() test on Mono on Windows

Merge pull request #3504 from ntherning/fix-PropertyInfoTest.ConstantValue-fails-to-delete-locked-file-on-windows

PropertyInfoTest.ConstantValue() shouldn't fail if temp file cannot be deleted

PropertyInfoTest.ConstantValue() shouldn't fail if temp file cannot be deleted

Happens when running this test on Windows. Just swallow the exception if the
File.Delete() fails.

Ignore the MarshalTest.GetHINSTANCE() test on Mono on Windows

Won't work on Mono even when running on Windows.

[Mono.Data.Sqlite] Using the Version class to do the version comparison correctly

Typo fix

Add a man page for cert-sync (#3424)

[runtime] Add a mono_threads_close_thread_handle () function and use it to close thread handles instead of CloseHandle (). (#3497)

Merge pull request #3492 from lewurm/fix-insert-insn

[mini] emit sequence point before branching off for tail call [was: respect seq points when adding instruction at end of block]

Merge pull request #3499 from kumpera/fix_logging

[logging] Ensure glib logging redirection and remove unwanted new pub api functions.

[Mono.Data.Sqlite] Instead of try/catch and dlopen/dlsym, use a cross-platform, correct way of determining API features: version

[threading] Update comments.

[runtime] Introduce abort protected blocks. Use it within async invokes and unhandled exceptions.

There are a few places in the runtime that should not be aborted as it would render it unusable.

cctor execution was the initial trigger for this changeset, but then edge triggering of TAE turned out
to expose this issue on other places.

This patch introduces those regions and apply them to the following places:

ves_icall_System_Runtime_Remoting_Messaging_AsyncResult_Invoke: Not delivering the completed CB
screws up any completion logic the user setup up.

mono_unhandled_exception: Protect delivery of the unhandled exception. Can't fail during it or we'll
effectively double fault and abort.

[runtime] Add test for the interactions between thread abort and .cctors.

This covers bug #43320

[runtime] Don't start aborting a thread if it's running any .cctor. Fixes #43320.

A thread abort must not start if the thread is running a .cctor.
OTOH, if the cctor self aborts, it should start.

Threads now track how many cctors they are running in the cctor_exec_depth variable.

The abort machinery (in async_abort_critical) checks that flag and bail from
starting the abort.

The cctor code (in mono_runtime_class_init_full) ensures that cctor_exec_depth is bumped
around execution of the cctor itself. Additionally, error handling was cleaned up as it was pretty
inconsistent.

[runtime] Add mono_current_thread_has_handle_block_guard callback to the runtime EH callbacks.

This callback allows the runtime to query mini on whether there's a block guard in play.
It allows for the runtime to correctly decide when to start an abort.

[logging] Ensure that glib logging goes through the new logging mechanics and that the domain prefix is preserved.

Call g_log_set_default_handler in mono_trace_set_log_handler and mono_trace_set_log_handler_internal to ensure that g_ logging is properly routed.

Additionally, change the internal logging callback to take the log message as a single string to simplify things.

[Mono.Data.Sqlite] Make the fields read-only

[Mono.Data.Sqlite] Added iOS specific checks for entry point existence:
 - using dlfcn.dlsym to avoid P/Invoke exceptions

[logging] Move internals that were landed in public headers by mistake.

[runtime] Fix a warning.

[profiler] Fix some off-by-ones in header size calculation.

Merge pull request #3493 from alexrp/master

Follow-up PR for #3481

[corlib] Disable CS0169 for RuntimeStructs.cs.

[llvm] Error out when using --aot=llvmonly in a runtime compiled without llvm.

[arm] Make mono_arch_get_call_target () work on direct branches too, previously it returned NULL, which broke the code in handle_thunk () for tail calls.

[profiler] Include arguments, architecture, and OS in format header.

Bumped the major/minor version to 1.0 for this change.

[profiler] Fix dump_header () to do fflush () in all non-gzip code paths.

[runtime] Export API functions to get arch/os/wordsize strings.

[profiler] Remove duplicate #include.

[profiler] Update documentation for time fields.

[profiler] Document the TYPE_EXCEPTION format.