[packaging] Fix prerequisite in Windows script

Use CoreFX sources for System.Drawing.Text (#5036)

Use CoreFX sources for System.Drawing.Imaging (#5035)

* Use CoreFX sources for System.Drawing.Imaging

* Bump API snapshot

Use source from CoreFX for System.Drawing.Drawing2D (#5034)

[packaging] Remove msiexec prerequisite from Windows script

We're using lessmsi now.

[packaging] Exit Windows packaging script on error

Also quote passed in mdk path.

[metadata] s/HAVE_ONLINE_VES/ENABLE_ILGEN/g

[configure] move HAVE_ONLINE_VES away from autotools to a header file

Bump corefx

[Mono.Security] Fix typo in CryptographicException message

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

[MacSDK] Remove quotes in nuget wrapper script

None of the other wrapper scripts quote the Mono path and this
broke the Windows packaging script which extracts nuget from the
Mac package.

Revert "[exceptions] only do SIGSEGV related actions for SIGSEGVs in mono_handle_native_crash"

This reverts commit a27bc024f1732edb6b1edddd96d6c90f4a2983f8.

We never want the crash handler to be recursive.

[aot] fix compilation for targets that do not set MONO_ARCH_HAVE_FULL_AOT_TRAMPOLINES (#5017)

Merge pull request #5011 from kumpera/quiet-mode

Don't emit profiler events we don't need to.

[profiler] Don't emit sync point on restart the world unless gc moves are enabled.

[profiler] Add + prefix to file names to append the pid. This makes enabling it with env vars doable.

Bump bockbuild (#5023)

[ci] Get rid of remaining usages of WORKSPACE env var

Most of it was already removed in ea519ada8c38cae67f56e27ca70d32923ce9f23a

Bump corefx

[packaging] Fix path to wrapper.bat for Windows MSI

Merge pull request #5002 from BrzVlad/feature-sgen-modes

[sgen] SGen modes

Merge pull request #5021 from BrzVlad/fix-parallel-fixes

[sgen] Fix parallel fixes

[profiler] Fix issue found by review.

[packaging] Simplify Windows packaging scripts

We don't need to build in a separate checkout of mono anymore.
Also use .pkg passed in from upstream build.

[packaging] Move Windows MSI scripts into repository

They lived in https://github.com/mono/release before, moving them
into the main repo allows us to version them more easily.

Note: I intend to do more cleanup on those scripts in the future,
this is just the first step.

[man] Add entries for sgen modes and new major/minor types

[sgen] Add sgen modes

Group gc params configurations into general gc modes. The default mode (SGEN_MODE_NONE) is what we've been using so far, meaning old default features. The other modes can include experimental features.

[profiler] Remove heapshot request polling from all over the place and depend only on the finalizer callback.

We the profiler now emiting a lot less events by default, all the existing polling no longer makes sense as they won't matter in the common path.

Instead we make sure the finalization end callback is always installed and poll from it.

This introduces the risk of a heapdump never happening or experiencing long delays because there are either too many finalizers or one of them hung.
It's an acceptable risk.

[build] Bumps mcs bootstrap requirement

[msvc] Update csproj files (#4898)

[profiler] When using heapshot, don't collect roots when not doing a heapshot.

[profiler] Don't emit sync points during GC if we don't need to.

[profiler] Don't emit thread and gc events if they are not enabled.

Merge pull request #4967 from kumpera/profiler-arg-cleanup

Rework profiler argument handling.

Bump corefx

Fix OpenMappedWebConfiguration throws: System.IndexOutOfRangeException, when using RazorGenerator under mono (#5013)

Fix System.Numerics xunit tests and add them to CI (#5006)

* Fix System.Numerics xunit tests and add them to CI

Xunit.Extensions.NetCore.dll is available as a net46 version now
so we can just use it instead of building it from our buildtools fork.
This also allows us to remove the fork from our submodules.

We were missing running the System.Numerics xunit tests in CI,
this adds them so we catch breaks like these earlier.

* Skip tests on Windows, they seem to fail there

[mcs] C#7 type pattern matching

[sdb] Fix passing of nullable parameters in invokes. Fixes #57160. (#5012)

[sgen] Avoid using parallel minors for small nurseries

It is not effective and it can lead to regressions, due to the added synchronizations and work splitting.

[sgen] Disable parallel mode if not enough cores

[sgen] Use logical cpu count for number of workers

It actually leads to better performance. The downside is the creation of more sgen workers. This is currently capped at 8.

Remove the mono-profiler- prefix from all source files in the profiler subdir.

Additionally, rename arg-parsing.c to log-args.c so it's clear it's part of the log profiler.

[profiler]Use g_warning and remove bad usage() line

Merge pull request #4998 from kumpera/fix_56684

[corlib] Don't pass RuntimeMethodHandle across domains. Fixes #56684

Merge pull request #4997 from lambdageek/bug-57222

[sre] FieldBuilder:RuntimeResolve shouldn't lookup by name (Fixes #57222)

Bump corefx

[marshal] allocate dummy string always in root domain

otherwise it can happen that the dummy string is allocated in a
different domain, and when it's onloaded the GC root points to an
invalid object.

Bunch of minor fixes.

Remove env var.

update usage, print warning on output:-FILENAME and add a TODO.

Remove spurious header. Change from enum to bucket of defines.

[test] Test SRE of a class with two methods with the same name.

Regression test for https://bugzilla.xamarin.com/show_bug.cgi?id=57222

Although methods aren't affected by this bug, I added a regression test for
this case anyway in case the implementation changes.

Currently MethodBuilder.RuntimeResolve() goes through a different codepath - it
calls RuntimeType.GetMethod (MethodInfo) which calls into an icall which looks
for a method with a matching token rather than a matching name.

[corlib] Don't pass RuntimeMethodHandle across domains. Fixes #56684

We can't pass RuntimeMethodHandle across AppDomains since that can lead to memory corruption.

The way this happens is that when Domain (A) passes a method handle (M) to Domain (B), (B) will get the reflection object for
which will add that MonoMethod to the per-domain reflection hashtable. This won't cause the assembly of (M) to be registered with (B).

When (A) is unloaded, it might be the only domain referencing the assembly of (M), cause it to unload resulting in (B) having a dangling
pointer to (M) on its reflection hashtable.

The fix is to serialize enough lookup data across the wire so we can do type/method lookup using reflection on the target domain.
It might be slower, but this only happens on some remoting slowpaths anyways.

Finally, we can't simply register the assembly of (M) into domain (B) as that list (MonoDomain::domain_assemblies) is used for type loading,
thus breaking the target domain in some scenarios.

[sre] FieldBuilder:RuntimeResolve shouldn't lookup by name. (Fixes #57222)

RuntimeType:GetField (FieldInfo) looks up the field by name and attributes,
which doesn't work for SRE because it is legal to create multiple fields with
the same name and type and attributes.  So lookup using the
RuntimeFieldHandle (aka MonoClassField*) instead.

[test] Test SRE of a class with two fields with same name

Regression test for https://bugzilla.xamarin.com/show_bug.cgi?id=57222

Merge pull request #4995 from lambdageek/dev-monodis-show-field-tokens

[monodis] Document --show-tokens and show field tokens

[amd64] Fix a check in mono_emit_stack_alloc (). (#4985)

[monodis] Show field tokens in --show-tokens mode

[monodis] Print table name (when valid) in --show-tokens mode

[monodis] Document --show-tokens

Merge pull request #4994 from lewurm/interp-aot-mode-fix-windows

[win] set HAVE_ONLINE_VES

[win] set HAVE_ONLINE_VES

[interp] disable bug-45841-fpstack-exceptions.exe on CI

[interp] provide stub for icall_trampoline for x86 and ppc

keep the linker happy

[interp] root domain must be used for AOT'd wrappers used by interpreter

[interp aot] documentation

[marshal] some things are required in the interpreter despite DISABLE_JIT

[configure] introduce HAVE_ONLINE_VES

[interp] fix when DISABLE_JIT is set

[interp] add InterpMethodArguments struct to offset-tool

[interp] add MONO_AOT_MODE_INTERP

[interp] add --aot=interp, so we can run `mono --full-aot --interpreter`

[tests] Re-enable bug-45841-fpstack-exceptions.exe (#4927)

* [tests] Re-enable bug-45841-fpstack-exceptions.exe

* [tests] Pass correct parameters in bug-45841.

[mcs] Better detection of null operator inside conditional expression. Fixes #57232

Cosmetic; remove [Category ("Martin")] from the test.

[System]: Fix possible initialization issue in MonoTlsProviderFactory. (#4987)

* [Mono.Security]: Add test for MonoTlsProviderFactory.GetProvider ().

* [System]: Fix possible initialization issue in MonoTlsProviderFactory.

[System]: Actually disable the keychain code and SecItemImport() on XI. (#4983)

[mcs] C#7 out variable declaration

[MacSDK] Bump libgdiplus to latest (#4984)

We were using a very old commit of libgdiplus. This brings in
the latest changes.

[interp] factor out everything that allocates a MonoLMFExt on the stack in order to reduce stack pressure in the main interpreter loop

[interp] factor out MINT_JIT_CALL instruction from interpreter switch in order to reduce stack pressure

[sgen] Make sure workers process jobs only if they are running (#4974)

This simplifies logic and prevents serial context being used by multiple workers at the same time.

System.Drawing: Start consuming sources from corefx

Closes https://github.com/mono/mono/pull/4980

[MacSDK] Update version number in msbuild.py

[mini] set MONO_ARCH_HAVE_INIT_LMF_EXT on architectures that implement mono_arch_init_lmf_ext

it worked before, because we relied on implicit linking

[mini] guard mono_arch_init_lmf_ext prototype properly

[arm] don't set MONO_ARCH_SOFT_DEBUG_SUPPORTED on android cross compiling

follow up fix for https://github.com/mono/mono/pull/4956

this breaks the cross compiler on windows for android:

```
  [9:07:36]                       /bin/sh ../../libtool  --tag=CC   --mode=compile /Users/builder/android-toolchain/mxe/bin/i686-w64-mingw32.static-gcc -DHAVE_CONFIG_H -I. -I/U  sers/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini -I../..   -DWINVER=0x0600 -D_WIN32_WINNT=0x0600 -D_WIN32_IE=0x0501 -D_UNICODE -DUNICODE -  DWIN32_THREADS -DFD_SETSIZE=1024 -g -Wall -Wunused -Wmissing-prototypes -Wmissing-declarations -Wstrict-prototypes  -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wno  -cast-qual -Wwrite-strings -Wno-switch -Wno-switch-enum -Wno-unused-value -Wno-attributes -Wno-format-zero-length -D__ARM_EABI__ -DARM_FPU_VFP=1 -DNO_UNALIGNED_ACCESS -I/User  s/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono -I/Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/eglib/src -I../../eglib  /src -I/Users/builder/data/lanes/5000/ff452183/source/xamarin-android/build-tools/mono-runtimes/obj/Release//llvm/installed-win32/usr/include -DNDEBUG -D__NO_CTYPE_INLINE -D_  GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DLLVM_API_VERSION=4 -O -I../arch/arm -fvisibility=hidden -DXAMARIN_PRODUCT_VERSION=0 -static   -static-libgcc -std=gnu99 -fno-strict-aliasing -fwrapv -DMONO_DLL_EXPORT -Wno-unused-but-set-variable -g -Wall -Wunused -Wmissing-prototypes -Wmissing-declarations -Wstrict-  prototypes  -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wno-cast-qual -Wwrite-strings -Wno-switch -Wno-switch-enum -Wno-unused-value -Wno-attributes -Wno-format-ze  ro-length -mno-tls-direct-seg-refs -Werror-implicit-function-declaration -MT libmini_la-mini-arm-gsharedvt.lo -MD -MP -MF .deps/libmini_la-mini-arm-gsharedvt.Tpo -c -o libmin  i_la-mini-arm-gsharedvt.lo `test -f 'mini-arm-gsharedvt.c' || echo '/Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/'`mini-arm-gsharedv  t.c
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c: In function 'mono_arch_is_single_step_event  ':
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c:7242:2: error: unknown type name 'siginfo_t'
  [9:07:36]                         siginfo_t *sinfo = info;
  [9:07:36]                         ^
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c:7248:11: error: request for member 'si_addr'   in something not a structure or union
  [9:07:36]                         if (sinfo->si_addr >= ss_trigger_page && (guint8*)sinfo->si_addr <= (guint8*)ss_trigger_page + 128)
  [9:07:36]                                  ^
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c:7248:57: error: request for member 'si_addr'   in something not a structure or union
  [9:07:36]                         if (sinfo->si_addr >= ss_trigger_page && (guint8*)sinfo->si_addr <= (guint8*)ss_trigger_page + 128)
  [9:07:36]                                                                                ^
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c: In function 'mono_arch_is_breakpoint_event'  :
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c:7262:2: error: unknown type name 'siginfo_t'
  [9:07:36]                         siginfo_t *sinfo = info;
  [9:07:36]                         ^
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c:7267:11: error: request for member 'si_signo  ' in something not a structure or union
  [9:07:36]                         if (sinfo->si_signo == DBG_SIGNAL) {
  [9:07:36]                                  ^
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c:7269:12: error: request for member 'si_addr'   in something not a structure or union
  [9:07:36]                          if (sinfo->si_addr >= bp_trigger_page && (guint8*)sinfo->si_addr <= (guint8*)bp_trigger_page + 128)
  [9:07:36]                                   ^
  [9:07:36]                       /Users/builder/data/lanes/5000/ff452183/source/xamarin-android/external/mono/mono/mini/mini-arm.c:7269:58: error: request for member 'si_addr'   in something not a structure or union
  [9:07:36]                          if (sinfo->si_addr >= bp_trigger_page && (guint8*)sinfo->si_addr <= (guint8*)bp_trigger_page + 128)
```

Bump API snapshot

[System]: Correctly implement close and shutdown in SslStream. (#4969)

* [System]: Correctly implement close and shutdown in SslStream.

* Cleanup.

* Cosmetic.

* More ReferenceSources/SR2.cs into common.sources to make it build.

Use correct path separator for satellite assemblies in cross builds (#4976)

When creating a bundle on Windows to run on Android, macOS/iOS or Linux we need
to use the target system's path separator when storing satellite assemblies in
the bundle instead of the host system's one. Failing to do so will embed (on
Windows) the satellite assemblies with paths like

   fr_FR\myassembly.resources.dll

while at runtime Mono will try to look up the assembly by

   fr_FR/myassembly.resources.dll

and fail to find it, obviously. The `--style` parameter is now consulted to see
what path separator character should be used.

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

Bump corefx

[man] Remove mention of MONO_AOT_CACHE from mono.1

The env var was removed a long time ago in b8766a812d64d819948e1b7c6482a6ac80d3b4eb
and was replaced by options to --aot. This came up in Gitter today.

implement Image.SetPropertyItem() method v4 (#1167)

This is thought to be fixing https://bugzilla.novell.com/show_bug.cgi?id=406307.

v4: state that my changes are licensed under MIT X11 license

[w32handle] Fix deadlock on SignalAndWait (#4973)

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

Merge pull request #4962 from lambdageek/dev-handles-sockets

[w32socket] Use coop handles for many Socket icalls

[w32socket] Use coop handles for System.Net.Sockets.Socket.cancel_blocking_socket_operation

[coop handles] Add mono_internal_thread_handle_ptr () function

MonoInternalThreadHandle is not really needed - all MonoInternalThread
instances are supposed to be pinned.  But for uniformity, the icall marshaller
makes handles anyway.  This function lets us get back to a raw ptr.

[runtime] typedef MonoThread MonoThreadObject

MonoThreadHandle is already taken.  So define MonoThreadObject as an alias for
MonoThread and then make MonoThreadObjectHandle the coop handle type.

[w32socket] Use coop handles for icalls in System.Net.Sockets.Socket

- Use coop handles for create_object_from_sockaddr

  Eventually want to rewrite its callers to use
  create_object_handle_from_sockaddr directly.

- Use coop handles for

  - LocalEndPoint_internal

  - RemoteEndPoint_internal

- inside create_sockaddr_from_object

  Eventually callers should be updated to use create_sockaddr_form_handle
  directly.

- Use coop handles for

  - ReceiveFrom_internal

  - Bind_internal

  - Connect_internal

  - SendTo_internal

- Remove dead code (create_object_from_sockaddr, create_sockaddr_from_object)

- Use coop handles for

  - Receive_internal WSABUF[]

  - Receive_internal byte[]

  - Send_internal

  - Send_array_internal

  - SetSocketOption_internal

  - GetSocketOption_arr_internal

  - GetSocketOption_obj_internal

  - IOControl_internal

  - SendFile_internal

- Remove more dead code

- Use coop handles for

  - Socket_internal

  - Select_internal

- Trivial coop handles conversion for several icalls in System.Net.Sockets.Socket

  None of these functions use managed objects, so just add a MonoError* argument.