Merge pull request #896 from echampet/webresource

[Fix] ReusableHashAlgorithm doesn't set the Key of the ValidationAlgorit...

[bcl] Avoid printing debug messages to stdout in the default trace listener when a debugger is connected. Fixes #13538.

[amd64] Avoid passing the pc/pc offset to the throw corlib exception trampoline in llvm mode, instead simply use the return address pushed by the call itself. In JIT mode, this is needed because the throw code is generated out of line so its not inside the try range, but in llvm mode, the try-range encompasses the call.

[jit] Pass the type token index instead of the full type token to the throw corlib exceptions functions on most platforms to reduce the size of the generated code.

Merge pull request #2798 from BrzVlad/fix-sgen-timestamp

[sgen] Fix sgen binary protocol timestamp

TV_ELAPSED was cast to long instead of gint64, overflowing on some platforms.

[bcl] Avoid 'sed: net_4_x_System.ServiceModel_test.dll.excludes: No such file or directory' warnings during the build.

[arm] In the llvm throw corlib exception trampoline, use the normal caller ip instead of the argument passed in by the caller since that might not point directly to the call instruction. Fixes #39420.

[runtime] Make an argument name more descriptive.

Merge pull request #2797 from kumpera/android_krait_bug

[android] Fix Krait CPU bug on Android.

Merge pull request #2785 from alexanderkyte/cached_info_leak

[runtime] Free cached_info tables on cleanup

[SRE] Fix compile warning in encode_cattr_value

Landed mono_object_isinst_checked PR in a way that broke encode_cattr_value.

[sgen] Fix sgen binary protocol timestamp

TV_ELAPSED was cast to long instead of gint64, overflowing on some platforms.

Merge pull request #2621 from BrzVlad/feature-conc-preclean

[sgen] Concurrent precleaning of card table

Merge pull request #2795 from lambdageek/dev/marshal-no-raise

[marshal] Don't raise an exception, just assert

Merge pull request #2770 from lambdageek/dev/monoerror-object_isinst_mbyref

[runtime] MonoError-ize mono_object_isinst{,_mbyref}

Also mark them external only.  Runtime should use mono_object_isinst_mbyref_checked and mono_object_isinst_checked

[aot] Emit a .subsections_via_symbols directive on arm.

[aot] Fix a memory leak.

[runtime] Avoid setting klass->instance_size if its already set, this can happen when loading cached class info from an AOT image. This could lead to random crashes/asserts since klass->instance_size is directly modified while it is computed.

[runtime] Fix a memory leak in ves_icall_Type_GetNestedTypes ().

Merge pull request #2779 from akoeplinger/fix-socket-closed

[System] Use Monitor.Wait/Pulse in SafeSocketHandle instead of Thread.Sleep

[System] Use Monitor.Wait/Pulse in SafeSocketHandle instead of Thread.Sleep

We are quite frequently seeing an error on the ARM workers on Jenkins:

```
Test Case Failures:
1) MonoTests.System.Net.Sockets.SocketTest.SendAsyncFile : System.Exception : Could not abort registered blocking threads before closing socket.
at System.Net.Sockets.SafeSocketHandle.ReleaseHandle () [0x000be] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/mcs/class/System/System.Net.Sockets/SafeSocketHandle.cs:77
at System.Runtime.InteropServices.SafeHandle.DangerousReleaseInternal (Boolean dispose) <0x42cb0050 + 0x0020b> in <filename unknown>:0
at System.Runtime.InteropServices.SafeHandle.InternalDispose () <0x446ba758 + 0x00023> in <filename unknown>:0
at System.Runtime.InteropServices.SafeHandle.Dispose (Boolean disposing) [0x00000] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/external/referencesource/mscorlib/system/runtime/remoting/callcontext.cs:804
at System.Runtime.InteropServices.SafeHandle.Dispose () [0x00000] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/external/referencesource/mscorlib/system/runtime/remoting/callcontext.cs:795
at System.Net.Sockets.Socket.Dispose (Boolean disposing) [0x00047] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/mcs/class/System/System.Net.Sockets/Socket.cs:3321
at System.Net.Sockets.Socket.Dispose () [0x00000] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/mcs/class/System/System.Net.Sockets/Socket.cs:3327
at System.Net.Sockets.Socket.Close () [0x00007] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/mcs/class/System/System.Net.Sockets/Socket.cs:3256
at MonoTests.System.Net.Sockets.SocketTest.SendAsyncFile () [0x000e7] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/mcs/class/System/Test/System.Net.Sockets/SocketTest.cs:4331
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x0003d] in /media/ssd/jenkins/workspace/test-mono-mainline/label/debian-armhf/mcs/class/corlib/System.Reflection/StrongNameKeyPair.cs:103
```

