Merge pull request #2542 from akoeplinger/remove-changelog

Remove ChangeLog files from the repo

Merge pull request #2554 from ludovic-henry/coop-blocking-interrupt

[coop] Use signals for blocking syscall interrupt

Merge pull request #2559 from lewurm/hazard-pointer-clear

[hazard pointers] add write barrier on clear as well

[binary-reference-assemblies] Add a symlink from 4.0/mscorlib.dll to 4.0-api/mscorlib.dll

Some programs like NUnit check if a file exists there and break otherwise.

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

[docs] remove a deleted file from EXTRA_DIST

Revert "[rules.make] to support 'make dist' on MacOS"

This reverts commit a234d21471806a4ab9e5b5ae42ec88cdbda1e144.

It isn't necessary after f1dad87178085f688afbcddcb77cf70d193ed995.

[Runtime Documentation] More work on runtime documentation

[hazard pointers] add write barrier on clear as well

[coop] Use signals for blocking syscall interrupt

[Documentation] Document some of Mono's embedding API, improve the style

Remove stale files

[rules.make] to support 'make dist' on MacOS

Document Coop mode in README

[test] Use using TimerTest

[tests] Use explicit delegates to get more information from stacktrace

[proc] Fix potential buffer overflow

Merge pull request #2530 from lambdageek/monoerror-mono_string_new

[runtime] Use MonoError for mono_string_new

We are refraining from changing runtime callers of `mono_string_new` to `mono_string_new_checked` in one commit. String creation is used in several hundred places and threading error handling through all the callers is very intrusive. Additionally, AFAIK the only error condition from mono_string_new is out-of-memory. In a lot of cases there's no way to meaningfully recover.

We should revisit individual callers when it makes sense, but just mechanically updating everywhere doesn't get us a lot of benefit.

Remove from the documentation non-public APIs, or move to files that flag them as internal

Merge pull request #2524 from lambdageek/monoerror-mono_string_new_utf16_checked

[runtime] Use MonoError for mono_string_new_utf16

Updated the runtime template documentation with all new APIs in Mono

Merge pull request #2518 from lambdageek/monoerror-mono_method_get_object

[runtime] Use MonoError for mono_method_get_object

Depends on #2532

Issues:

