Add UnmanagedCallersOnly attribute to SafeDeleteSslContext.ReadFromConnection/WriteToConnection methods (#55947)

* Add UnmanagedCallersOnly attribute to  SafeDeleteSslContext.ReadFromConnection/WriteToConnection methods

* Fix the build

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
* Fix the build

* Add SslSetConnection interop method to make sure the right SafeDeleteSslContext instance is associated to an ssl session

* Update entrypoints.c with new DllImport

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Use OID constructor for certificate SignatureAlgorithm.

This fixes an issue where it was not possible to get the SignatureAlgorithm
on Windows if the signature algorithm does not have a FriendlyName, like
a GOST certificate. This changes the SignatureAlgorithm property to use the
OID constructor so that the value can always be accessed, and the FriendlyName
is lazily resolve when the property is accessed.

JIT: Don't fold NEG(NEG(X)) to X when NEG(x) is a CSE candidate (#57478)

[wasm] [debugger] Cache debugger information on Proxy side to avoid going to debugger-agent everytime. (#57404)

* Caching some debugger information to avoid going to debugger-agent everytime.

* Apply suggestions from code review

Co-authored-by: Ankit Jain <radical@gmail.com>
* Addressing @radical comments.

* Adding cache for generic type.

* Fix merge.

* Fixing test behavior.

* Fixing debuggerproxy.

* Fixing merge
Fixing tests.

Co-authored-by: Ankit Jain <radical@gmail.com>

Avoid most of the (small) regressions for seeded and derived Random instances (#57530)

* Split Random compat implementation to address most regressions

When we introduced the new Random algorithm, we did so by factoring the old algorithm out into an implementation strategy class that is instantiated for all use other than `new Random()`.  This ends up penalizing other uses (providing a seed and/or deriving from Random) by adding more virtual dispatch than is strictly necessary, in particular for `new Random(seed)`.  This PR negates most of that (expected) regression by splitting the compat implementation in two, one class for `new Random(seed)` and one for `new DerivedRandom()`/`new DerivedRandom(seed)`; the former no longer needs to make virtual calls back out to the parent type.  The former is also one that a consumer can't really do anything to improve, whereas in the derived case, the derivation may override to provide a more optimal implementation.

* Change NextUInt64 implementation to reduce number of calls

We haven't shipped this yet, so we can change its implementation to make 3 calls instead of 8 and to delegate to a different overload of Next.

* Consolidate NextBytes to use span

Skip CSE for non-hoistable floats (#57438)

Add `zip` tool to the install list for Android tests (#57569)

On default WSL Ubuntu `zip` is not installed, and trying to build Android tests fails with rather cryptic errors.

[iOS] [Android] Remove PNSE from some HttpClientHandler methods (#57555)

Since HttpClientHandler on mobile can support both the underlying native handler and SocketsHttpHandler, it doesn't make sense to always throw PNSE on some methods due to only the native handler not supporting them.

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

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

[mono] Check for additional implemented variant interfaces (#57086)

* [class-setup-vtable] Use flags bitmask in check_interface_method_override

No funcitonal change yet.

* [class-setup-vtable] Check additional slots when explicitly implementing variant interfaces

If a class implements a variant interface, consider whether it is explicitly
implementing (as opposed to obtaining by being a subclass of some base class)
some variant interfaces.

Two examples:
public interface IFactory<out T> { T Get(); }
public class Foo {}
public class Bar : Foo {}
public class FooFactory : IFactory<Foo> { public Foo Get() => new Foo(); }
public class BarFactory : FooFactory, IFactory<Bar> { public new Bar Get() => new Bar(); }

In this case, BarFactory explicitly implements IFactory<Bar> and also
IFactory<Foo>.

Conversely for contravariant gparams:
interface ITaker<in T> { string Consume (T x); }
class Foo {}
class Bar : Foo {}
class BarTaker : ITaker<Bar> { public string Consume (Bar x) => "consumed Bar"; }
class FooTaker : BarTaker, ITaker<Foo> { public string Consume (Foo x) =>
"consumed Foo"; }

In this case FooTaker implements ITaker<Foo> but also ITaker<Bar>.

When this happens, the signature of the implementing method 'Bar
BarFactory:Get()' doesn't match the signature of the implemented interface
method 'Foo IFactory<Foo>:Get()'.

We should check the signature parameters of the candidate method and the
implemented method, but I think the interface setup code already checks this
for us.

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

* oops, disable vtable tracing

* Use explicit variant interface to implement base type interfaces

Fix DacHeapWalker for USE_REGIONS (#56796)

[QUIC] QuicStream add ReadsCompleted (#57492)

Add ReadsCompleted API that exposes ReadState=ReadsCompleted, set ReadState to ReadsCompleted if FIN flag arrives in RECEIVE event, fix ReadState changing to final stauses, expand ReadState transition description

Fixes #55707

[main] Update dependencies from dotnet/runtime dotnet/emsdk (#57491)

* Update dependencies from https://github.com/dotnet/runtime build 20210815.6

Microsoft.NETCore.ILAsm , Microsoft.NETCore.DotNetHostPolicy , Microsoft.NETCore.DotNetHost , runtime.native.System.IO.Ports , Microsoft.NET.Sdk.IL , System.Runtime.CompilerServices.Unsafe , System.Text.Json
 From Version 6.0.0-rc.1.21411.3 -> To Version 6.0.0-rc.1.21415.6

* Update dependencies from https://github.com/dotnet/emsdk build 20210816.1

Microsoft.NET.Workload.Emscripten.Manifest-6.0.100
 From Version 6.0.0-rc.1.21415.1 -> To Version 6.0.0-rc.1.21416.1

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

System.Net.NetworkInformation UnsupportedOSPlatform attributes (#57281)

* Added UnsupportedOSPlatform attributes to PNSE thowing methods

* Replaced unix with the list of OSes

Fix System.Text.Json IAsyncEnumerator disposal on cancellation (#57505)

* Ensure WriteStack.Pending task is awaited on exception. Ensure IAsyncDisposable instances are disposed exactly once.

Fixes #57360.

* Update src/libraries/System.Text.Json/tests/Common/CollectionTests/CollectionTests.AsyncEnumerable.cs

Move Select_AcceptNonBlocking_Success to non-parallel collection (#57503)

Re-enable NameResolution tests on SLES, Debian 9 and Ubuntu 16.04 (#57509)

The underlying infrastructure issue was addressed in https://github.com/dotnet/core-eng/issues/13889

Fixes #55271
Fixes #56295

don't fail LongModuleFileNamesAreSupported test if we can't load assembly on x86  (#57471)

[wasm] Redesign of JS objects marshaling and lifecycle (#57098)

* thenable and datetime cleanup

* removed ownsHandle for readability, removed AnyRef from hierarchy, removed old refcount debug code

* rename _csOwnedObjects

* rename _csOwnedObjects

* redirect to Inflight counter

* renames

* rename and move

* rename

* New+BindCoreObject -> CreateCsOwnedObject single call

* remove refcount frames

* differentiate gcHandles

* get rid of weak GCHandle

* feedback

* added MappedType enum

* naming consistency

* feedback

* rename _cs_owned_objects_by_js_handle

* split and rename files

* shouldAddInflight to more methods

* improved in-flight references

* assert for JSObject not disposed

* introduce in-flight assert

* move should_add_in_flight arg to fisrt position, so that we could bind it in convert

* doc

* cleanup

* doc

* implement fallback for missing support of FInalizationRegistry/WeakRef in onlder browsers

* feedback

* no polyfill feedback

* feedback

* name consistency feedback

* use symbols for helper fields

* feedback

Initialize gtUseNum before use (#57256)

* Initialize gtUseNum

* Add test case

* catch the exceptions in test case

* kind of disable test

* try to exclude the test one more time

Add more Metrics API overloads taking tags (#56940)

* Add more Metrics API overloads taking tags

Fix handling of new connection in MsQuicListener (#57319)

* update

* feedback from review

Set CMAKE_OSX_DEPLOYMENT_TARGET for all Mono AOT compilers on macOS (#57507)

* Set CMAKE_OSX_DEPLOYMENT_TARGET for all Mono AOT compilers on macOS

Not just if we're targetting iOS/tvOS/macOS since the AOT compiler for WASM or Android can run on macOS too.

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

* Use RealTargetOS if it's available

Add linux mariner to rolling builds for libraries tests (#57322)

* Add linux mariner to rolling builds for libraries tests

* Adjust ChasChaPoly crypto test for mariner

* Disable tests failing on mono

[mono][llvm] Access mono_thread_interruption_request_flag directly from llvm generated code, similarly to mono_polling_required. (#57263)

Check for OpenSSL availability for OpenSsl asymmetric primitives

If you attempt to use any of the *OpenSsl primitives on macOS, this will currently abort the whole process.
This changes them to check for OpenSSL's presence and throw an appropriate exception instead of aborting.

When used as an internal implementation, the aborting behavior remains.

Revert "[mono] Fix a few corner case overflow operations (#57407)" (#57501)

This reverts commit e81efc832ae56978ae26ff0dbf17cb5622cee735.

Localized file check-in by OneLocBuild Task (#57384)

[debugger][wasm] Support DebuggerProxyAttribute (#56872)

* Implementing debugger proxy

* fix compilation

* Implement debuggerproxy attribute.

* Reusing code for DebuggerProxy method and DebuggerDisplay method.

* Fix unit tests.

* Fixing unit tests that uses List<T>.

* Fix unit tests that uses List.

* Addressing @radical comments.

* Using flags enum as suggested by @lewing.

* Fixing merge.

* Addressing @radical comments.

* Addressing @radical comments.

Co-authored-by: Larry Ewing <lewing@microsoft.com>

Account for type mismatch of `FIELD_LIST` members in LSRA (#57450)

* Add the test

* Fix the bug

Assertion propagation can replace a TYP_UBYTE field
in a field list with a TYP_INT constant, so LSRA
must use the actual field type when deciding whether
an internal register to allocate must be byteable.

Qualify `sorted_table` allocation with `nothrow` (#57467)

It's clearly what we intended, given the following line.

Rename transport packages to follow convention (#57504)

* Rename transport packages to follow convention

As agreed on in https://github.com/dotnet/windowsdesktop/pull/1936, we
want to follow a common schema when naming our transport packages.

Also updating the docs to reflect the name changes and to also
accomodate for the WindowsDesktop transport package.

* Update src.proj

* Update src.proj

Generate proper DWARF reg num for ARM32 (#57443)

After introduction of VFP-v3 ARM S0-S31 no longer can be generated using LLVM because  numbering of registers to start from 256 and only D0-D31 are used.
So this change encode S0 as D0, S2 as D1, etc. Also use reg nums for DXX registers.
This change fix generation of CFI codes,
which trigger issue with generation of DWARF using LLVM in NativeAOT
See https://developer.arm.com/documentation/ihi0040/c/?lang=en#dwarf-register-names
See https://github.com/dotnet/runtimelab/issues/1388

Enable System.Linq.Queryable and disable https://github.com/dotnet/runtime/issues/50712 (#57464)

Mark individual tests for 51211 (#57463)

Fix Length for ReadOnlySequence created out of sliced Memory owned by MemoryManager (#57479)

Add JsonConverter.Write/ReadAsPropertyName APIs (#57302)

* add JsonConverter.Write/ReadAsPropertyName APIs

* address feedback

Remove workaround for dotnet/sdk#19482 (#57453)

Do not drain HttpContentReadStream if the connection is disposed (#57287)

Fixes #56159.

[mono] Fix a few corner case overflow operations (#57407)

* [interp] Fix a few overflow conversions

Comparison for overflow was incorrect because for example, (int8)-128.5 = -128, without overflows, even if -128.5 is smaller than the minimum integer. We use instead the float equality comparison between trunc (val) and (int)val.

* [mono][jit] Fix invalid uses of SHL instead of MUL optimization

mono_is_power_of_two is meant to be used only on unsigned numbers. In some cases we pass a signed value instead. This is typically not a problem because negative numbers are not detected as a power of two, except for the special number -2147483648, which is coded as 0x80000000, therefore a power of two.

* Enable tests

make use of ports in SPN optional (#57159)

* make port optional in SPN

* fix tests

* feedback from review

* Update src/libraries/System.Net.Http/src/System/Net/Http/SocketsHttpHandler/AuthenticationHelper.NtAuth.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* fix build

Co-authored-by: Stephen Toub <stoub@microsoft.com>

Fixed H/3 stress server after the last Kestrel change (#57356)

disable a failing stress test. (#57473)

Eliminate temporary byte array allocations in the static constructor of `IPAddress`. (#57397)

Update dependencies from https://github.com/dotnet/emsdk build 20210815.1 (#57447)

Microsoft.NET.Workload.Emscripten.Manifest-6.0.100
 From Version 6.0.0-rc.1.21409.2 -> To Version 6.0.0-rc.1.21415.1

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

[main] Update dependencies from mono/linker (#57344)

* Update dependencies from https://github.com/mono/linker build 20210812.1

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.6.21409.3 -> To Version 6.0.100-preview.6.21412.1

* Update dependencies from https://github.com/mono/linker build 20210813.1

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.6.21409.3 -> To Version 6.0.100-preview.6.21413.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Larry Ewing <lewing@microsoft.com>

Improve serializer performance (#57327)

[wasm] Don't use workload for tfm < net6.0 (#56606)

* Bump sdk used for testing to 6.0.100-rc.1.21402.6

* InstallWorkloadFromArtifacts: Add OnlyUpdateManifests parameter

* [wasm] Run Wasm.Build.Tests with sdk/no-workload/+EMSDK

Currently, Wasm.Build.Tests are run in two modes:

1. sdk(`$(SdkVersionForWorkloadTesting)`)+workload installed
2. sdk(from global.json)+EMSDK+LocalBuild targets

this commit changes (2) to:

2. sdk(`$(SdkVersionForWorkloadTesting)`), no workload installed, +
   EMSDK+LocalBuild targets

This makes it closer to what a user would actually use.

Also, for the workload tests, this removes `$(WasmNativeWorkload)`
always being set to `true`. The workload should be setting that, and not
the tests.

* Wasm.Build.Tests: Add test for net5.0 blazor projects

* Disable unrelated builds for now

* Revert "Disable unrelated builds for now"

This reverts commit 6fa0efe2ffcfdbcfaba3bdf519a7717809c22672.

* Rename OnlyWasmBuildTests -> TestWasmBuildTests

Support long module path (#57335)

* Support long path

* while (true)

* Calculate new length based on array length

* Add a test

* use a smaller value to ensure that at least for DEBUG builds

we test the code path that rents a bigger array from ArrayPool

* add few extra asserts to the test to see why it fails in CI

* Assembly.LoadFile used the way this test is implemented fails on Mono

* don't run the test on Windows 7, for some reason it does not work

* Apply suggestions from code review

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* handle truncated module names, simplify the implementation and dispose tmp modules before throwing exception

* make sure that when the module is not found by the test, the test runner prints available module paths

* Apply suggestions from code review

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* polishing

Co-authored-by: NextTurn <45985406+NextTurn@users.noreply.github.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Update source-index-stage1.yml (#57435)

Again apply https://github.com/dotnet/runtime/commit/c842beb5a94642e7ddd06e262c50f67e297ce986 as the fix was overridden in the latest Arcade update.

Fix `FileProviders.Physical.GetLastWriteTimeUtc` crash (#57415)

* Update PollingFileChangeToken.cs

* Return min date on fail

[hot_reload] Allow the .ctor token to change for CustomAttribute updates (#57388)

* [hot_reload] Allow the .ctor token to change for CustomAttribute updates.

If the "type" (really .ctor) token is a MemberRef, it won't be literally the
same token value in the update as in the baseline because Roslyn emits
additional MemberRef, TypeRef and AssemblyRef rows instead of trying to reuse
the baseline ones.

We could try to resolve the baseline and delta tokens and check that they map
to the same MonoMethod*, but that is problematic (it may trigger assembly
loading and class initialization) and also unnecessary - without the
ChangeCustomAttribute capability Roslyn will not allow edits that change the
constructor or its arguments.  So just drop the extra check.

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

* Add testcase

ILCompiler.Reflection.ReadyToRun has to target netstandard2.0 for ILSpy usage (#57416)

IOException path (#57389)

[tests] Set the EventSourceSupport feature switch (#57374)

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

The `EventSource_ExistsWithCorrectId` test requires the event source
switch be set to true to work correctly after aggressive linking.

[main] Update dependencies from dotnet/arcade dotnet/hotreload-utils dotnet/roslyn-analyzers dotnet/msquic (#57402)

[main] Update dependencies from dotnet/arcade dotnet/hotreload-utils dotnet/roslyn-analyzers dotnet/msquic

[main] Update dependencies from mono/linker (#57203)

* Update dependencies from https://github.com/mono/linker build 20210811.1

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.6.21409.3 -> To Version 6.0.100-preview.6.21411.1

* Update dependencies from https://github.com/mono/linker build 20210811.2

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.6.21409.3 -> To Version 6.0.100-preview.6.21411.2

* Update dependencies from https://github.com/mono/linker build 20210811.3

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.6.21409.3 -> To Version 6.0.100-preview.6.21411.3

* Suppress trimmer warning

* Fix warning code value

* Fix netstandard build of libs

* Revert the code fix and just fix the XML suppression

* Revert project change

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: vitek-karas <vitek.karas@microsoft.com>

[wasm] Make sure the TargetOS is capitalized (#54927)

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

The $(TargetOS) is used in $(TargetFrameworks) and is case sensitive.
Make sure it is capitalized for Browser, so that build.cmd works when
used like:

    ./build.cmd -bl -os browser -subset libs

It fixes build errors like:

    C:\r\src\libraries\shims\manual\System.forwards.cs(8,88): error CS0234: The type or namespace name 'ZLibException' does not exist in the namespace 'System.IO.Compression' (are you missing an assembly reference?) [C:\r\src\libraries\shims\manual\System.csproj]
    C:\r\src\libraries\shims\manual\System.forwards.cs(9,77): error CS0234: The type or namespace name 'CookieVariant' does not exist in the namespace 'System.Net' (are you missing an assembly reference?) [C:\r\src\libraries\shims\manual\System.csproj]
    C:\r\src\libraries\shims\manual\System.forwards.cs(10,77): error CS0234: The type or namespace name 'PathList' does not exist in the namespace 'System.Net' (are you missing an assembly reference?) [C:\r\src\libraries\shims\manual\System.csproj]

Properly account for padding in PUTARG_SPLIT with GT_FIELD_LIST (#57279)

We were writing the stack part to the outgoing stack area by a loop
incrementing the offset by the written size in each iteration. However,
it is possible due to padding and optimization that the written size is
smaller than the delta to the next field. Instead, use the offset of
each field minus the offset of the first stack field to find the offset
in the outgoing arg area.

Fix #57064

Harvest signatures of methods being compiled by the token resolver (#57366)

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

I believe this is the primary fix for the issue; when a type only
ever occurs in method signatures in an assembly, never in the
implementation IL, JIT never calls resolveToken for it and so it
doesn't make it to the module token resolver table.

My investigation revealed additional potential problems in the
product around exported type resolution and the IL linker;
I described them in the issue thread but I believe this to be
both the most appropriate and the safest primary fix.

Thanks

Tomas

Avoid finalizer race in the SslStreamNetworkStreamTests cert creation (#57381)

[libraries] Update System.Runtime.Extensions tests for iOS,MacCatalyst,tvOS (#57210)

* [libraries] Update AppDomainTests for iOS,MacCatalyst,tvOS

* Update skip reasons

* Add MacCatalyst to Case Insensitive and remove ActiveIssues

* Replace ActiveIssue with ConditionalTheory based on RemoteExecutor

* Fix test for iOS/tvOS/Catalyst

* Skip UserName test on iOS tvOS

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Fix thread pool completed work item counter to remove finalized nodes (#57305)

* Fix thread pool completed work item counter to remove finalized nodes

- New nodes are being added to the set but finalized ones were not being removed
- This would result in double-counting the completed work item count for a thread that exits, and memory would grow over time

* Add a separate finalization helper for each node to track thread-local lifetimes

* Remove unnecessary IDisposable

* Fix nullability issues

* Update initialization of thread-local as suggested

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Add UnsupportedOSPlatform to RC2.Create for Android (#57289)

* Add UnsupportedOSPlatform to RC2.Create for Android

* Add missing resource

* Reuse existing resource string

* Code review feedback

Fix compile issue with escaped characters in logging generator message (#57379)

Ignore IOException in PollingFileChangeToken (#57376)

* Ignore IOException in PollingFileChangeToken

* Don't use local method since FileStatus is a struct

* errno needs to be saved before calling InvalidateCaches

Improve AppendLiteral for short literals (#57217)

* Improve AppendLiteral for short literals

DefaultInterpolatedStringHandler.AppendLiteral is special, in that the 99.9% use case is it's called as part of string interpolation by compiler-generated code where the argument is always a literal.  That means when the method is inlined, the JIT can specialize the implementation based on knowing the literal's Length is a constant.  Which means we can provide specialized implementations for common lengths, resulting in faster and smaller code.  This does so for strings of length 1 and 2, as they're quite common in code bases examined, and it does so for both the default handler and also the handler used to format into a span.

The exact same code as before gets generated by the JIT for literals of length > 2.  The main downside here is if someone does manually call this CompilerServices method with a variable, the call site will end up bloated with the special cases.

* Address PR feedback

* Disable some CodeDom tests on mono interpreter

imul instruction size misprediction (#57179)

* Add assert

* Remove the assert in emitInsSizeCV

* Add a check for includeRexPrefixSize

* Remove the codeSize() capping code added to fix #12840

* Make immediate only 4bytes long for non-mov instructions

* Delete a commented code

Add Capture.ValueSpan (#57357)

Fix Build Failures if there are no packages produced in the release branch (#57311)

* no packages

* Update eng/pipelines/libraries/build-job.yml

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>
* change parameter name and add main condition to running of tests

* remove package testing condition as we no longer run package testing on helix

* Update eng/pipelines/libraries/build-job.yml

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>
* Update upload-intermediate-artifacts-step.yml

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

remove non -required package downloads and package reference (#57367)

fix no_progress attribute (#57377)

Fix swapping evaluation order in assignments with CSE defs (#57364)

In some cases we were swapping the evaluation order of operands in
assignments when there were CSE defs in the subtrees. Change the logic
to properly forego the swapping in this situation.

Fix #57121

Add ILStrip task to trim method bodies (#57359)

This is used for iOS when assemblies are FullAOT compiled since the method body IL is unnecessary there.

Handle early wakeup of counter loop (#57170)

* Fix #56695
* prevent a negative delta value

* PR feedback + bug fix
* bug fix is preexisting: if a user other than the one who is listening sets the interval to 0 while the counter is enabled, it makes the counter continuously submit values. Changed this so that an interval value of <= 0 results in the counter being disabled

* Add assert after offline conversation

[metadata] Generic params in covariant return checking is ok (#57298)

* [metadata] Generic params in covariant return checking is ok

The issue is that `mono_type_is_reference` does not look at whether a generic
parameter has a constraint that would make it a reference type (there is a big
comment near `mono_type_is_reference` that making it do so would break generic
sharing).

The issue is if you have something like:

```

class BaseClass {}

public class C {
       public virtual BaseClass SomeCovariantFunction() => ...
}

public class G<T> : C where T : BaseClass {
       public override T SomeCovariantFunction() => ...
}
```

Here when setting up `G<T>` we do a check that the override of
SomeCovariantFunction that returns T is a valid, which boils down to checking
that `T` is ok for a covariant return from `BaseClass`.  This is mostly the
same as `BaseClass` is-assignable-from `T`, except we also need to check that
`T` is not a valuetype that implements interface `BaseClass`.  That check was
wrong.  The fix is to check for `is_valuetype`, not `!is_reference` on `T`.

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

* fixup

Mark operands of non-RMW as delayFree  (#57252)

* For non-RMW intrinsics, always delayfree the operands

* Add test case

* Update test case to add Avx2.IsSupported

Fix invoking of `FileSystemWatcher` (#53151)

Call `protected` `On...` event raising methods from `Notify...` methods,
to invoke `SynchronizingObject` when required.

Fix #52644

improve SNI handling when IP is passed in as DnsEndPoint to QuicListener (#57255)

[libraries] Skip FormKC and FormKD on iOS, tvOS, and MacCatalyst (#57317)

* [libraries] Skip FormKC and FormKD on iOS, tvOS, and MacCatalyst

* Remove associated ActiveIssue

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

[mono][interp] Fixes for compilation of huge methods (#57346)

* [interp] Disable inlining if interpreter var limit is excedeed

Currently, the interpreter has a 64k limit for the stack space to be used by vars. If we reach this limit, retry compilation with inlining disabled, since inlining can increase significantly the method code.

* [interp] Improve cprop speed by removing redundant memclear

Instead of clearing the defs for all vars, clear only for the vars used within a basic block, as the jit does. Otherwise, for large methods with many locals and many bblocks, we end up spending most of the time clearing memory. Improves speed of cprop by 20-30%, while for huge methods it can improve it by orders of magnitude.

HttpStress: Optimize LogHttpEventListener (#56983)

Eliminate autoflush, blocking calls and Task allocations in LogHttpEventListener, which can go expensive, when logging several thousands of lines per sec.

[wasm] Disable native stripping for wasm samples. (#57316)

WinHttp: Remove [SupportedOSPlatformAttribute] with specific Windows version (#57340)

Zero upper 4 bytes of the second operand of Vector3.Dot() in LowerHWIntrinsicDot() on Arm64 (#57328)

Quic Abort after Dispose is no-op (#57318)

Allow calling Abort(Read|Write) after disposing. The only safe option to do so is to make them no-op in that case.

Fixes #56683

Add missing WindowsDesktop libraries (#57339)

These two libraries are exposed in WindowsDesktop's shared framework but are missing from the transport package.

Add support for function pointers in ilverify (#57163)

* Provide logic verifying access to function pointer.

Extend AccessVerificationHelpers.CanAccess method so that it is
able to handle FunctionPointerType by making sure that a current class
can access all the types returned or accepted by the method pointed
by the given function pointer.

Fix #43502

* Add support for function pointers in ilverify importer

* Add ilverify tests for function pointers

* Trim trailing whitespace

* Use public surface of FunctionPointerType

* Check both return type and arguments of signature

* Delete CanAccessMethodSignature and simplify

* Add a test case for inaccessible return type

Add Quic back to aspnetcore transport pack (#57338)

Also make sure that the documentation file is placed next to the
reference assembly if such is included as part of a ProjectReference.

Also move the IsAspNetCore property into packaging.targets as it's not
needed by the binplacing system anymore.

[main] Update dependencies from dnceng/internal/dotnet-optimization dotnet/arcade dotnet/runtime-assets dotnet/hotreload-utils (#57278)

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20210811.3

optimization.linux-x64.MIBC.Runtime , optimization.windows_nt-x64.MIBC.Runtime , optimization.windows_nt-x86.MIBC.Runtime , optimization.PGO.CoreCLR
 From Version 1.0.0-prerelease.21410.2 -> To Version 1.0.0-prerelease.21411.3

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

Microsoft.DotNet.XUnitExtensions , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.Build.Tasks.Workloads , 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.CodeAnalysis , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.GenFacades , Microsoft.DotNet.GenAPI , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.SharedFramework.Sdk
 From Version 6.0.0-beta.21406.6 -> To Version 6.0.0-beta.21410.8

* Update dependencies from https://github.com/dotnet/runtime-assets build 20210811.3

System.ComponentModel.TypeConverter.TestData , System.Drawing.Common.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Windows.Extensions.TestData
 From Version 6.0.0-beta.21405.1 -> To Version 6.0.0-beta.21411.3

* Update dependencies from https://github.com/dotnet/hotreload-utils build 20210811.1

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 1.0.2-alpha.0.21404.1 -> To Version 1.0.2-alpha.0.21411.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Juan Hoyos <juan.hoyos@microsoft.com>
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

updated msquic bits (#57285)

Throw NotSupportedException when serializing Date/TimeOnly types (#57273)

* throw NotSupportedException when serializing Date/TimeOnly types

* address feedback

Fix source indexer failing official builds (#57336)

Works around https://github.com/dotnet/arcade/issues/7747 to unblock official builds.
This change will revert itself with the next Arcade update.

Create WindowsDesktop transport package (#57314)

As we don't include reference assemblies in our packages anymore, we
need a transport package for WindowsDesktop which contains the necessary
reference assemblies. Using the same layout as aspnetcore's package
which worked well in the past.

fix IncompleteReadCantSetPositionBeyondEndOfFile test (#57271)

* use ten unique arrays, not the same array 10 times (this should fix the test failure)

* don't run this test for the old FileStream implementation

Expression.Default and Expression.New with structs with parameterless constructors (#57034)

Add triple slash comments for new System.Drawing API added in 6.0 (#57313)

* Add triple slash comments for new System.Drawing API added in 6.0

Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>

Disable System.Drawing.Common tests for mobile workloads and skip a few other tests (#57114)

Since the library is PNSE by default, make sure the tests do not run on mobile.
Also skips a few other tests.

Closes #57184

[mini] Add GC Unsafe transitions in mono_pmip (#57290)

Add a new mono_pmip_u that doesn't do the transition.

The intent is that Mono developers in the debugger can call still call
mono_pmip_u if the thread state machine is in a broken state.

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

More xml docs (#57039)

* Update Handle docs

* Some easy xmldoc updates

* typo

* typo

* another

* Build breaks with M.IO.R.csproj

* updates

Turn on disabled tracing tests (#56507)