Let's see if implementing the suggestion from https://github.com/mono/mono/pull/1612/files#r26393224
fixes this. I couldn't reproduce the test failure even on the same ARM machine so this is a bit of
an opportunistic change, but it feels like the more "correct" solution anyway.

I decided to inline AbortRegisteredThreads as at the only place we're calling it we already know
blocking_threads can't be null and acquired the lock.

Merge pull request #2789 from lambdageek/dev/monoerror-threading

[threadpool-ms] MonoError-ize

1. Convert icalls that throw `NotImplementedException` to do it via mono_error_set_pending_exception
2. Add MonoError outargs to `mono_threadpool_ms_enqueue_work_item` and `mono_threadpool_ms_begin_invoke`

[runtime] MonoError-ize mono_object_isinst

Mark it external only. Runtime should use mono_object_isinst_checked.

[threadpool-ms] MonoError-ize mono_threadpool_ms_begin_invoke

Merge pull request #2748 from lambdageek/dev/monoerror-mono_value_box

[runtime] Make mono_value_box use MonoError

Mark it external only.  Runtime should use mono_value_box_checked.

[runtime] MonoError-ize mono_object_isinst_mbyref

Also mark it external only.  Runtime should use mono_object_isinst_mbyref_checked

[MonoError] change mono_error_set_pending_exception to return a bool

Return TRUE if pending exception was set, FALSE otherwise.

The idea is that you can write:
  if (mono_error_set_pending_exception (error)) {
    /* cleanup */
    return;
  }

[android] Fix Krait CPU bug on Android.

Krait cpus don't ignore thumb ITT state in ARM mode (it's unspecified CPU behavior).
The linux kernel then didn't bother resetting that state when landing an ARM signal handler.

The end result was that Krait cpus would skip the first few instructions of the signal handler
and things would explode in the most amazing possible ways.

Patch by Mark Probst <mark.probst@gmail.com>.

[marshal] Don't raise an exception, just assert

Don't raise deep in the marshalling code, just assert.

(Raising means we'd have to convert the code to use MonoError and then
thread it out through all of the marshalling code resulting in a lot of
code churn)

[runtime] MonoError-ize mono_nullable_box

[runtime] Make mono_value_box use MonoError

Mark it external only.  Runtime should use mono_value_box_checked.

[threadpool-ms] MonoError-ize mono_threadpool_ms_enqueue_work_item

Take a MonoError outarg; return boolean on success/failure.

[threadpool-ms] MonoError-ize some not implemented icalls.

1. ves_icall_System_Threading_ThreadPool_ReportThreadStatus
2. ves_icall_System_Threading_ThreadPool_PostQueuedCompletionStatus

Merge pull request #2791 from lambdageek/dev/monoerror-cleanup-deadcode

[runtime] Remove dead code and don't throw in embedding API functions

[sgen] Add option for enabling/disabling precleaning

[sgen] Preclean mod union entries before the finishing pause

When the concurrent mark ends, we have another concurrent phase where we scan the cardtable trying to clean as many cards as possible. The way we achieve this is by first copying all the mod union cards (for each block/los_obj) to a separate temporary card_table. When scanning we will only scan these cards. Therefore we will first preclean these cards in the actual mod_union card_tables, followed by a full memory barrier (which assures us that, when scanning, we will observe the changes that led to the corresponding card marks) and then by a normal mod_union scan, which can remark some of these cards, without any races with the preclean phase.

