[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

Make --aot=full able to emit executables on 64 bit Windows

- COFF does not use .local or .size
- COFF does use .def to mark functions and .data to mark data
- The seh_handler functions should not be called on full aot builds (Zoltan says they are unnecessary in full aot, and they attempt to jit)

The executables do not yet run without crashing.

Support tool-prefix in mono/mini fullaotcheck

* Support MONO_FULLAOT_ADDITIONAL_ARGS env var for make fullaotcheck, appended to --aot parameters
* To support above, permit "blank" parameters (ie trailing commas on parameter list) to --aot=
* --aot tool-prefix parameter now supported for LD as well as AS

Fix debug Windows builds in MSVC 2015

We were sometimes passing invalid input to isalnum(). On the debug version of MSVCRT this would result in an assert.

[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

[runtime] Make memory allocation for method headers consistent, stop leaking

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

[runtime] Free leaked short-lived MonoType

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

Merge pull request #2764 from esdrubal/fileinfo_tostring

[corlib] Fix FileInfo.ToString after MoveTo

[mono-symbolicate] Use SeqPointInfo instead of PInvoking.

[mono-symbolicate] Created SeqPointInfo.cs

SeqPointInfo is a c# implementation of the debug data decoding process
that is done natively by the runtime.

[corlib] Removed unused icall

icall StackFrame.GetILOffsetFromFile is no longer used by mono-symbolicate.

[System.ServiceModel] Fixes reflection call to undefined method

This fixes an issue while using WCF on android.

Fixes #36080

[System.ServiceModel] Test EndpointAddress10

The issue only occurs in monodroid.

Covers #36080

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

[runtime] Prevent leaking inflated headers by marking them transient, only free in one place.

[runtime] Make mono_basic_block_split take the header it uses

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.

[System.Net.Http] Make HttpClientTest.RequestUriAfterRedirect test more reliable

It was the only test in the class that required network access (for looking up xamarin.com in the redirect).
Replacing the redirect target with a second local HttpListener should be more resilient to network/DNS issues.

mono/mini make fullaotcheck can now be run on Windows

Test harness now uses the proper path separator and also now supports
MONO_EXECUTABLE if you built with MSVC.

Merge pull request #2753 from xmcclure/checked-build-updates

Changes to "checked build" feature

Here are some changes to checked builds I made while trying to debug some coop gc problems. The changes made are:

- Replace the checked build configure flags with a single --enable-checked-build= flag
- Runtime environment variable required for checked build asserts to run
- Report the thread transition history for more kinds of asserts
- For the `mono_threads_reset_blocking` functions, add an assert that guarantees proper nesting

In addition to this, I have documented the checked build feature as implemented here, see my pull request for the website: https://github.com/mono/website/pull/187

Overall my goals with this patch were

- Make it easier to do rapid-fire testing with the checked builds (runtime switch, default-off)
- Make the checked build implementation more internally consistent (more uniform names etc)

Notice I do make the change that instead of checked build sub-features being disabled with a blacklist, you now must enable them with a whitelist. I think this is an improvement because it makes it easier to just casually keep a checked build around  (for one thing I would have the option to ship debug builds to users with the checked build modes included but dormant) but I could be talked out of it if this change turns out to be controversial. The only thing I feel strongly about is that if the configure flag is changed back to a blacklist the environment variable should switch to a blacklist also.

If we accept this change the CI bots will need to adopt the new configure flag and environment variable.

Merge pull request #2746 from lambdageek/dev/monoerror-mono_reflection_create_runtime_class

[System.Reflection.Emit] MonoError in mono_reflection_create_runtime_class

Depends on #2745
Bubble MonoError up to `mono_reflection_create_runtime_class`.  Use `mono_error_set_pending_exception` instead of `mono_error_raise_exception` And try to release resources on the way out.

[SRE] Refactor a few icalls

The icalls follow the pattern:
    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

[SRE] Some cleanups in mono_method_body_get_object_checked

Don't leak method header on failure.

[SRE] MonoError for fieldbuilder_to_mono_class_field

Add mono_threads_reset_blocking nesting checks to gc checked mode

Rename checked build assertion_fail to mono_fatal_with_history, use in more places

Merge pull request #2772 from WeiHeCN/master

Update mini-llvm.c

These header files are already included in mini-llvm-cpp.h.

Update mini-llvm.c

[reflection] MonoError for mono_reflection_call_is_assignable_to

[SRE] MonoError for mono_reflection_create_runtime_class

1. Don't raise while holding locks.

2. Change remove_instantiations_of_and_ensure_contents (called via
  g_hash_table_foreach_remove) to populate a MonoError instead of
  throwing.  If there are multiple errors, we just stash the first one
  in user_data and clean up the rest.

3. Use mono_erro_set_pending_exception instead of
  mono_error_raise_exception since it's an icall.

Merge pull request #2745 from lambdageek/dev/monoerror-sre-internals

[System.Reflection.Emit] Propagate MonoError through some more static functions

Some externally visible changes include:
* Bubble MonoError up to one more SRE icall: `System.Reflection.Emit.CustomAttributeBuilder.GetBlob()`
* `fix_partial_generic_class` and `ensure_generic_class_runtime_vtable` now expect their callers to set the class failure state which they do under slightly more circumstances now than previously (now: whenever there's an error set from the above functions; before: only when we had trouble initializing the generic type definition.)
* Previous API for `mono_reflection_bind_generic_parameters` didn't make it clear whether NULL was a valid return value or not.  Now when we return NULL we always set the `error` out-arg. So non-NULL is success and NULL is failure.

[reflection] MonoError for mono_marshal_spec_from_builder