Merge in 'release/6.0' changes

Update Arcade + Workloads (#72947)

Update Arcade and the workloads project to leverage the new build tasks.

This also fixes the publishing issues we previously saw in the blob artifacts because of duplicate files. Instead, we'll create unique zip files per SDK featurebands for the workloads that will be used for creating the VSDROPs in dotnet-stage.

Merge in 'release/6.0' changes

Update branding to 6.0.9 (#73270)

Merge in 'release/6.0' changes

Update windows.10.amd64.server20h2 to windows.amd64.server2022 (#72868)

* Update windows.10.amd64.server20h2 to windows.amd64.server2022, as 20H2 is end-of-life this month

Merge in 'release/6.0' changes

Fix building with clang 14 (#69990)

Under clang 14, building runtime on Linux with a simple ./build.sh
fails with hundreds of errors that all look like this:

    runtime/src/libraries/Native/Unix/System.Native/pal_networking.c(2610,9):
    error GD58BE485: mixing declarations and code is incompatible with
    standards before C99 [-Wdeclaration-after-statement]
    [runtime/src/libraries/Native/build-native.proj]

This issue is not present in the main branch. It appears to have been
fixed by https://github.com/dotnet/runtime/pull/66410/ which turned off
that warning.

Tested with:

    $ clang --version
    clang version 14.0.0 (Fedora 14.0.0-1.fc36)
    Target: x86_64-redhat-linux-gnu
    Thread model: posix
    InstalledDir: /usr/bin

Merge in 'release/6.0' changes

[release/6.0] Fixing the handling of Positive NaN in Math.Min for float/double (#70865)

* Fixing the handling of Positive NaN in Math.Min for float/double (#70795)

* Adding tests validating Positive NaN for Max, MaxMagnitude, Min, and MinMagnitude

* Fixing the handling of Positive NaN in Math.Min for float/double

* Fixing the Max/Min code comments to use greater and lesser

* Adding a code comment clarifying the sign toggling behavior

* Apply suggestions from code review

Co-authored-by: Jonathan Giannuzzi <jgiannuzzi@users.noreply.github.com>
* Apply suggestions from code review

Co-authored-by: Jonathan Giannuzzi <jgiannuzzi@users.noreply.github.com>

Fix RSA OAEP decryption in Android with non-power-of-two key lengths (#71696)

createdump improvements for better VS4Mac Watson bucketing (#71863)

* Createdump changes for VS4Mac

Add target process terminated/alive message.

Smaller MacOS dumps. Don't add share_mode == SM_EMPTY regions.

Add crashreport success status message for VS4Mac.

Launch createdump from SIGTERM handler directly to reduce the time it takes to get the crash report/dump for VS4Mac.

* Add more instrumentation to diagnose SIGTERM problem

* Remove useless thread state logging

Forbid XML external entity resolution in crypto APIs

Merge in 'release/6.0' changes

Merge pull request #72040 from wtgodbe/internal-merge-6.0-2022-07-12-1057

Merging internal commits for release/6.0

Merge in 'release/6.0' changes

[release/6.0] [iOS][Android] Fix crash in Exception.CaptureDispatchState (#71171)

* [iOS][Android] Fix crash in Exception.CaptureDispatchState

There is a crash in `Exception.CaptureDispatchState` when called from one thread at the same time another calls into `Exception.RestoreDispatchState`. The
reason for the crash is due to the way we do not update `foreignExceptionFrames` in a thread-safe way.  `foreignExceptionFrames` is used in both methods and can crash when the size changes before the array is copied.

The fix is to lock when reading from and writing to `foreignExceptionFrames`.

Fixes https://github.com/dotnet/runtime/issues/70081

* Lock all of CaptureDispatchState

* Marek's feedback

* Remove unnecessary field

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

Merge commit '0ec02c8c96e2eda06dc5b5edfdbdba0f36415082' into internal-merge-6.0-2022-07-12-1057

[mono] Avoid calling mono_metadata_type_hash () on the container class in mono_generic_class_hash (). (#71764)

If a class inherits from a generic instance instantiated with itself, then its possible
for mono_generic_class_hash () to be called while the container class is not fully
initialized yet, i.e. byval_arg.type is 0. This would cause the generic class to change
its hash.

Fixes https://github.com/dotnet/runtime/issues/71424.

Co-authored-by: Zoltan Varga <vargaz@gmail.com>

Backport 71436 (#71731)

format correctly chuncked response with Content-Length header (#69016) (#70744)

Co-authored-by: Badre BSAILA <54767641+pedrobsaila@users.noreply.github.com>

Merge in 'release/6.0' changes

Remove GeneratePackageOnBuild missed on branding PR (#71743)

Merge in 'release/6.0' changes

Update branding to 6.0.8 (#71668)

Merge in 'release/6.0' changes

Revert Arcade update & Workload Manifest change (#71110)

* Revert "[release/6.0] Update dependencies from dotnet/arcade dotnet/llvm-project (#68871)"

This reverts commit 3e0c89b69f70690579d2f1c0a04577a674ef1cf6.

* Revert "Update workloads with Arcade improvements (#71046)"

This reverts commit cd19e8bf35d18cef6f30d896313cbeb1be2a06dd.

* Update global.json

Merge in 'release/6.0' changes

Update workloads with Arcade improvements (#71046)

Merge in 'release/6.0' changes

[tests] Extend partial disablement of SkippingHiddenFiles (#65280) (#70763)

Co-authored-by: Mitchell Hwang <16830051+mdh1418@users.noreply.github.com>

Merge in 'release/6.0' changes

[release/6.0] Update dependencies from dotnet/arcade dotnet/llvm-project (#68871)

* Update dependencies from https://github.com/dotnet/arcade build 20220504.3

Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.ApiCompat , Microsoft.DotNet.XUnitExtensions , Microsoft.DotNet.GenAPI , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.GenFacades , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.Helix.Sdk
 From Version 2.5.1-beta.22161.1 -> To Version 2.5.1-beta.22254.3

* Update dependencies from https://github.com/dotnet/arcade build 20220511.7

Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.ApiCompat , Microsoft.DotNet.XUnitExtensions , Microsoft.DotNet.GenAPI , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.GenFacades , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.Helix.Sdk
 From Version 2.5.1-beta.22161.1 -> To Version 2.5.1-beta.22261.7

* Update dependencies from https://github.com/dotnet/llvm-project build 20220513.3

runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools
 From Version 11.1.0-alpha.1.21416.1 -> To Version 11.1.0-alpha.1.22263.3

* Update dependencies from https://github.com/dotnet/arcade build 20220614.7

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 6.0.0-beta.22161.1 -> To Version 6.0.0-beta.22314.7

* Use VS preview pools for Windows build jobs, and remove Windows version number from queue name

* Revert "Use VS preview pools for Windows build jobs, and remove Windows version number from queue name"

This reverts commit 5478426a751b299afd19f81e15abf033f699d76c.

* Update dependencies from https://github.com/dotnet/llvm-project build 20220616.1

runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools
 From Version 11.1.0-alpha.1.21416.1 -> To Version 11.1.0-alpha.1.22316.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: carlossanlop <1175054+carlossanlop@users.noreply.github.com>

Merge in 'release/6.0' changes

Fix Assembly.LoadFrom resolver (#70537)

Path.GetFullPath fails on empty path. Check for empty requestor path earlier.

Fixes #67802

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

Merge in 'release/6.0' changes

Merge pull request #70741 from vseanreesermsft/internal-merge-6.0-2022-06-14-1031

Merging internal commits for release/6.0

Merge commit '7cca709db2944a09b4db6ca7b20c457ff260fb5a' into internal-merge-6.0-2022-06-14-1031

Merge in 'release/6.0' changes

Add Fedora 37 RID (#65392)

Fedora 37 has now started development, and the distribution now
identifies itself as Fedora 37 instead of Fedora 36:
https://fedorapeople.org/groups/schedule/f-37/f-37-key-tasks.html

Co-authored-by: Carlos Sanchez <1175054+carlossanlop@users.noreply.github.com>

Merge in 'release/6.0' changes

Fix async handle leak when sync RandomAccess read/write on async handle fails (#69956) (#70591)

* Fix async handle leak when sync RandomAccess read/write on async handle fails

* Use UnsafeAllocateNativeOverlapped in GetNativeOverlappedForAsyncHandle

This isn't invoking user code and it's entirely for synchronous operations; there's no reason it needs to capture ExecutionContext.

Merge in 'release/6.0' changes

Do not add x86 host to PATH on 64-bit machines (#70299)

Co-authored-by: Nikola Milosavljevic <nikolam@microsoft.com>

Merge in 'release/6.0' changes

[6.0] Modify thread pool thread counting to be a bit more defensive (#70479)

* [6.0] Modify thread pool thread counting to be a bit more defensive

- Port of https://github.com/dotnet/runtime/pull/70478 to 6.0
- An unexpected underflow in one or more thread counts can lead to a large number of threads to be created continually
- Prevented underflows in changes to thread counts, such that following an unexpected underflow, subsequent paired increments and decrements would avoid repeating the underflow
- Verified by creating an unexpected underflow in the debugger

* Address feedback

Merge in 'release/6.0' changes

[release/6.0][iOS] Grab the default timezone using native API (#70288)

Backport of https://github.com/dotnet/runtime/pull/70149

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

Merge in 'release/6.0' changes

Switch the ClientPre Helix queues to Client (#70570)

Merge in 'release/6.0' changes

JIT: Abandon loop search if we are about to walk off end of the bbNext chain (#69525)

In #69323 the 6.0.4 jit caused an AV because it walked off the end of the
bbNext chain during `optFindNaturalLoops`.

Analysis of a customer-provided dump showed that `MakeCompactAndFindExits`
might fail to find an expected loop block and so walk the entire bbNext chain
and then fall off the end. Details from the dump suggested that this happened
because a prior call to `MakeCompactAndFindExits` had moved most but not all of
a loop's blocks later in bbNext order, leaving that loop's bottom block earlier
in the bbNext chain then it's top. This ordering was unexpected.

I cannot repro this failure. The customer was using PGO and it's likely that
earlier PGO-driven block reordering contributed to this problem by interleaving
the blocks from two loops. We can recover the root method PGO schema from the
dump, but applying this is insufficient to cause the problem. This method does
quite a bit of inlining so it's likely that some inlinee PGO data must also be
a contributing factor.

At any rate, we can guard against this case easily enough, and simply abandon
recognition of any loop where we fail to find an expected loop block during
the bbNext chain walk.

Co-authored-by: Andy Ayers <andya@microsoft.com>

[Release/6.0] JIT: fix interaction of GDV and boxing (#60355) (#69630)

* JIT: fix interaction of GDV and boxing (#60355)

In #56126 we disabled GDV if the object being boxed came from a call with a
hidden return buffer pointer.

Fix this and enable GDV for these cases by inserting the box allocation statements
before the call.

* add test case

Fix address adjustment in bundle unmap for macos universal binaries (#69689)

Co-authored-by: vsadov <8218165+VSadov@users.noreply.github.com>

Fix singlefile on OSX ARM64 (#68845) (#69930)

* Same alignment in the bundle on OSX as on Linux

* Extra VA gap between section in casse we run from sf bundle

* Rename prevSectionEnd -> prevSectionEndAligned and make it aligned.

* Suppress assert for now on OSX

[release/6.0] Host error message updates (#70059)

* Update error messages for framework and SDK resolution failure

Partial backport of #67022

* Update error message for runtime not found

Backport of #67564

OpenSslX509ChainProcessor: ignore NotSignatureValid on last element. (#70343)

Merge in 'release/6.0' changes

Update branding to 6.0.7 (#70372)

* Update branding to 6.0.7

* Remove GeneratePackageOnBuild from all projects which added it in 6.0.6

Co-authored-by: Carlos Sanchez <1175054+carlossanlop@users.noreply.github.com>

Merge in 'release/6.0' changes

[release/6.0] add RID for Alpine 3.16 (#68925)

* add RID for Alpine 3.16

* fix packaging

Co-authored-by: wfurt <tweinfurt@yahoo.com>

Merge in 'release/6.0' changes

[release/6.0] Add windows runtime packs as a workload (#69393)

Backport of https://github.com/dotnet/runtime/pull/68981

At certain points in time internally, the sdk will rev its version and insert into VS. It's at this point the sdk version will be ahead of the runtime version as we have not published anything into nuget. In MAUI and WinUI scenarios, this creates a bit of a pickle as the sdk will try to download a new version of the windows runtime pack, but it will not be available.

To get around this limitation, we are introducing a workload that will be inserted into VS where you can optionally install Windows runtime packs.

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

[release/6.0] Restore FP pairs when unwinding in ARM64 (#69359)

* Add regression test
* Restore FP pairs when unwinding in ARM64
* Disable windows arm64 version of the test

Merge in 'release/6.0' changes

[release/6.0] Docker fixes for the enterprise linux testing pipeline (#68913)

* attempt to fix up EnterpriseTests (#65981)

* attemp to fix up EnterpriseTests

* fix text

* increase timoeout on build step

* fix temeout

* add jobs

* update job

* fix formating

* fix name

* update server check

* restore original resolver before tests

* fix path

* experiment

(cherry picked from commit 9174037cc6b58de847c9c88b7dcceef2b4fda440)

* upgrade docker to get clang9 to prevent coreclr crashes (#68311)

(cherry picked from commit edb9c498889b85ba2be015afca2f2cc73c30355c)

* Use Docker images from mcr.microsoft.com in EnterpriseTests (#68875)

We started getting warnings in the build about using images from docker directly, see https://docs.opensource.microsoft.com/tools/nuget_security_analysis/container_registry_analysis/

The image from dotnet-buildtools-prereqs-docker can be used instead.

(cherry picked from commit bafae1b8e6f1c6ce9798acf013b015a60def3381)

Co-authored-by: Tomas Weinfurt <tweinfurt@yahoo.com>

[release/6.0] Detect the default locale name during startup on Apple platforms (#68934)

* Detect the default locale name during startup on Apple platforms

This change adds a function to lookup the current NSLocale and extract the language + country code to load into ICU by default.  Previously, we would defer to uloc_getDefault in ICU, which would return a value we would ignore (en_US_POSIX) and result in falling back to invariant mode.

Fixes https://github.com/dotnet/runtime/issues/68321

* Link in foundation and address feedback

* Link in foundation in coreclr

* Check LANG env variable first

* Defer to icu first, since it's looking for LANG and certain values.

* #ifdef not #if

* PR feedback

* Remove redundant string replace

* pal_locale.m needs to be in a different location than upstream

* cmake tweak to make sure foundation gets linked in

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

Merge in 'release/6.0' changes

Fix memory leak at AssemblyLoadContext unload (#69334)

I have found there was a small memory leak when unloading
AssemblyLoadContext. A customer had a quite extreme testing case that
ran millions of iterations of creating an AssemblyLoadContext, loading
some assemblies into it and then unloading it. After 80 million
iterations on their machine with 8GB memory, the testing app was getting
out of memory.

Using PerfView, I was able to locate the source of the leak. It was a
missing destruction of the Module::m_pJitInlinerTrackingMap.

Co-authored-by: Jan Vorlicek <janvorli@microsoft.com>

Merge in 'release/6.0' changes

[release/6.0] Prevent debugger deadlock with code versioning logic (#68367)

* Release lock to prevent deadlock with code versioning logic when tracing a stub

* Update debugger.cpp

Merge in 'release/6.0' changes

Minimal version of #67150 (#68876)

Merge in 'release/6.0' changes

JIT: fix scalability issue in redundant branch optimizer (#68972)

In methods with long skinny dominator trees and lots of redundant branches
the jit can spend too much time trying to optimize the branches.

Place a limit on the number of redundant branches with matching VNs that
the jit will consider for a given branch.

Fixes #66067.

Co-authored-by: Andy Ayers <andya@microsoft.com>

Merge in 'release/6.0' changes

[release/6.0] Deadlock mitigation for debugging (#69121)

* WIP

* add logging

Update ceemain.cpp

Merge in 'release/6.0' changes

Merge pull request #69151 from vseanreesermsft/internal-merge-6.0-2022-05-10-1208

Merging internal commits for release/6.0

Merge commit '70ae3df4a6f3c92fb6b315afc405edd10ff38579' into internal-merge-6.0-2022-05-10-1208

Merge in 'release/6.0' changes

[release/6.0] [mono] Preserve FirstChanceExceptionEventArgs ctor (#68265)

* [mono] Preserve FirstChanceExceptionEventArgs ctor

The runtime is creating the exception so we need to preserve it.
See https://github.com/xamarin/xamarin-android/issues/6626

* PR feedback: use DynamicDependencyAttribute instead

Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Merge in 'release/6.0' changes

Fix passing too small args for PUTARG_STK on macOS arm64 ABI (#68113)

Backport of #68108

[release/6.0] Do not reset assigned interval twice (#68327)

* Do not reset the assignedInterval again

* Add test case

Co-authored-by: Kunal Pathak <Kunal.Pathak@microsoft.com>

[release/6.0] Fix JIT using too wide indirections when returning small structs (#68379)

* Fix JIT using too wide indirections when returning small structs

Fix #58874
Fix #64802
Fix #68157

* Add test

* Run jit-format

* Do not leak in case of merged tests

* Small test nit

* Another test nit

* Final test nit

* Make test compile

Co-authored-by: Jakob Botsch Nielsen <jakob.botsch.nielsen@gmail.com>

[release/6.0] Unloadability bugs fixes (#68674)

* [Release/6.0] Port unloadability fixes

This change ports two unloadability fixes for issues found by a
customer.
* #68550 that adds reference between native
`LoaderAllocator`s of two collectible `AssemblyLoadContexts` when one of
them is used to resolve assembly using the other one. This reference
ensures that the one that provides the resolved `Assembly` isn't
collected before the one that uses it.
* #68336 that adds a missing lock around `m_LoaderAllocatorReferences`
iteration during assembly load context destruction.

* Remove breaking change part

The change in main has added an exception in case a collectible
`Assembly` is resolved into a non-collectible `AssemblyLoadContext`.
Although that is incorrect, there are cases when it would work. So the
added exception is a breaking change that I think we likely don't want
to get into 6.0

[release/6.0] Fixed the yp_spin_count_unit to be a factor of the original_spin_count_unit rather than a continually increasing value  (#68881)

* Fixed the yp_spin_count_unit to be a factor of the original_spin_count_unit rather than a continually increasing value

* Fixed type of the spin counts

Merge in 'release/6.0' changes

[release/6.0] Disable ZipFile_Unix.UnixCreateSetsPermissionsInExternalAttributes (#68867)

Co-authored-by: carlossanlop <carlossanlop@users.noreply.github.com>

Merge in 'release/6.0' changes

[release/6.0] JIT: fix bug where a gc struct is not zero initialized (#68050)

Port of #67825 to release/6.0.

Merge in 'release/6.0' changes

[release/6.0] Fix in-process decoding of DateTime in EventSource (#68352)

* Fix in-process decoding of DateTime in EventSource

* Update eventlistener test to validate DateTime is correctly decoded in-process

* Use UTC time everywhere

Co-authored-by: John Salem <josalem@microsoft.com>

Merge in 'release/6.0' changes

[release/6.0] add support for parsing Unified TLS hello (#68425)

* add support for parsing Unified TLS hello

* feedback from review

* update comment

[release/6.0] Handle parameterless ctors in structs in STJ's ReflectionEmitMemberAccessor (#67901)

* [release/6.0] backport #62989 "Handle parameterless ctors in structs in STJ's ReflectionEmitMemberAccessor"

* Add explicit parameterless constructors in the tests

* Update ConstructorTests.ParameterMatching.cs

* address feedback

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>

Merge in 'release/6.0' changes

[release/6.0] Use x64 directory if running x64 process on arm64 apphost (#68583)

* Use x64 directory if running x64 process on arm64 apphost

Backport of #59890

* Fix is_emulating_x64 on Windows (#68671)

Co-authored-by: Mateo Torres-Ruiz <mateoatr@users.noreply.github.com>

Merge in 'release/6.0' changes