[sgen] Move assert to the right place

[runtime] Enclose define parameters in parentheses

[sgen] Add separate scan function for handling a single pointer entry

This way we avoid duplicating cardtable marking code, making sure we have consistent code.

For example, when scanning los objects in the mod union scan step, we didn't check whether the object is cemented or not, leading to additional unnecessary global remsets for the next nursery collection.

[sgen] Move scan vtype with the other object ops

[sgen] Simplify defines for object ops

[sgen] Allow read only access on block list without taking GC_LOCK

Still, block list should not be accessed while sweep is running.

[sgen] Allow read only access of los object list without taking GC_LOCK

Still, object lists should not be accessed while sweep is running.

[sgen] Extract the handle array code for reuse

We add a new data structure which contains a bucket of entries, exponentially increasing in sizes. The structure is customizable through a few callbacks.

Setting is_slot_set_func enables us to also lookup empty slots, instead of only adding entries at the end of the list.

Setting set_slot_func enables the use of a customized function for setting the slot. If this function is not set, the slots and array fields will be set directly, therefore the structure won't be thread safe.

[sgen] Don't prefetch unused memory when scanning the card table

When doing the mod union scan, we use the block's cardtable (instead of the global one), which we also cannot prefetch, since that would mean dereferencing the block info.

Merge pull request #2793 from radical/fix-mcs-args

[mcs] Accept and ignore command line args supported by csc that we don't

[runtime] mark mono_domain_unload external only

Runtime should use mono_domain_try_unload

[runtime] Remove CHECK_MONOTYPE macro

It's dead code.

[runtime] Delete mono_assert and mono_assert_not_reached

Runtime should use g_assert and g_assert_not_reached or set a MonoError,
as needed.

Also remove REAL_PRINT_REG dead macros from mini-x86.c and mini-amd64.c

[runtime] Change mono_object_castclass_mbyref not to throw

Mark the function external only. (There are no uses inside the runtime).

Change behavior to return NULL if unable to cast, instead of throwing.

Merge pull request #2790 from lambdageek/dev/monoerror-corlib-execeptions

[MonoError] Helpers for some common corlib exceptions

[arm] Fix support for soft breakpoints.

[mcs] Accept and ignore command line args supported by csc that we don't

support:

/analyzer
/appconfig
/baseaddress
/deterministic
/errorendlocation
/errorlog
/features
/link
/moduleassemblyname
/pathmap
/pdb
/preferreduilang
/publicsign
/reportanalyzer
/ruleset
/sqmsessionguid

[runtime] Make the appdomain-unload.exe test more verbose.

[aot] Reenable the emission of line number info using GAS .file/.loc directives, recent versions of clang seems to support it.

[jit] Fix the dwarf info structure when there is no line number table.

[jit] Remove the 'appending' support from the dwarf writer, its no longer used.

[jit] Get rid of the create_delegate_trampoline_data () helper function which only has one caller.

Merge pull request #2768 from lambdageek/dev/monoerror-cominterop

[com] MonoError-ize cominterop.c

Merge pull request #2663 from esdrubal/islocal

[System] Fixes HttpListenerRequest.IsLocal.

Create xammac_net_4_5 variant of System.Runtime.Remoting

Trigger a rebuild of Mono.Security

[Mono.Security]: MonoTlsProvider.InvokeSystemCertificateValidator() now takes 'ref X509Chain'.

[System]: Add private 'X500DistinguishedName' API to store the canonical encoding and use it for fast comparison.

[System]: SystemCertificateValidator.CheckUsage(): lazily convert to X509Certificate2 if needed.

[System]: Make 'X509Certificate2' and 'X509Chain' lazily-initialized.

Add new internal 'X509Certificate2Impl' and 'X509ChainImpl' classes similar to
'X509CertificateImpl' in corlib and only initialize them when actually needed.

