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

This reverts commit 94dcde72917979d9b25b3786e214f08cbc4d2299.

Bump corefx (#5167)

* Bump corefx

* Bump API snapshot submodule

[jit] Optimize ldloc+ldfld operations for valuetypes, so they don't require making a copy of the valuetype. Hopefully fixes #56452. (#5166)

[ci] Tell babysitter where profiler stress result XML lives

This allows it to parse the XML and provide better results
about which tests failed.

[Microsoft.Build] Improve assert in BuildSubmissionTest.EndBuildWaitsForSubmissionCompletion

It started failing more frequently, let's see if we can get better data.
Also did the same change to a similar test.

[jit] Fix mono_debug_count () so it doesn't confuse COUNT=0 with COUNT not being used.

[System.ServiceModel] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight-saving time

[Mono.Posix] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight-saving time

[System] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight-saving time

[corlib] Don't use DateTime.Now for measuring elapsed time

Use Stopwatch or DateTime.UtcNow instead which aren't affected
by things like daylight saving time

[jit] Use the right type when handling shared versions of RuntimeHelpers.IsReferenceOrContainsReferences (). The previous version only worked by accident. (#5160)

[utils] Fix CppCheck warning in mono-threads-posix.c (#5161)

After the refactoring in 9aaa0083d0def5b940372dbd9375336b036249fd
CppCheck complained about using 'result' uninitialized (looks like
it can't deduce that g_error will abort the process anyway).

[msvc] Update csproj files (#5162)

[mcs] C#7 tuples var deconstruct

[mcs] Adds missing file

[mcs] C# 7 tuple (foundation only).

Still couple of advanced features not implemented and parser cannot
deal with unnamed tuple type at type declaration level.

Merge pull request #5159 from MaurGi/patch-1

Console.Error.WriteLine in System.Net.NetworkInformation

Console.Error.WriteLine in System.Net.NetworkInformation

Console.Error.WriteLine ("Got a bad hardware address length for an AF_PACKET {0} {1}",...

This Console.WriteLine interferes with Apache Storm that reads from the console I/O.

See bug: https://bugzilla.xamarin.com/show_bug.cgi?id=57887

Merge pull request #5144 from kumpera/handle-fixes-part1

[io-layer] Fix two race conditions, one on FD handle creation and one on interrupt install.

Merge branch 'master' into handle-fixes-part1

Merge pull request #5156 from lambdageek/bug-57850+57851

[sre] Deal with ResolveEventHandler returning an AssemblyBuilder (Fixes #57850, #57851)

Fix a couple of bugs in the situation where a `ResolveEventHandler` happens to return an `AssemblyBuilder`.

* [#57850](https://bugzilla.xamarin.com/show_bug.cgi?id=57850) - if the `ResolveEventHandler` returns a reflection only assembly, throw a `FileNotFoundException`
* [#57851](https://bugzilla.xamarin.com/show_bug.cgi?id=57851) - don't look for a `ReferenceAssemblyAttribute` on assemblies returned by a `ResolveEventHandler` if they happen to be an `AssemblyBuilder`. (the native code path to get there is to try and instantiate a custom attribute in some other assembly, hence the fairly convoluted test in `assemblyresolve_event5` and its helper files)

[loader] Don't try to look for ReferenceAssemblyAttribute on dynamic assemblies (Fixes #57851)

.NETFramework seems to ignore this custom attribute and allows instantiating
types from on assembly builders.

[loader] Throw exn if ResolveEventHandler returns a refonly assembly (Fixes #57850)

(Also depends on previous commit to ensure that MonoAssembly:refonly is set
correctly for dynamic assemblies)

[sre] Mark AssemblyBuilderAccess.ReflectionOnly assemblies as refonly

This will ensure they show up in AppDomain.ReflectionOnlyGetAssemblies(), and
won't be allowed to execute

[sre] use AssemblyBuilderAccess bitmask to set MonoDynamicImage flags

In particular this makes sure that AssemblyBuilderAccess.ReflectionOnly aren't
allowed to run.  And AssemblyBuilderAccess.CollectAndRun aren't allowed to save.

[sre] Add MonoAssemblyBuilderAccess enum in native code

in sync with System.Reflection.Emit.AssemblyBuilderAccess

[mono-error] Add mono_error_set_file_not_found

To throw System.IO.FileNotFoundException

[tests] ResolveEventHandler that returns an AssemblyBuilder shouldn't crash mono

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

[runtime] Use pthread_cond_timedwait_relative_np () function on osx if available. (#5147)

[aot] Load aot images only in the root domain. (#5151)

[tests] Throw an error if ResolveEventHandler returns a reflection only AssemblyBuilder

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

[tests] Add some missing files to EXTRA_DIST

[runtime] Make runtime invoke wrappers for dynamic methods non-shareable, so they can be freed when the dynamic method is freed. (#5152)

[System] Request 32-bit capabilities

This fixes a crash that happened when running in the debugger.
This fixes Xamarin-21261.

Change-Id: I0ccda9f3c11ec02457a932073665721be5e4e83e

Merge pull request #5140 from alexrp/profiler-stress-fpe-fix

[acceptance-tests] Don't pass a zero sampling/heapshot frequency in the profiler stress runner.

[msvc] Update csproj files (#5149)

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

[bcl] Drop a bunch of unused _2_1.cs files (#5112)

* [bcl] Drop a bunch of unused _2_1.cs files

They weren't used anywhere, not even mobile, so we can remove them.

Revert "[runtime] Use pthread_cond_timedwait_relative_np () function on osx if available. (#5143)"

This reverts commit 616ea50718aba3af237637f77d9c58acdca9a61f.

Revert this as it fails to build on wrench.

[runtime] Use pthread_cond_timedwait_relative_np () function on osx if available. (#5143)

[io-layer] Fix two race conditions, one on FD handle creation and one on interrupt install.

mono_w32handle_new_fd: We must hold scan_mutex while initializing an unallocated handle otherwise
mono_w32handle_foreach might think it's a valid handle.

mono_w32handle_timedwait_signal_handle: We must increment the refcount before installing the interrupt handle
as the other end can be called right after we finish installing and would trigger a failure.

[w32handle] Replace mono_w32handle_{ref,unref} by mono_w32handle_{duplicate, close} (#5106)

* [w32handle] Fix handle dumped ref

* [w32handle] Replace exposing mono_w32handle_{ref,unref} by mono_w32handle_{duplicate,close}, it is closer to what the win32 API exposes.

* [w32handle] Remove unrefing of handles on shutdown

If one of the handle is a file handle for example, then w32file has already be cleaned up, so we cannot guarantee that the handle we are going to close is not going to call into w32file code.

Implement GC roots ourselves in Boehm rather than relying on their concept of roots which is mainly for static data segments in binaries and not implemented on all platforms (cannot remove roots on Windows) (#564)

[packaging] Remove git_branch from msbuild and mono-extensions (#5141)

We're checking out a specific commit for both now so the git branch
is no longer required.

Also fixes warnings that a 2017-06 branch doesn't exist in mono-extensions.

Manually invoke Xvfb, don't depend on the xvfb plugin (#5132)

[build] Add a new profile

[nullgc] When building nullgc, don't try to link Boehm libs

Previous nullgc commit (52d9a55a861527ed4027f8e10a97e3cc880baf4a) didn't actually work in a clean tree.

[acceptance-tests] Don't pass a zero sampling/heapshot frequency in the profiler stress runner.

Also change the code to make it more likely that we actually do some runs
without sampling or heapshot enabled.

[w32handle] Add some debugging code to help track down handle wait problems. (#5136)

[w32handle] Zero out the handle specific data in the handle dtor to help debugging. (#5135)

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