[msvc] Update csproj files (#5131)

[man] Fix AOT temp path option description

It's `temp-path` in the code, not `temp_path`

Merge pull request #5120 from lambdageek/bug-57744

[reflection] Convert correct MonoError to an exn in Assembly.GetTypes ()

Fix various compiler warnings (#5064)

* Fix compiler warnings (w32)

- add explicit (long long (unsigned)) type casts

- change %ld to %lld once, to match the general style of these logging messages

* Fix compiler warnings (threadpool)

- add explicit (void *) type casts

* Fix compiler warnings (boehm)

- add an explicit (long) type cast; the values are defined as gint64, an upgrade to at least "%ld" seems like a good idea

- about dropping const: const does not seem to be necessary as g_getenv() returns pointers to freshly allocated buffers whenever called. const might have been used to protect the return value of g_getenv() on unix systems with earlier versions of Mono (as a comment in gmisc-unix.c suggests). keeping const here causes a compiler warning, as the const qualifier gets dropped when *env is being used with g_free(). that led to the decision to drop the qualifier altogether.

* Fix compiler warnings (sgen)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

* Fix compiler warnings (misc - metadata)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

* Fix compiler warnings (misc)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

- add an explicit (void *) type cast; changing the return type to char* might be more usable and could be discussed

- ret is not being used and can be removed; fixes a -Wunused-but-set-variable warning

* Fix compiler warnings (mini)

- specify the arguments (void) to avoid -Wstrict-prototypes warnings

* [fixup!] Correct indentation

* [fixup!] Correct (void *)
- use GUINT_TO_POINTER and MONO_NATIVE_THREAD_ID_TO_UINT instead of (void *) to cast mono_native_thread_id_get () to %p

* [fixup!] Correct indentation of boehm-gc.c and add FIXME

* include <inttypes.h> and test the G_GINT64_FORMAT macro

* [fixup!] Use G_G*_FORMAT and PRIx32 macros instead of type casts wherever possible

[packaging/msbuild] Bump to get new changes on xplat-master (#5127)

[jit] Avoid emitting op_load_got_addr opcodes on x86 when using llvmonly, its not needed. (#5125)

[reflection] Convert correct MonoError to an exn in Assembly.GetTypes () (Fixes #57744)

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

[bcl] Delete our ReflectionTypeLoadException

We use the referencesource version

[cil-strip] Move comment out of .sources file

I forgot that we only run gensources.sh on libraries but not
tools so the comments don't get stripped out...

[runtime] Add more logging in case mono_os_cond_timedwait () fails.

Bump API snapshot

[System.Core] Implement MemoryMappedFile.OpenExisting

Reuse the existing icall infrastructure to give the same handle back.

This PR implements all OpenExisting methods (with the access rights parameter discarded) and the SafeHandle property.
In the runtime, we don't check the capacity parameter if we try to open a mmap file, and we fallthrough the handle refcount case.

Revert "[cil-strip] Remove old private copy of Mono.Cecil"

(reverted from commit 4ea31227587b9df3a244d9c32a5a87a7afa70436)

Revert "[cil-strip] Upgrade to latest Mono.Cecil API"

(reverted from commit 0c9eb70a4646364f4b425f69b366be766043bef9)

Using newer Cecil means metadata tokens will be different which
breaks AOT compilation: https://github.com/mono/mono/pull/5102#issuecomment-310801767

[msvc] Update csproj files

[llvm] Align aot variables to 8 bytes, parts of the aot code depend on it.

[null_gc] Get null GC building again.

- Had to borrow a gchandles implementation from mono/metadata/boehm-gc.c
- Added coop handles support for NULL GC
- Fixed typos in configure.ac
- To use: `./configure --with-libgc=none --without-sgen --enable-boehm`
  You get a mono-boehm which actually is not collecting anything.

[packaging/msbuild] Bump to get new changes on xplat-master (#5113) (#5114)

[System] Fix compilation on watchos (#5108)

[cil-strip] Remove old private copy of Mono.Cecil

No longer needed now that we're using the new Cecil.

[cil-strip] Upgrade to latest Mono.Cecil API

This greatly simplifies the code and as a bonus also shaves off
about 300ms when stripping mscorlib.dll.

Merge pull request #5103 from alexrp/profiler-signal-ack-fix

[profiler] Do the signal ack after checking that the thread is detached.

[aot] Fix aot image loading messages in static mode.

Merge pull request #5084 from BrzVlad/feature-worker-context

[sgen] Implement sgen worker context

[sgen] Use concurrent mark concurrently with parallel minors

[sgen] Implement work context for thread pool threads

We need to run jobs on the thread pool threads for parallel minor, parallel major, concurrent mark and concurrent sweep, some of which can run simultaneously. Instead of making separate threads reserved for each type of work, we create a work context abstraction (this contains a set of pending jobs to run and the sgen-worker callbacks). When a thread pool thread looks for work, it searches for work (jobs and idle work) through all contexts and finishes work from each one of them, from the highest to the lowest priority.

This is implemented by having different thread pool contexts (which contain the job list, work callbacks that indicate whether it should work and its ineherent priority), overall unrelated to the GC. The sgen worker infrastructure also creates separate worker contexts (which contain the object ops to be used and information about each worker, like the state and the private gray queue). We use at most two worker contexts (for minor and for major) and each worker context makes use of a sgen thread pool context.

[sgen] Remove redundant code

We always init worker distribute gray queue when initialising worker infrastructure.

[sgen] Include split count in the parallel job

We were getting the split_count at job run time which makes little sense. Include it in the job, alongside the job index.

[sgen] Run the scan pinned object job concurrently with minors

Concurrent mark doesn't normally scan objects in the nursery since they can be moved. At the end of the concurrent mark, we were locking over the entire gc pause in order to make sure that the we can scan the latest pinned object list (aka none of those objects can be moved). Lock only over the pinning phase of the collection, since it is enough.

Merge pull request #5098 from BrzVlad/fix-sgen-pagesize

[sgen] Fix sgen pagesize

Merge pull request #5104 from mono/bump-bockbuild

Bump bockbuild

Bump bockbuild

[profiler] Do the signal ack after checking that the thread is detached.

Otherwise we could crash in mono_thread_info_current ().

[aot] fixes temp-path argument only working when using LLVM compiler. (#5099)

Signed-off-by: Chris Lapa <chris@lapa.com.au>

[packaging/msbuild] Bump to get new changes on xplat-master

[packaging/msbuild] Update commit hash to point to xplat-master HEAD (#5100)

[sgen] Performance improvement for ms_block_size computations

This reduces regressions for switching from MS_BLOCK_SIZE compile time constant
to ms_block_size variable by 85% on the tests where this was showing up. In the
future we should implement a block allocator, which would, amongst other benefits,
enable us to remove the block-size/page-size dependency.

[sgen] Fix MS block size calculation.

The block size must be a multiple of the system page size. Page size simply
cannot be determined reliably at compile time, so we must obtain the page size
through a system call on startup and use it as the block size. We still enforce
a minimum block size of 16kb on systems where the page size is smaller than
that.

The old code hardcoded page sizes for certain architectures with large pages.
THe problem is that those architectures don't necessarily have to have large
pages and on systems where this wasn't the case, we'd end up with a block size
much larger than needed.

Rebase of @alexrp's work.

Merge pull request #5095 from kumpera/fix_fullaot

[aot] Fix the name of the mono_gc_wbarrier_range_copy icall. FullAOT with direct calls requires it.

[arm] Handle large offsets in OP_SEQ_POINT. Fixes #57476. (#5094)

Merge pull request #5090 from alexrp/profiler-class-unload-removal

[profiler] Remove class unload events.

[profiler] Remove class unload events.

These are never actually emitted by the profiler API because they make no sense
in the context of Mono's metadata layer.

Merge pull request #5089 from alexrp/profiler-deadlock-fix

[profiler] Fix a deadlock due to recursive use of the buffer lock.

[aot] Fix the name of the mono_gc_wbarrier_range_copy icall. FullAOT with direct calls requires it.

Merge pull request #5092 from alexrp/mini-coverage-increment

[mini] Use an atomic add for the code coverage counter when possible.

Docs: added missing ) to RelaxngPattern snippet (#5093)

[mini] Use an atomic add for the code coverage counter when possible.

[runtime] Add a few missing error_init () calls to the mono_custom_attrs_from_... functions. Fixes #57488. (#5076)

[sre] Fix the signature of the create_dynamic_method () icall, make it static since the C implementation only takes two parameters, this only caused problems if the icall set the error, since it overwrite the DynamicMethod object instead. Fixes #57301. (#5081)

Merge pull request #5091 from alexrp/mini-enter-leave-cleanup

[mini] Remove codegen special cases around MONO_PROFILE_ENTER_LEAVE.

[mini] Remove codegen special cases around MONO_PROFILE_ENTER_LEAVE.

Enter/leave instrumentation is now done with icalls at the IR level, so these
special cases are no longer needed.

[profiler] Fix a deadlock due to recursive use of the buffer lock.

The lock no longer supports recursion as of 10ba222234b0ff23bfbe29c6f2dfd7673ec2ca7f.
Not that it ever really did, since the old implementation was buggy.

[threads] Enable blocking transition with MONO_ENABLE_BLOCKING_TRANSITION env variable (#5087)

[utils/mmap] Prefer sysconf () over getpagesize () in mono_pagesize ().

The former is async-signal-safe, while the latter makes no guarantees and also
isn't actually portable.

Merge pull request #5079 from lambdageek/dev-setvalue

[coop handles] Implement various SetValue methods

Merge pull request #5082 from kumpera/fix-ro-fs-file-delete

[io-layer] When deleting a file, check for EROFS and verify if the file exists. Fixes #57629.

[runtime] Update comment with link to Linux Kernel fix.

[interp] disable context-static.exe on CI as it suffers from stack usage issue

[interp] fix BARRIER_IF_VOLATILE usage

[interp] Handle the `no.` prefix instruction as a no-op.

[interp] Add support for the `volatile.` prefix instruction.

[interp] Add MINT_MONO_MEMORY_BARRIER.

[interp] Fix oplength of MINT_MONO_TLS.

[interp] Update comment to clarify meaning of oplength.

[mcs] Add missing vector cloning of definite assignment on-true. Fixes #57505

Merge pull request #5072 from alexrp/profiler-stability

[profiler] A bunch of profiler bug/stability fixes.

[mcs] Fixes regression introduced by a0fc3f4dcf571edc997f53066c09671650e96670 to consider user set values. Fixes #57242

[io-layer] When deleting a file, check for EROFS and verify if the file exists. Fixes #57629.

Linux has this particular behavior of returning ENOFS when unlinking from an read only FS.
Even if the file doesn't exists! Darwin does the sane thing and return ENOENT in this case.

To workaround this, if unlink returns EROFS, we stat the file and deal with that.

Bump roslyn to 2.3 beta 2

[runtime] Use coop handles for System.Array.SetValue

[runtime] Use coop handles for System.Array.SetValueImpl

[runtime] Make array_set_value_impl single-exit

[runtime] Don't call icall ves_icall_System_Array_SetValueImpl directly

add a helper function array_set_value_impl and call that from elsewhere in
native code.

[runtime] Add mono_error_set_invalid_cast ()

[runtime] Use coop handles for System.Reflection.MonoField.SetValueInternal

Bump corefx

[runtime] Add mono_nullable_init_from_handle

[msvc] Update csproj files (#5075)

[coop handles] Add MONO_HANDLE_SET_FIELD_REF macro

Set a field in an object when you have a MonoClassField*

[coop handles] Add mono_object_handle_pin_unbox

[bcl tests] do not attempt to delete non-existing files

This fixes an issue we see on the BCL test suite when run in the context
of Xamarin Android:

```
I/mono-stdout( 2534):     Certificate1 [FAIL] : TearDown : System.UnauthorizedAccessException : Access to the path "temp.b64" is denied.
```

It seems like those files aren't generated anymore.  Not sure why this
starts to be a problem now.

[profiler] Remove an unnecessary check that could hide potential bugs.

[acceptance-tests] Update the profiler-stress test runner for the profiler's new argument style.

This allows us to test a lot more combinations than previously.

[profiler] Fix 'no' argument parsing.

[profiler] Implement a new buffer reader/writer lock algorithm.

See the comment + assertions to get an idea of how it works.

[profiler] Require an ack of the previous sampling signal before sending another.

This avoids situations where we could end up filling the signal queue
completely, leading to the undocumented EAGAIN return value from pthread_kill,
which breaks ~everything in the STW/GC code because it thinks an error return
value from pthread_kill means the thread has exited.

The actual reason why this happened is that when a thread is suspended, we mask
all signals except the restart signal for that thread. This means that a
program with a large number of threads or where GC collections could take
longer than usual would end up with tens of thousands of sampling signals
queued up, eventually reaching the limit of the user-level signal queue (see
`ulimit -i` and `RLIMIT_SIGPENDING`).

We only do this on systems where we use real time signals. Non-RT signals do
not have this problem as the kernel will throttle them.

[utils/threads] Export small ID functions as part of the API.

These will be used in the log profiler.

Also, make mono_thread_info_register_small_id () only allocate a new small ID
if one isn't already allocated for the current thread. This is necessary so
that we can allocate a small ID for a thread and not have it be overwritten
if/when that thread is later attached probably (which also results in this
function being called).

[utils/threads] Print an error and abort if pthread_kill returns an error that isn't ESRCH.

[profiler] Remove a FIXME that likely won't be addressed now.

[profiler] Ensure that buffer size is adjusted to whatever an event actually needs.

Very large events could previously result in potential overflows because
buffers always had the same size (BUFFER_SIZE, 64k).

[profiler] Remove the buffer size hack in ensure_logbuf_unsafe ().

This is blatantly hiding potential bugs. This kind of code is not okay.

[profiler] Attach threads automatically in all callbacks.

This hardens the profiler against cases where an unattached thread causes some
event callback to be invoked. This is particularly relevant when embedding.

[profiler] Flush the dumper thread every second.

If a program is idle, we likely won't get enough samples to prompt the dumper
thread to send its log buffer to the writer thread. This can make it seem like
the profiler isn't working when it actually is. So, just send the dumper
thread's log buffer every second to make sure data is delivered in a timely
fashion.

[profiler] Take the reader lock when modifying the thread-local method list.

It can be modified by other threads during sync points.

[threads] Store MonoInternalThread in MonoThreadInfo for use when detaching (#5058)

* [threads] Make mono_thread_attach_full static

* [threads] Remove  prefix on MonoThreadInfoCallbacks callbacks

* [threads] Rename mono_threads*_init functions to mono_thread_info*_init

* [threads] Remove dead method mono_runtime_is_critical_method

* [threads] Move MonoThreadInfoCallbacks to metadata/threads

* [threads] Store MonoInternalThread in MonoThreadInfo for use when detaching

When the native thread would exit, the TLS keys are destroyed. There is a TLS destructor for MonoThreadInfo, but not for MonoInternalThread. If MonoInternalThread is destroyed before MonoThreadInfo, then MonoThreadInfoCallbacks.thread_detach wouldn't successfully detach the current MonoInternalThread as mono_thread_internal_current would return NULL. This would lead to MonoInternalThread still laying around, while their corresponding MonoThreadInfo has already been destroyed. This fixes this problem, since we now rely on MonoThreadInfo to store the MonoInternalThread for detach.

Merge pull request #5070 from alexrp/profiler-version-bump

[profiler] Bump log profiler version to 2.0 due to breaking argument changes.

[profiler] Bump log profiler version to 2.0 due to breaking argument changes.

Merge pull request #5062 from alexrp/profiler-library

[profiler] Introduce the Mono.Profiler.Log library. (take 2)

[profiler] Introduce the Mono.Profiler.Log library. API still unstable.

Merge pull request #5063 from alexrp/profiler-format-changes

[profiler] A few small format changes