- [x] ~~depends on #2486~~
- [x] ~~Should use `mono_object_new_checked` #2532~~
- [x] ~~Should use `mono_string_new_checked`~~ (`mono_string_new` will assert instead of raising an exception - #2530)

[llvmonly] Implement support for stack walks if _Unwind_Backtrace () is available.

[runtime] Switch to mono_method_get_object_checked

Mark mono_method_get_object external only.

[runtime] Use MonoError for mono_method_get_object

[reflection] Fix build breakage

Merge pull request #2558 from lewurm/thread-suspension-race

[threads] avoid race between setting the state flag and suspension

there's a short window where we release the lock and another thread
could modify the state.  That is what we saw on the ARM machines for a
while, e.g. in `subthread-exit.cs` there's a race between
`mono_thread_execute_interruption` and `mono_thread_suspend_all_other_threads`:

> Thread 2 (Thread 0x429ff430 (LWP 5189)):  // SUB THREAD
> #0  0x40398e30 in nanosleep () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #1  0x0028c0ac in monoeg_g_usleep (microseconds=86650) at gdate-unix.c:53
> #2  0x0027b1d8 in suspend_sync_nolock (id=1079603792, interrupt_kernel=1) at mono-threads.c:913
> #3  0x0027b2d4 in mono_thread_info_safe_suspend_and_run (id=1079603792, interrupt_kernel=1, callback=0x1ab109 <suspend_thread_critical>, user_data=0x429fe534)
>     at mono-threads.c:935
> #4  0x001ab28a in suspend_thread_internal (thread=0x401c0120, interrupt=1) at threads.c:4807
> #5  0x001a91ec in mono_thread_suspend_all_other_threads () at threads.c:3297
> #6  0x00158d8c in ves_icall_System_Environment_Exit (result=0) at icall.c:6378
> #7  0x400e7d9c in ?? ()
> Backtrace stopped: previous frame identical to this frame (corrupt stack?)
>
> Thread 1 (Thread 0x40597250 (LWP 5186)):  // MAIN THREAD
> #0  0x4039a5f4 in __libc_do_syscall () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #1  0x4039796e in do_futex_wait () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #2  0x403979da in sem_wait@@GLIBC_2.4 () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #3  0x002788d8 in mono_os_sem_wait (sem=0x388248, flags=MONO_SEM_FLAGS_NONE) at ../../mono/utils/mono-os-semaphore.h:163
> #4  0x00279722 in mono_thread_info_wait_for_resume (info=0x388210) at mono-threads.c:144
> #5  0x0027ad5e in mono_thread_info_end_self_suspend () at mono-threads.c:700
> #6  0x001ab2d2 in self_suspend_internal (thread=0x401c0120) at threads.c:4822
> #7  0x001aab0a in mono_thread_execute_interruption () at threads.c:4337
> #8  0x001a6a82 in ves_icall_System_Threading_Thread_Sleep_internal (ms=1200) at threads.c:1217
> #9  0x400da75c in ?? ()
> Backtrace stopped: previous frame identical to this frame (corrupt stack?)

Merge pull request #2563 from OffBase/master

Fix stream leak when failing to open a zip file

[runtime] Use MonoError for mono_string_new

[runtime] Switch to mono_string_new_utf16_checked

Merge pull request #2536 from lambdageek/monoerror-mono_field_get_object

[runtime] Use MonoError for mono_field_get_object

Merge pull request #2532 from lambdageek/monoerror-mono_object_new

[runtime] Use MonoError for mono_object_new

[llvmonly] Implement limited support for stack walks so new StackTrace () works.

Fix stream leak in all cases

[corlib] Fixes Environment version for .net 4.6

Fix stream leak when failing to open a zip file

Merge pull request #2556 from BrzVlad/feature-sgen-stats

[sgen] sgen stats

Fix the build.

[llvm] Fix the loaded llvm build broken by 204f2985d7c9835e8cefaf3439d51ccaf0d3d2b0.

Merge pull request #2557 from ludovic-henry/monoerror-return-macro

[MonoError] Add error checking macro

fixup! [MonoError] Add error checking macro

[threads] avoid race between setting the state flag and suspension

there's a short window where we release the lock and another thread
could modify the state.  That is what we saw on the ARM machines for a
while, e.g. in `subthread-exit.cs` there's a race between
`mono_thread_execute_interruption` and `mono_thread_suspend_all_other_threads`:

> Thread 2 (Thread 0x429ff430 (LWP 5189)):  // SUB THREAD
> #0  0x40398e30 in nanosleep () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #1  0x0028c0ac in monoeg_g_usleep (microseconds=86650) at gdate-unix.c:53
> #2  0x0027b1d8 in suspend_sync_nolock (id=1079603792, interrupt_kernel=1) at mono-threads.c:913
> #3  0x0027b2d4 in mono_thread_info_safe_suspend_and_run (id=1079603792, interrupt_kernel=1, callback=0x1ab109 <suspend_thread_critical>, user_data=0x429fe534)
>     at mono-threads.c:935
> #4  0x001ab28a in suspend_thread_internal (thread=0x401c0120, interrupt=1) at threads.c:4807
> #5  0x001a91ec in mono_thread_suspend_all_other_threads () at threads.c:3297
> #6  0x00158d8c in ves_icall_System_Environment_Exit (result=0) at icall.c:6378
> #7  0x400e7d9c in ?? ()
> Backtrace stopped: previous frame identical to this frame (corrupt stack?)
>
> Thread 1 (Thread 0x40597250 (LWP 5186)):  // MAIN THREAD
> #0  0x4039a5f4 in __libc_do_syscall () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #1  0x4039796e in do_futex_wait () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #2  0x403979da in sem_wait@@GLIBC_2.4 () from /lib/arm-linux-gnueabihf/libpthread.so.0
> #3  0x002788d8 in mono_os_sem_wait (sem=0x388248, flags=MONO_SEM_FLAGS_NONE) at ../../mono/utils/mono-os-semaphore.h:163
> #4  0x00279722 in mono_thread_info_wait_for_resume (info=0x388210) at mono-threads.c:144
> #5  0x0027ad5e in mono_thread_info_end_self_suspend () at mono-threads.c:700
> #6  0x001ab2d2 in self_suspend_internal (thread=0x401c0120) at threads.c:4822
> #7  0x001aab0a in mono_thread_execute_interruption () at threads.c:4337
> #8  0x001a6a82 in ves_icall_System_Threading_Thread_Sleep_internal (ms=1200) at threads.c:1217
> #9  0x400da75c in ?? ()
> Backtrace stopped: previous frame identical to this frame (corrupt stack?)

[error] Use error_return_(val_)if_nok at some places

[MonoError] Add error checking macro

[binary-reference-assemblies] Make sure only .dll files are included in the wildcard

The wildcard `$(wildcard ../../../external/binary-reference-assemblies/v4.5/*)` would include the directory entry `../../../external/binary-reference-assemblies/v4.5/Facades/` which returns an error during "make dist".

Fixing this by including only the .dll files in the wildcards.

[binary-reference-assemblies] Include 4.5 Facades in tarball

[Mono.Security.Providers.OldTls]: Don't build on mobile.

(cherry picked from commit f379b7df517b18cea025cb3f53e144bc0e851b9f)

[sgen] Add binary protocol entries for evacuation

This way we can detect which collections are doing evacuation, which can be useful both for debugging and performance reasons.

[sgen] Remove fixed FIXME comment

[sgen] Add binary protocol entry for worker finish

This will allow us to detect actual concurrent M&S end and whether the concurrent M&S was forced to finish.

[sgen] Add binary protocol entries for finishing the gray stack

These entries have a timestamp associated with them, allowing us to gather information about time spent during each collection.

[sgen] Add binary protocol entries for cardtable scanning

These entries have a timestamp associated with them, allowing us to gather information about time spent during each collection.

[sgen] Fix HEAVY_STATISTICS build warning

[Makefile] fix make dist on libgc

[jit] Avoid merging some bblocks which are not after each other when inlining since it can cause the bblock between them to be ignored later. Fixes #38250.

[amd64] Fix the stack alignment check.

Merge pull request #2551 from ludovic-henry/coop-products

[coop] Add support for entering/exiting GC safe/unsafe region for products

[coop] Add support for entering/exiting GC safe/unsafe region for products

[coop] Ensure we exit blocking mode when invoking managed code

mono_runtime_invoke might be called by P/Invoked code, which is in blocking mode. Not switching back to running can lead to a crash when hitting a safepoint.

[arm] Fix a code generation problem when a virtual call is made with a large offset, the offset generation code would overwrite a source register. Fixes #27303.

[aot] Emit gsharedvt wrappers only in llvmonly mode.

[mcs] Filter out nested private types of forwarded types

Merge pull request #2538 from ludovic-henry/fix-sgen-38012

[sgen] Fix register scanning on ARM

This bug would trigger a use after sweep in System.Threading.Tasks.Task.FinishContinuations:3621.

The issue would arise on ARM, as `MonoContext` and `ARCH_NUM_REGS` are defined as followed :

```
typedef struct {
  mgreg_t pc;
  mgreg_t regs [16];
  double fregs [16];
  mgreg_t cpsr;
} MonoContext;

\#define ARCH_NUM_REGS 14
```

As you can see, the MonoContext structure is bigger than 14 words, and it does not even covers the last 3 values of `regs`. By using pointer arithmetic, we ensure that we do not miss some parts of the `MonoContext` structure.

The observed behaviour in System.Threading.Tasks.Task.FinishContinuations would be that `continuationObject` would not be marked, and thus freed; `continuationObject` would still contains the pointer to the old location, meaning it wouldn't be null. We would then try to check its type via a call to the `as` operator. This would call the `isinst` IL opcode which would load `continuationObject->vtable->klass->supertypes [...]`, which would trigger a segfault, as `supertypes` would be null.

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

Merge pull request #2541 from ludovic-henry/fix-coop-hang

[coop] Fix hang in ves_icall_System_Net_Dns_GetHostByName_internal

[binary-reference-assemblies] Install 4.5 api facades

[coop] Fix hang in ves_icall_System_Net_Dns_GetHostByName_internal

[llvm] Make mini-llvm-loader.c simpler by putting all the function pointers into a structure.

[amd64] Add support for float arguments/return values to the dyncall code.

Merge pull request #2547 from ermshiperete/Xamarin-33183

[SWF] Fix behavior when scrollbar values change in TreeView

[llvm] Remove the goto based error handling code, use explicit error checks and returns instead.

[msvc] Update order.xml and csproj files

Merge pull request #2546 from ermshiperete/libtoolize

Check for libtoolize rather than libtool

[mcs] Stop after fields initializers resolve errors

[runtime] Disable the constant-logical-operand warning.

[runtime] Make the MONO_CURRENT_ABI_ALIGNOF macros c++ safe.

[jit] Remove unused MONO_PATCH_INFO_MONITOR_ENTER/EXIT.

Merge pull request #2544 from ermshiperete/Xamarin-24641

[corlib] Fix handling of ignorable chars when using ordinal comparison

Check for libtoolize rather than libtool

Current Debian packaging puts libtool in its own package (libtool-bin)
leaving only libtoolize in the main package. The mono build process
uses libtoolize not libtool, so during autogen check for libtoolize
not libtool.

[corlib] Fix handling of ignorable chars when using ordinal comparison

* recognizes ignorable characters when CompareOptions.Ordinal is
  specified
* adds unit tests
* fixes bug 24641

Change-Id: I5102e7392fa54b5671c7f2536cb8778f847e9040

[SWF] Fix behavior when scrollbar values change in TreeView

* only call EndEdit in scrollbar value changed handlers when currently
  editing a node
* fix for bug 33183

Change-Id: I54407e00314f4bfbdc175e883199d3d1ef057911

[llvmonly] Fix stack trace calculation.

[amd64] Exclude the GSHAREDVT tests as well since they don't work yet.

[jit] Disable the DYNCALL tests on amd64 again, not all cases are supported right now.

Remove ChangeLog references from Makefile and .gitattribute

[jit] Use mono_method_get_full_name () to print out method names in a few places.

[llvmonly] Fix the fallback to no_gsharedvt_in_wrapper, which was broken by the error handling changes.

[llvmonly] Catch all exceptions in mono_jit_exec () to avoid terminating the runtime with an unhelpful libc++ assertion.

[mcs] Rework naming convention of lifted state machine fields to include scope index. Fixes #32315 (1st part)

[sgen] Fix register scanning on ARM

This bug would trigger a use after sweep in System.Threading.Tasks.Task.FinishContinuations:3621.

The issue would arise on ARM, as `MonoContext` and `ARCH_NUM_REGS` are defined as followed :

```
typedef struct {
  mgreg_t pc;
  mgreg_t regs [16];
  double fregs [16];
  mgreg_t cpsr;
} MonoContext;

\#define ARCH_NUM_REGS 14
```

As you can see, the MonoContext structure is bigger than 14 words, and it does not even covers the last 3 values of `regs`. By using pointer arithmetic, we ensure that we do not miss some parts of the `MonoContext` structure.

The observed behaviour in System.Threading.Tasks.Task.FinishContinuations would be that `continuationObject` would not be marked, and thus freed; `continuationObject` would still contains the pointer to the old location, meaning it wouldn't be null. We would then try to check its type via a call to the `as` operator. This would call the `isinst` IL opcode which would load `continuationObject->vtable->klass->supertypes [...]`, which would trigger a segfault, as `supertypes` would be null.

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

Remove ChangeLog files from the repo

They weren't updated in the last 6 years and aren't helpful anymore (e.g. by causing unrelated matches during git grep searches).

[sgen] Abort with a message on failure to initialize binary protocol

[corlib] Improve CancellationTokenSource test

Get rid of the Thread.Sleep calls that are prone to timing issues.

Merge pull request #2522 from ludovic-henry/test-socketresponder-task

[SocketResponder] Use Task instead of Thread + Use explicit delegate so it appears in stacktraces

[System.ServiceModel.Internals] Add a missing .sources file for watchOS. Fixes #38145.

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

Merge pull request #2448 from BrzVlad/feature-cprop-opt

[jit] Improve strength reduction handling within cprop

Merge pull request #2531 from esdrubal/systemweb

[System.Web] HeadersCollection read only exception

[llvmonly] Do a stack walk from mono_llvmonly_get_calling_assembly (), the runtime might support stack walks on some platforms.

[runtime] Avoid an assert if a runtime compiled without DISABLE_COM is used with an mscorlib from the mobile_static profile.

[runtime] mono_reflection_resolve_custom_attribute_data cleanup error handling

[runtime] Use MonoError for mono_field_get_object

Mark mono_field_get_object external only.
Use mono_field_get_object_checked in the runtime.

[llvmonly] Remove bind-to-runtime-version from the aot flag used to compile the assemblies, it makes testing harder.