[corlib]: Cleanup the internal 'X509CertificateImpl' API.

[ppdb] Disable the metadata verifier for ppdb tables for now. Fixes #39631.

[MonoError] Use mono_error_set_execution_engine helper

[MonoError] Use mono_error_set_not_supported helper

[MonoError] Use mono_error_set_not_implemented helper

[MonoError] add helpers for some corlib exceptions

System.ExecutionEngineException
System.NotSupportedException
System.NotImplementedException

[man] Remove documentation for the nonexistent MONO_ENABLE_SHM environment variable.

Merge pull request #2784 from mhutch/msbuild-fixes

Fix regressions in Microsoft.Build.dll

[cil-stringreplacer] Use Cecil as reference dependency

[cil-stringreplacer] Update debug symbol

Merge pull request #2783 from alexanderkyte/strdup_memory_leaks

[runtime] Fix memory leaks related to short-lived strings

Merge pull request #2775 from lambdageek/dev/monoerror-reflection-icalls

[SRE] MonoError-ize some icalls

* Cleanup some memory that would leak on failure; improve a few error messages
* Add MonoError to `mono_reflection_call_is_assignable_to` and `fieldbuilder_to_mono_class_field`.
* Update several SRE icalls to follow the pattern:
  ```c
    void
    the_icall (args... )
    {
       MonoError error;
       (void) the_real_work_function (args..., &error);
       mono_error_set_pending_exception (&error);
    }

    static gboolean
    the_real_work_function (args..., MonoError *error)
    {
       ... /* return TRUE on success, FALSE and set error on failure */ ...
    }
  ```
  Do this to:
        1. `mono_reflection_register_with_runtime`
        2. `mono_reflection_setup_internal_class`
        3. `mono_reflection_create_internal_class`
        4. `mono_reflection_generic_class_initialize`
        5. `mono_reflection_event_builder_get_event_info`
        6. `mono_reflection_initialize_generic_parameter`
        7. `mono_reflection_create_dynamic_method`

Merge pull request #2782 from madewokherd/process-sessionid

[System.Diagnostics] Return dummy value from Process.SessionId.

[System.Diagnostics] Return dummy value from Process.SessionId.

[sgen] Switch the default GC bridge to the new mode.

[Microsoft.Build] Shut down single-use build manager

Merge pull request #2764 from esdrubal/fileinfo_tostring

[corlib] Fix FileInfo.ToString after MoveTo

Merge pull request #2534 from mhutch/install-api-html

Install mono-api-html

We install its counterpart mono-api-info already

[runtime] Free cached_info tables on cleanup

Install mono-api-html

We install its counterpart mono-api-info already

[Microsoft.Build] Set IsBackground on builder thread

It should not prevent the process from terminating. Fixes
issue where the thread is not shut down if ProjectInstances
are built directly instead of via the build manager.

[Microsoft.Build] Fix exception disposing BuildManager

[runtime] Remove string memory leak related to basename

[runtime] Fix reassignment over string-pointing var without freeing

Merge pull request #2778 from xmcclure/checked-build-updates-2

Fix patch merge error in PR#2753, was breaking checked build

Fix patch merge error in PR#2753, was breaking checked build

Add note about acceptance-tests to top level README

Fix "make dist"

[gsharedvt] Add a test for #39528.

[runtime] Remove some dead test code added by c736aba5c94c20e6faad6b736f8e3a834e26d95f.

[runtime] Implement support for passing icall methods as delegates in pinvoke. Fixes #39347.

[runtime] Remove an mistaken assert. Fixes #39568.

[jit] Remove an mistaken assert. Fixes #39746.

Merge pull request #2777 from xmcclure/win-makefile-aotcheck

Make `make fullaotcheck` in mono/mini runnable on Windows

Run with
MONO_EXECUTABLE=/path/to/msvc/mono.exe LIST_SEPARATOR=";"
This assumes you are running the test harness in Cygwin and you built mono with MSVC. Also, the test does not currently succeed.
Being able to run the test is a zeroth step toward making the test pass.