platform/upstream/dotnet/runtime.git
3 years agoAdd static PBKDF2 one-shot methods
Kevin Jones [Mon, 22 Feb 2021 20:53:59 +0000 (15:53 -0500)]
Add static PBKDF2 one-shot methods

This avoids the disposable object creation for Rfc2989DeriveBytes and the
state tracking.

Since the one-shots just call into platform routines (when available) it
should allow for higher iteration counts for the same clock time.

3 years agoEmptyAiaResponseIsIgnored should use incomplete chain.
Kevin Jones [Mon, 22 Feb 2021 20:51:47 +0000 (15:51 -0500)]
EmptyAiaResponseIsIgnored should use incomplete chain.

Since the certificate names were the same, different tests were
colliding with caches since they had the same certificate name.

In addition to the simple fix, this changes the test to build a partial
chain using two intermediates. The leaf's issuing intermediate is
added to the ExtraStore and the second intermediate is expected
to be fetched by AIA (and fail with an empty response).

This is to help coerce chain builders treat it as a partial chain,
and can actually make some progress, as opposed to the leaf being
treated as an invalid root.

3 years agoMake sure pal_icushim_static.c is linked in even if no symbols are used from it,...
Zoltan Varga [Mon, 22 Feb 2021 20:35:17 +0000 (15:35 -0500)]
Make sure pal_icushim_static.c is linked in even if no symbols are used from it, so the EMSCRIPTEN_KEEPALIVE functions in it are actually kept. (#48550)

3 years agoSupport PolynomialMultiplyWideningLower and PolynomialMultiplyWideningUpper (#48525)
Fan Yang [Mon, 22 Feb 2021 20:01:10 +0000 (15:01 -0500)]
Support PolynomialMultiplyWideningLower and PolynomialMultiplyWideningUpper (#48525)

3 years agoMinor msbuild code cleanup in installer partition (#48603)
Viktor Hofer [Mon, 22 Feb 2021 19:38:21 +0000 (20:38 +0100)]
Minor msbuild code cleanup in installer partition (#48603)

* Minor code clean-up in installer partition

* Move prepare-artifacts.proj into installer root

3 years agoPackage dotnet-pgo as a dotnet tool (#48542)
David Wrighton [Mon, 22 Feb 2021 19:30:05 +0000 (11:30 -0800)]
Package dotnet-pgo as a dotnet tool (#48542)

- The package is marked as non-shipping as we're not ready to make it part of the official product at this time, but this should make it easier to consume in downstream repos.

3 years agoTransition JIT memory management from domains to memory managers (#48483)
Zoltan Varga [Mon, 22 Feb 2021 19:02:32 +0000 (14:02 -0500)]
Transition JIT memory management from domains to memory managers (#48483)

* [jit] Add a MonoJitMemoryManager structure, which is the same as the MonoJitDomainInfo structure, but owned by a MonoMemoryManager instead of a domain.

* Move static_rgctx_trampoline_hash to MonoJitMemoryManager.

* Move jump_target_hash to MonoJitMemoryManager.

* Move jump_trampoline_hash to MonoJitMemoryManager.

* Move jit_trampoline_hash to MonoJitMemoryManager.

* Move delegate_trampoline_hash to MonoJitMemoryManager.

* Move method_code_hash to MonoJitMemoryManager.

* Move more fields to MonoJitMemoryManager.

* Move llvm data to MonoJitMemoryManager.

* Move arch_seq_points field to MonoJitMemoryManager.

* Remove unused fields from MonoJitDomainInfo.

* Use the default alc memory manager only for now.

* Remove MonoJitDomainInfo which is now unused.

3 years agoRead basic properties of a X509 certificate using Android-built-in Crypto APIs (...
Elinor Fung [Mon, 22 Feb 2021 18:50:45 +0000 (10:50 -0800)]
Read basic properties of a X509 certificate using Android-built-in Crypto APIs (#48425)

3 years agoRemove hardcoded PackageVersion in Platforms pkg (#48596)
Viktor Hofer [Mon, 22 Feb 2021 18:05:55 +0000 (19:05 +0100)]
Remove hardcoded PackageVersion in Platforms pkg (#48596)

Mapping the package to `$(ProductVersion)` would be wrong in servicing cases. Hence just removing the hardcoded value entirely.

3 years agoFix ThreadPool events validation test (#48588)
Sung Yoon Whang [Mon, 22 Feb 2021 17:28:45 +0000 (09:28 -0800)]
Fix ThreadPool events validation test (#48588)

* Fix ThreadPool events validation test

* space

3 years agoRemove unaligned offsets for mapped layout (#42464)
Sven Boemer [Mon, 22 Feb 2021 17:22:39 +0000 (09:22 -0800)]
Remove unaligned offsets for mapped layout (#42464)

* Remove unaligned offsets for mapped layout

* Assert on windows only

- Unix will use the mapped layout with an offset for bundled assemblies
- Windows still should't create a mapped layout for bundles

3 years ago[master] Update dependencies from mono/linker (#48597)
dotnet-maestro[bot] [Mon, 22 Feb 2021 17:12:32 +0000 (18:12 +0100)]
[master] Update dependencies from mono/linker (#48597)

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.2.21118.4 -> To Version 6.0.100-preview.2.21122.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
3 years agoDelete publish-final.proj and clean-up installer.tasks unused files (#48598)
Viktor Hofer [Mon, 22 Feb 2021 16:54:36 +0000 (17:54 +0100)]
Delete publish-final.proj and clean-up installer.tasks unused files (#48598)

* Delete publish-final.proj and delete unused installer.tasks files

* Avoid extra restore phase for tasks

The repo local tasks don't participate in the repo restore anymore,
hence avoiding the extra restore phase which should improve reliability
and build times.

3 years agoDelete updateversions.proj (#48593)
Viktor Hofer [Mon, 22 Feb 2021 15:53:49 +0000 (16:53 +0100)]
Delete updateversions.proj (#48593)

dead code

3 years agoFix doc comment typo in JsonSerializerOptions.cs (#48590)
Eric Mutta [Mon, 22 Feb 2021 13:54:03 +0000 (16:54 +0300)]
Fix doc comment typo in JsonSerializerOptions.cs (#48590)

The following sentence in the doc comments for the `IgnoreReadOnlyFields` member:

> A property is read-only if it isn't marked with the <c>readonly</c> keyword.

Should read:

> A field is read-only if it is marked with the <c>readonly</c> keyword.

3 years agoFix another doc comment typo in JsonSerializerOptions.cs (#48591)
Eric Mutta [Mon, 22 Feb 2021 13:52:10 +0000 (16:52 +0300)]
Fix another doc comment typo in JsonSerializerOptions.cs (#48591)

* Fix another doc comment typo in JsonSerializerOptions.cs

The following sentence in the doc comments for the `IncludeFields` member:

> Determines whether fields are handled serialization and deserialization.

Should read:

> Determines whether fields are handled **during** serialization and deserialization.

* Update src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonSerializerOptions.cs

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>
3 years agoAdd LINQ APIs for Index and Range (#28776) (#48559)
Dixin [Mon, 22 Feb 2021 13:43:14 +0000 (05:43 -0800)]
Add LINQ APIs for Index and Range (#28776) (#48559)

* Implement dotnet/runtime#28776: Implement LINQ APIs for index and range.

* Implement dotnet#28776: Implement LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Code review update.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Code review update.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Code review update.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update ElementAt, keep the original behavior.

* Implement dotnet#28776: LINQ APIs for index and range. Update ElementAt, keep the original behavior.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update ElementAt, keep the original behavior.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Add unit tests for ElementAt, ElementAtOrDefault.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update unit tests.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update unit tests.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update unit tests.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update for merge.

* Implement #28776: LINQ APIs for index and range. Update unit tests.

3 years ago[interp] Inline more AggressiveInlining methods (#48513)
Vlad Brezae [Mon, 22 Feb 2021 13:34:08 +0000 (15:34 +0200)]
[interp] Inline more AggressiveInlining methods (#48513)

* [interp] Inline more AggressiveInlining methods

By default, the interpreter stops inlining when encountering a call that it cannot inline or a throw. Inline the method anyway if the aggressive inlining attribute is present.

* [interp] Fix some long conditional branch opcodes

3 years agoDelete unused installer properties (#48569)
Viktor Hofer [Mon, 22 Feb 2021 11:49:14 +0000 (12:49 +0100)]
Delete unused installer properties (#48569)

* Delete unused msbuild properties, targets and files

3 years agoImprove S.DS.P tests, fix async API, and improve error messages (#47703)
Dan Moseley [Mon, 22 Feb 2021 01:25:21 +0000 (17:25 -0800)]
Improve S.DS.P tests, fix async API, and improve error messages (#47703)

3 years agoSimplify connection usability logic (#48446)
Geoff Kizer [Mon, 22 Feb 2021 00:17:48 +0000 (16:17 -0800)]
Simplify connection usability logic  (#48446)

* simplify connection usability logic

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>
3 years agoJIT: remove JitGuardedDevirtualizationGuessBestClass (#48574)
Andy Ayers [Sun, 21 Feb 2021 23:32:34 +0000 (15:32 -0800)]
JIT: remove JitGuardedDevirtualizationGuessBestClass (#48574)

And slightly rearrange the two bits of code that consider guarded
devirualization with the goal of eventually unifying them.

Fixes #48564.

3 years agoAdd disabling test policy to the docs (#48575)
Anirudh Agnihotry [Sun, 21 Feb 2021 21:47:14 +0000 (13:47 -0800)]
Add disabling test policy to the docs (#48575)

* add disabling test policy

* Update docs/infra/buildtriage.md

Co-authored-by: Dan Moseley <danmose@microsoft.com>
3 years agoFix #28035 Refactor DLL imports interop classes (#48556)
Badre BSAILA [Sun, 21 Feb 2021 21:45:59 +0000 (22:45 +0100)]
Fix #28035 Refactor DLL imports interop classes (#48556)

* refactor interop classes

* move interop files to subdirectory

3 years agoEnable CG2 determinism tests on all architectures (#48526)
Tomáš Rylek [Sun, 21 Feb 2021 21:05:37 +0000 (22:05 +0100)]
Enable CG2 determinism tests on all architectures (#48526)

I have noticed that these two tests are still conditionally
executed on x64 only. I don't see why they shouldn't work on
other architectures.

Thanks

Tomas

3 years agoupdate branding to preview3 (#48530)
Anirudh Agnihotry [Sun, 21 Feb 2021 19:00:48 +0000 (11:00 -0800)]
update branding to preview3 (#48530)

3 years agoReuse SDK target for latest commit hash (#48561)
Adeel Mujahid [Sun, 21 Feb 2021 12:06:16 +0000 (14:06 +0200)]
Reuse SDK target for latest commit hash (#48561)

* Reuse SDK target for latest commit hash

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
* Further clean-up corehost.proj

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
3 years agoAdd assembly name to SuperPMI collection (#48534)
Bruce Forstall [Sat, 20 Feb 2021 21:56:50 +0000 (11:56 -1000)]
Add assembly name to SuperPMI collection (#48534)

* Add assembly name to SuperPMI collection

For each class, add JIT/EE calls to getAssemblyName so the SuperPMI
collections contain this information. This can be useful to try and track
down where a particular function in a large method context hive came
from.

Only do this when `COMPlus_EnableExtraSuperPmiQueries=1` is set.

* Formatting

3 years agoEnable Hot Reload API on Linux (#48497)
Mike McLaughlin [Sat, 20 Feb 2021 21:51:39 +0000 (13:51 -0800)]
Enable Hot Reload API on Linux (#48497)

Enable Hot Reload API on Linux

Only define EnC_SUPPORTED for vm/ee on xplat. Doesn't enable the ENC ICorDebug API.

Enable hotreload api tests on Linux/MacOS. API not implemented on arm/arm64

Remove all the ENC_DELTA_HACK code. Remove more ENC_DELTA_HACK code from ilasm

3 years ago[mono] Remove MonoRuntimeUnhandledExceptionPolicy (#48555)
Aleksey Kliger (λgeek) [Sat, 20 Feb 2021 20:26:30 +0000 (15:26 -0500)]
[mono] Remove MonoRuntimeUnhandledExceptionPolicy (#48555)

policy was always set to MONO_UNHANDLED_POLICY_CURRENT.  The
legacy (pre-.NET 2.0) policy was not used.

3 years agoJIT: workaround for unreliable change detection in fgReorderBlocks (#48516)
Andy Ayers [Sat, 20 Feb 2021 17:17:32 +0000 (09:17 -0800)]
JIT: workaround for unreliable change detection in fgReorderBlocks (#48516)

This method costs trees, which can in turn modify the IR by swapping operands.
As a result the bool value returned doesn't properly reflect whether any
changes happened.

This impacts proper reporting phase status by `optOptimizeLayout'. Since phase
status just gates post-phase dumps and checks, we'll just claim this phase
always modifies IR.

Add similar workaround for `optInvertLoops`.

Fixes #48494.
Fixes #48495.

3 years agoFix JIT rolling build (#48552)
Adeel Mujahid [Sat, 20 Feb 2021 16:34:53 +0000 (18:34 +0200)]
Fix JIT rolling build (#48552)

3 years agoUpdate hardcoded rids for targeting/runtime pks (#48459)
Viktor Hofer [Sat, 20 Feb 2021 12:18:35 +0000 (13:18 +0100)]
Update hardcoded rids for targeting/runtime pks (#48459)

* Update hardcoded rids for targeting/runtime pks

3 years agoFormat (#48548)
Next Turn [Sat, 20 Feb 2021 05:07:06 +0000 (13:07 +0800)]
Format (#48548)

3 years agoBuilding singlefilehost with clr partition. (#48254)
Vladimir Sadov [Sat, 20 Feb 2021 04:40:14 +0000 (20:40 -0800)]
Building singlefilehost with clr partition. (#48254)

* building single file host with clr partition

* make coreclr_static a STATIC lib again

3 years ago[mono] Implement public hot reload API (#48380)
Aleksey Kliger (λgeek) [Fri, 19 Feb 2021 20:11:03 +0000 (15:11 -0500)]
[mono] Implement public hot reload API (#48380)

* [mono] Implement public hot reload API

Also add ApplyUpdateSdb that takes byte[] arguments for use with
mono/debugger-libs as its awkward to call a ROS<byte> method from the debugger.

Contributes to https://github.com/dotnet/runtime/issues/45689

* Update src/mono/System.Private.CoreLib/src/System/Reflection/Metadata/AssemblyExtensions.cs

Co-authored-by: Marek Safar <marek.safar@gmail.com>
* Don't allow changes to System.Private.CoreLib

* formatting

* more formatting

* Add a TODO to remove ApplyUpdateSdb

* update DeltaHelper

* don't use ApplyUpdateSdb in DeltaHelper

* throw InvalidOpertionException for corlib changes with hot reload disabled

* Run the test on mono on all platoforms; CoreCLR windows only

* Use resource string

* Address review feedback

Co-authored-by: Marek Safar <marek.safar@gmail.com>
3 years agoTemporarily work around publishing issues with disk space (#48521)
Matt Galbraith [Fri, 19 Feb 2021 20:09:10 +0000 (12:09 -0800)]
Temporarily work around publishing issues with disk space (#48521)

3 years agoWasm download fix (#48514)
Drew Scoggins [Fri, 19 Feb 2021 19:08:57 +0000 (11:08 -0800)]
Wasm download fix (#48514)

* Don't use container for WASM

* Remove dashes

* Remove comments for testing

3 years agoDisable TestOnExecuteCustomCommand() for WSC (#48501)
Dan Moseley [Fri, 19 Feb 2021 18:59:19 +0000 (10:59 -0800)]
Disable TestOnExecuteCustomCommand() for WSC (#48501)

3 years agoRemove StringBuilderCache from Version.ToString (#48511)
Stephen Toub [Fri, 19 Feb 2021 18:38:31 +0000 (13:38 -0500)]
Remove StringBuilderCache from Version.ToString (#48511)

Rather than using a StringBuilder in both ToString and TryFormat, we can just write ToString in terms of TryFormat and use stack space for the small string being generated.  As part of this, I removed the extra code optimizing ToString(0) and ToString(1), as I can't find any real code anywhere that does that and it's not worth the extra lines.

3 years agoFix EqualityComparer<T>.Default for nullable (#48161)
Egor Bogatov [Fri, 19 Feb 2021 18:00:28 +0000 (21:00 +0300)]
Fix EqualityComparer<T>.Default for nullable (#48161)

3 years agoNo const arg to temp (#48308)
Alex Covington [Fri, 19 Feb 2021 17:55:18 +0000 (09:55 -0800)]
No const arg to temp (#48308)

* Starting to add IsInvariant helper

* Finished IsInvariant helper. Starting testing

* Removed IsInvariant from GenTree to avoid asserts

* Formatting

* Fixed missing not operator

* Changed AsOp() call to 'this' to avoid failing assertions

* Formatting

* Update src/coreclr/jit/morph.cpp

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
* Update src/coreclr/jit/compiler.h

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
* Apply suggestions from code review

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
* Change arguments to const to avoid cast

* Formatting

* delete `IsInvariant` from Compiler.

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
3 years agoDon't build Quic for platforms where it's not supported (#48480)
Marek Safar [Fri, 19 Feb 2021 17:46:58 +0000 (18:46 +0100)]
Don't build Quic for platforms where it's not supported (#48480)

3 years agoAnnotate framework assemblies as trimmable (#48428)
Sven Boemer [Fri, 19 Feb 2021 17:27:41 +0000 (09:27 -0800)]
Annotate framework assemblies as trimmable (#48428)

* Annotate framework assemblies as trimmable

This will opt them into trimming with the behavior described at https://github.com/mono/linker/blob/main/docs/design/trimmed-assemblies.md#assemblymetadataistrimmable-true.

* Add a condition to let projects opt out

A project just needs to set <SetIsTrimmable>false</SetIsTrimmable>, and
it won't get the trimmable metadata.

I also got rid of the SkipFrameworkAssemblyMetadata check since I don't
think the IsTrimmable metadata should be tied to whether we are building
a framework assembly, and this property was unused as far as I could tell.

3 years agoFix Socket.SendFile test: SliceBuffers_Success (#48457)
Anton Firszov [Fri, 19 Feb 2021 16:37:05 +0000 (17:37 +0100)]
Fix Socket.SendFile test: SliceBuffers_Success (#48457)

* do not expect a single receive

* use expected.Length

* enable test

3 years agoRemove trimming attributes from any assembly. (#48424)
Eric Erhardt [Fri, 19 Feb 2021 15:10:48 +0000 (09:10 -0600)]
Remove trimming attributes from any assembly. (#48424)

* Remove trimming attributes from any assembly.

These attributes are allowed to be in any assembly, for assemblies that are built using older TFMs where the attributes didn't exist.

Fix #48399

* Remove nullable attributes as well.

3 years agoNormalized names SIMD_OP_LLVM_* -> SIMD_OP_ARM64_* (#48489)
Nathan Ricci [Fri, 19 Feb 2021 14:41:27 +0000 (09:41 -0500)]
Normalized names SIMD_OP_LLVM_* -> SIMD_OP_ARM64_* (#48489)

Co-authored-by: Nathan Ricci <naricc@xam-emag-01.redmond.corp.microsoft.com>
3 years ago[master] Update dependencies from dotnet/xharness (#48452)
dotnet-maestro[bot] [Fri, 19 Feb 2021 10:51:53 +0000 (10:51 +0000)]
[master] Update dependencies from dotnet/xharness (#48452)

[master] Update dependencies from dotnet/xharness

3 years agoExpand PriorityQueue test coverage (#48420)
Eirik Tsarpalis [Fri, 19 Feb 2021 10:02:52 +0000 (10:02 +0000)]
Expand PriorityQueue test coverage (#48420)

* make comparer field readonly

* abstract IComparer in PriorityQueue generic tests

does minor refactorings and adds new tests

* add PriorityQueue property tests

* address feedback

* add seed for deterministic runs

* apply var fixes in property tests

* replace FsCheck properties with theories

3 years agoRemoving myself from codeowners file (#48502)
David Karlaš [Fri, 19 Feb 2021 09:24:33 +0000 (10:24 +0100)]
Removing myself from codeowners file (#48502)

In past I was involved in mono/debugger-agent.c, but not anymore, hence removing myself from CODEOWNERS file

3 years ago[interp] Small interp tweaks (#48412)
Vlad Brezae [Fri, 19 Feb 2021 09:07:44 +0000 (11:07 +0200)]
[interp] Small interp tweaks (#48412)

* [interp] Fix overflow in PROFILE_INTERP log

* [interp] Replace LDFLDA of offset 0 field with MOV

Commonly used when accessing first field of valuetypes, eg span operations. Will help with cprop and remove redundant instruction.

* [interp] Remove redundant opcode

3 years agoRemove redundant static ctor from LoggerFactory (#48478)
Marek Safar [Fri, 19 Feb 2021 08:51:54 +0000 (09:51 +0100)]
Remove redundant static ctor from LoggerFactory (#48478)

3 years agoUpdate dependencies from https://github.com/mono/linker build 20210218.4 (#48493)
dotnet-maestro[bot] [Fri, 19 Feb 2021 05:04:51 +0000 (05:04 +0000)]
Update dependencies from https://github.com/mono/linker build 20210218.4 (#48493)

[master] Update dependencies from mono/linker

3 years agoMake ApkBuilder optionally generate debuggable apk (#48486)
Elinor Fung [Fri, 19 Feb 2021 04:57:25 +0000 (20:57 -0800)]
Make ApkBuilder optionally generate debuggable apk (#48486)

3 years ago[mono] More domain cleanups. (#48291)
Zoltan Varga [Fri, 19 Feb 2021 04:13:58 +0000 (23:13 -0500)]
[mono] More domain cleanups. (#48291)

* Remove the MonoClassRuntimeInfo structure, store the runtime vtable directly in MonoClass.

* Remove the 'domain' argument from mono_class_vtable_checked ().

* Remove the 'domain' argument from the object allocation functions.

* Remove the 'domain' argument from the m_.._alloc methods.

* Revert accidental changes.

* Remove more unused 'domain' parameters.

* Remove more remoting code.

* Remove shared code support.

* Remove the 'domain' argument from the IMT builder functions.

* Remove the 'domain' argument from the m_method_ functions.

* Remove more domain usage from the JIT backends.

* Remove the MONO_ARCH_HAVE_PATCH_CODE_NEW ifdef, all architectures support it now.

* Remove 'domain' argument from mini_jit_info_table_find ().

* Remove 'domain' argument from mono_arch_unwind_frame ().

* Remove unused/unfinished AOT cache code.

* Remove the 'domain' argument from mono_resolve_patch_target ().

* Remove domains from the stack walk/exception handling code.

* Remove domains from mono-debug.c.

The memory in mono-debug.c is now owned by a MonoMemoryManager instead of by a domain.

* Remove unused mono-mlist.{h,c}.

* Fix the coverage profiler build.

* Fix the build.

3 years agoJSON Console Logger not requiring scopes to implement IReadOnlyCollection (#48453)
daveMueller [Fri, 19 Feb 2021 03:35:00 +0000 (04:35 +0100)]
JSON Console Logger not requiring scopes to implement IReadOnlyCollection (#48453)

3 years ago[wasm] Improve AOT builds (#48184)
Ankit Jain [Fri, 19 Feb 2021 02:40:41 +0000 (21:40 -0500)]
[wasm] Improve AOT builds (#48184)

* [wasm] pack: Let the workload configure the runtime pack path

* [wasm] update webassembly pack description

* [wasm] Makefile: fix being able to `make build`

* [wasm] Fix AOT compilation to work before publish

Instead of using runtime pack, and other assemblies from the
publish folder, we are now using them directly from the source
location. This means that `System.Private.CoreLib` can be in a different
directory, than the assembly being precompiled.

So, we need to set `MONO_PATH` to the relevant paths that will have the assemblies.

This manifests during blazor workload based builds.

* [wasm][aot] Don't strip the source assemblies in place

Currently we run `mono-cil-strip` on assemblies, in the AOT case, but we
do that in-place.

This is problematic because the assemblies from the runtime pack will
get modified in their original location (nuget/etc), since we don't use
them from the publish folder anymore.

- This would mean that a subsequent attempt to build with AOT would fail
  because now the original assemblies are already stripped!

- this also becomes apparent if you have the runtime pack installed in a
  system dotnet folder, eg. as a pack - `/usr/local/share/dotnet/packs/Microsoft.NETCore.App.Runtime.browser-wasm/`,
  because a regular user wouldn't have permissions to modify that.

Solution: Write the stripped assemblies into the obj dir

* [wasm] Make the final list of wasm assemblies as an item

- Blazor skips the generation of the app bundle when running
 `WasmBuildApp` target.
- Because of the previous commit, the locations for the final assemblies
  could be different (when aot+stripping), which needs to be surfaced
  for blazor. So, generate a `@(WasmAssembliesFinal)` item for that.

* [wasm] Allow enabling assert messages with $(MonoEnableAssertMessages)

* [wasm][aot] WasmApp.targets: check that we have .bc file for each assembly

Just a sanity check to ensure that due to some build authoring error, we
are not building dotnet.js/wasm with no .bc files! That just shows up as
an assert:

`* Assertion at /Users/radical/dev/r3/src/mono/mono/mini/aot-runtime.c:2651, condition `<disabled>' not met`

.. which is:
    `g_assert (info->version == MONO_AOT_FILE_VERSION);`

..  `0 != MONO_AOT_FILE_VERSION`

* [wasm] remove UsingTask for non-exitant WasmAOT

* MonoCrossAot pkg: use mono-aot-cross.exe on windows

* [wasm] Fix up emscripten command invocations for windows

* [wasm] WasmApp.targets: move some property inits closer to their use

* [wasm] fix build for browser-profile sample

* [wasm] fix emcc invocation for getting the version

* [wasm] Add new RunWithEmSdkEnv task

* [wasm] Use `RunWithEmSdkEnv` to run `emcc` commands

* [wasm] AotCompiler: use platform specific path separator

* [wasm] RunWithEmSdkEnv: quote path on windows

* [wasm] Use new ReadEmccVersion to get the versions

This simplifies the target too, and works on windows.

* Update src/installer/pkg/sfx/Microsoft.NETCore.App/Microsoft.NETCore.App.MonoCrossAOT.Sdk.props

Co-authored-by: Pranav K <prkrishn@hotmail.com>
* fix messageimportance for debug msg

* Fix emcc invocation on windows, thanks to @pranavkm

* Don't strip assemblies with mono-cil-strip for aotinterp mode

* [wasm] Remove ReadEmccVersion task, and use msbuild to get the version

.. string instead. We do this by looking for the line with `'emcc'` in
it, since the other lines are the license text. Hopefully, this isn't
too brittle, since we are invoking `emcc --version`.

* Fix emcc version extraction to not depend on a specific string

Co-authored-by: Pranav K <prkrishn@hotmail.com>
3 years agoDo not emit Avx2 instruction `vpcmpeqd ymmReg, ymmReg, ymmReg` for `Vector256<T>...
Egor Chesakov [Fri, 19 Feb 2021 02:27:07 +0000 (18:27 -0800)]
Do not emit Avx2 instruction `vpcmpeqd ymmReg, ymmReg, ymmReg` for `Vector256<T>.AllBitsSet` intrinsic when Avx2 is not supported (#48383)

Emit `vcmptrueps ymmReg, ymmReg, ymmReg` instead

3 years agoProbe libnuma.so.1.0.0 before its symlinks (#48133)
Adeel Mujahid [Fri, 19 Feb 2021 01:41:07 +0000 (03:41 +0200)]
Probe libnuma.so.1.0.0 before its symlinks (#48133)

On Linux, libnuma.so{.1} are symlinks to libnuma.so.1.0.0.

When installed from default package manager; Ubuntu (libnuma-dev) and
Alpine (numactl-dev), found these two symlinks:
`{libnuma.so,libnuma.so.1} -> libnuma.so.1.0.0`

On Fedora (numactl-libs):
`libnuma.so.1 -> libnuma.so.1.0.0`
(there is no version-less variant)

PR adjusts probing fallback flow based on this info.

3 years agoUse newer microsoft.com certificate chain in X.509 tests
Kevin Jones [Fri, 19 Feb 2021 01:28:30 +0000 (20:28 -0500)]
Use newer microsoft.com certificate chain in X.509 tests

The old certificate chain used a root certificate that is deprecated
and is starting to disappear from some trust stores. The removal of
the root certificate led to a few failing tests, and other tests not
running since they were conditional on the trust of that certificate.

3 years agoAdd runs for Alpine performance measurement (#47304)
Drew Scoggins [Fri, 19 Feb 2021 00:47:58 +0000 (16:47 -0800)]
Add runs for Alpine performance measurement  (#47304)

* WIP

* Add Alpine specific steps

* Fix musl string

* Fix musl string

* Add comment block

* Update Helix precommand for Alpine

* Add container support for Perf leg

* Remove dupe container

* Add apk installs

* Add libgdiplus

* Fix typo and remove --force-install from pip command

* Remove --force-reinstall from Alpine job

* Add pip upgrade step

* Add cargo install

* Add back full runs

* Remove PR line

* Remove temp files

* Fix spacing issue

3 years agoResolve ILLink warnings in System.Linq.Queryable (#48305)
Eric Erhardt [Thu, 18 Feb 2021 23:52:11 +0000 (17:52 -0600)]
Resolve ILLink warnings in System.Linq.Queryable (#48305)

* Resolve ILLink warnings in System.Linq.Queryable

Contributes to #45623

* Respond to PR feedback.

Annotate public AsQueryable() method as RequiresUnreferencedCode.

3 years agoAdd hot reload apply changes API: AssemblyExtensions.ApplyUpdate (#48366)
Mike McLaughlin [Thu, 18 Feb 2021 23:02:17 +0000 (15:02 -0800)]
Add hot reload apply changes API: AssemblyExtensions.ApplyUpdate (#48366)

Add hot reload apply changes API: AssemblyExtensions.ApplyUpdate

Issue: https://github.com/dotnet/runtime/issues/45689

Currently Windows only. Fail hot reload API if debugging.

Added some simple invalid parameter testing to the new ApplyUpdate API.

3 years agoFix AV in NativeRuntimeEventSource QCalls (#48414)
Sung Yoon Whang [Thu, 18 Feb 2021 22:38:43 +0000 (14:38 -0800)]
Fix AV in NativeRuntimeEventSource QCalls (#48414)

* Fix AV in NativeRuntimeEventSource QCalls

* Add test

* Use WriteEventCore APIs to emit events on Mono

* Separate out Mono/CoreCLR implementations to different files

* Fix wasm build

3 years agoRemove partitioning from CancellationTokenSource (#48251)
Stephen Toub [Thu, 18 Feb 2021 22:00:38 +0000 (17:00 -0500)]
Remove partitioning from CancellationTokenSource (#48251)

When CancellationTokenSource was original created, the expectation was that a majority use case would be lots of threads in parallel registering and unregistering handlers. This led to a design where CTS internally partitions its registrations to minimize contention between threads contending on its internal data structures. While that certainly comes up in practice, a much more common case is just one thread registering and unregistering at a time as a CancellationToken unique to a particular operation (e.g. a linked token source) is passed down through it, with various levels of the chain registering and unregistering from that non-concurrently-used token source. And having such partitioning results in non-trivial allocation overheads, in particular for a short-lived CTS with which only one or a few registrations are employed in its lifetime. This change removes that partitioning scheme; all scenarios end up with less memory allocation, and non-concurrent scenarios end up measurably faster... scenarios where there is contention do take a measurable hit, but given that's the rare case, it's believed to be the right trade-off (when in doubt, it's also the simpler implementation).

As long as I was refactoring a bunch of code, I fixed up a few other things along the way:
- Avoided allocating while holding the instance's spin lock
- Made WaitForCallbackAsync into a polling async method rather than an async-over-sync method
- Changed the state values to be 0-based to avoid needing to initialize _state to something other than 0 in the common case
- Used existing throw helpers in a few more cases
- Renamed a few methods, and made a few others to be local functions

3 years agoRemove !CPU_HAS_FP_SUPPORT code (#48436)
Bruce Forstall [Thu, 18 Feb 2021 21:56:57 +0000 (11:56 -1000)]
Remove !CPU_HAS_FP_SUPPORT code (#48436)

* Remove !CPU_HAS_FP_SUPPORT code

CPU_HAS_FP_SUPPORT is always defined and we don't anticipate needing
the emulation support.

Fixes #47708

* Remove fgMorphToEmulatedFP declaration

3 years agoDisable slicebuffers_success and RunContinueWithStressTestsNoState test (#48442)
Anirudh Agnihotry [Thu, 18 Feb 2021 21:54:06 +0000 (13:54 -0800)]
Disable slicebuffers_success and RunContinueWithStressTestsNoState test (#48442)

* disable long failing tests

* keeping the skipOncoreclr attribute

3 years agomake symbolsPath to default pack path (#48384)
Anirudh Agnihotry [Thu, 18 Feb 2021 21:53:36 +0000 (13:53 -0800)]
make symbolsPath to default pack path (#48384)

3 years agoFix test failures (#48302)
Maxim Lipnin [Thu, 18 Feb 2021 20:53:18 +0000 (23:53 +0300)]
Fix test failures (#48302)

* Fix test failures

* Remove iOS/tvOS-specific platform detection

3 years agoJIT: change loop inversion edge weight updates and add phase (#48364)
Andy Ayers [Thu, 18 Feb 2021 20:34:35 +0000 (12:34 -0800)]
JIT: change loop inversion edge weight updates and add phase (#48364)

Rename `fgOptWhileLoop` to `optInvertWhileLoop` (using terminology from
Muchnick). Split off this transformation into a new phase so it is easier
to see its impact. Make the block updates / reorderings that follow into
a proper phase as well.

Use the test block exit likelihoods to update the profile weights for the
edges involved in loop inversion. Because edge weight updates are now
consistent we no longer need to recompute edge weights afterwards.

Rename `optOptimizeLoops` to `optFindLoops` since it no longer optimizes.

3 years agoUse llvm-objcopy on Linux x64 (#48444)
Adeel Mujahid [Thu, 18 Feb 2021 20:34:00 +0000 (22:34 +0200)]
Use llvm-objcopy on Linux x64 (#48444)

3 years agoSupport remaining Sha1 intrinsics on Arm (#48413)
Fan Yang [Thu, 18 Feb 2021 20:33:37 +0000 (15:33 -0500)]
Support remaining Sha1 intrinsics on Arm (#48413)

* Add support for Sha1 API with vector64 arguments

* Promote return to vector64

* Return a 2 x i32 instead of i64

* Update src/mono/mono/mini/mini-llvm.c

* Fix format

Co-authored-by: imhameed <imhameed@microsoft.com>
3 years agoFix all remaining System.Security.Cryptography.Algorithms test suite crashes. (#48432)
Jeremy Koritzinsky [Thu, 18 Feb 2021 20:15:23 +0000 (12:15 -0800)]
Fix all remaining System.Security.Cryptography.Algorithms test suite crashes. (#48432)

* Fix all remaining System.Security.Cryptography.Algorithms test suite crashes.

With this + PR #48348, the test suite runs to completion and reports the following results:

> Tests run: 2527 Passed: 1285 Inconclusive: 0 Failed: 1227 Ignored: 7

So we've got a ways to go to get things passing, but now we can easily track it at least for this assembly.

* PR Feedback.

3 years ago[coop] Enter GC Unsafe mode in mono_gc_init_finalizer_thread (#48465)
Aleksey Kliger (λgeek) [Thu, 18 Feb 2021 20:12:25 +0000 (15:12 -0500)]
[coop] Enter GC Unsafe mode in mono_gc_init_finalizer_thread (#48465)

when lazy thread creation is used, this external API function is called
from a thread in GC Safe mode, so enter GC Unsafe before calling runtime
functions.

Fixes a double transition to GC Safe in mono_gc_pthread_create.

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

3 years agoImprove SuperPMI asserts (#48449)
Bruce Forstall [Thu, 18 Feb 2021 19:14:55 +0000 (09:14 -1000)]
Improve SuperPMI asserts (#48449)

Include `GetIndex` method calls in the assert message, to distinguish
the different assert cases in error output. E.g., the asserts
change from:

```
'index != -1' failed ...
```

(which was identical for several cases), to:

```
'GetClassAttribs->GetIndex(CastHandle(classHandle)) != -1' failed ...
```

This makes it easier to know where the problem is without debugging the failure.

3 years agoDisplay more SuperPMI details in log file (#48448)
Bruce Forstall [Thu, 18 Feb 2021 19:13:37 +0000 (09:13 -1000)]
Display more SuperPMI details in log file (#48448)

In particular, display missing info asserts, which give some hint
about what change might have caused the missing info to be requested.

3 years agoFix SuperPMI SEH exception handling (#48447)
Bruce Forstall [Thu, 18 Feb 2021 19:13:18 +0000 (09:13 -1000)]
Fix SuperPMI SEH exception handling (#48447)

Capture data in the filter; don't save a PEXCEPTION_POINTERS
that points to data that is is transient to the filter.

3 years agoRun dotnet.sh/cmd script before running dotnet restore (#48375)
Bruce Forstall [Thu, 18 Feb 2021 19:10:27 +0000 (09:10 -1000)]
Run dotnet.sh/cmd script before running dotnet restore (#48375)

* Fix SuperPMI Linux collections

1. Run dotnet for jitutils with jitutils dir as the current working dir.
2. Run dotnet.sh/cmd script before running bootstrap (and `dotnet restore`)
to ensure `.dotnet` directory exists.

3 years agoFix PerfMap file naming in Crossgen2 to match Crossgen1 (#48426)
Tomáš Rylek [Thu, 18 Feb 2021 19:03:58 +0000 (20:03 +0100)]
Fix PerfMap file naming in Crossgen2 to match Crossgen1 (#48426)

For single-file builds, PerfMap file name is expected to have
the file name <output file>.ni.{managed assembly MVID}.map.
This change fixes Crossgen2 to conform to that naming style.

Thanks

Tomas

3 years agoSupport profiler activation for same bitness architectures (x64+arm64 and x86+arm...
Mikhail Pilin [Thu, 18 Feb 2021 18:47:24 +0000 (19:47 +0100)]
Support profiler activation for same bitness architectures (x64+arm64 and x86+arm) on Windows ARM64 and macOS Apple Silicon. #39699 (#46868)

3 years agoFix usage of relative path on bundle extraction (#47200)
Mateo Torres-Ruiz [Thu, 18 Feb 2021 18:10:47 +0000 (10:10 -0800)]
Fix usage of relative path on bundle extraction (#47200)

* Don't append boolean

* Set working dir for the command only
Test different bunddle options when extracting to relative path

* Add error message
Use bundleOption parameter

* Normalize extraction dir path
Look for expected extracted files

* Only test for bundle options that extract files to disk
Update GetExtractedFiles

* Don't change value of env var

* On windows, forward slashes should be converted to standard dir sep

* Set COREHOST_TRACE in test

* Fixup path after appending rel path
Remove COREHOST_TRACE

* Add extension

* Remove netcoreappcompatmode test with backslash

* Don't modify dir sep

* Update GetFilesNeverExtracted
Skip 'foo/bar' in Windows

3 years agoFix issue #48345 (#48391)
Katelyn Gadd [Thu, 18 Feb 2021 17:17:27 +0000 (09:17 -0800)]
Fix issue #48345 (#48391)

Fix issue #48345: Return value marshaling was incorrect for bound methods

3 years agoImplemented abs saturate, absolute compare intrinsics. (#48361)
Nathan Ricci [Thu, 18 Feb 2021 16:34:26 +0000 (11:34 -0500)]
Implemented abs saturate, absolute compare intrinsics. (#48361)

Co-authored-by: Nathan Ricci <naricc@xam-emag-01.redmond.corp.microsoft.com>
3 years agoAdd tvOS functional tests (#48331)
Maxim Lipnin [Thu, 18 Feb 2021 15:27:52 +0000 (18:27 +0300)]
Add tvOS functional tests (#48331)

3 years agoFix OOB src project configurations (#48318)
Viktor Hofer [Thu, 18 Feb 2021 14:38:27 +0000 (15:38 +0100)]
Fix OOB src project configurations (#48318)

* Fix OOB src project configurations

* Optimize netstandard2.1 DI assembly

* Move netcoreapp3.0 tfm to netstandard2.1

* Remove NetCoreAppCurrent config from M.E.DI

3 years agoDelete a few lines of dead code in ListDictionaryInternal (#48404)
Stephen Toub [Thu, 18 Feb 2021 14:01:06 +0000 (09:01 -0500)]
Delete a few lines of dead code in ListDictionaryInternal (#48404)

3 years agoImprove error message when bundler fails due to duplicate entries (#48336)
Vitek Karas [Thu, 18 Feb 2021 13:47:09 +0000 (05:47 -0800)]
Improve error message when bundler fails due to duplicate entries (#48336)

There are multiple possible cases which will make the bundler fail with duplicate entries. Currently there's no indication of which entries caused the problem, making it really hard to diagnose these failures.

This change prints out the source file paths and the colliding relative bundle path in the error message. This should help with diagnosing the failures.

3 years agoFix ILLink descriptors filename (#48430)
Mateo Torres-Ruiz [Thu, 18 Feb 2021 08:59:47 +0000 (00:59 -0800)]
Fix ILLink descriptors filename (#48430)

3 years agoCheck that rsp is within stack limits when redirecting a thread. (#48433)
Vladimir Sadov [Thu, 18 Feb 2021 07:25:31 +0000 (23:25 -0800)]
Check that rsp is within stack limits when redirecting a thread. (#48433)

* Check that rsp is within stack limits when redirecting a thread.

* move the check after/if SetThreadContext has failed

* just return  FALSE after failed SetThreadContext

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
3 years agoClarify the behavior of FilePatternMatch Path and Stem (#48093)
Jeff Handley [Thu, 18 Feb 2021 07:19:21 +0000 (23:19 -0800)]
Clarify the behavior of FilePatternMatch Path and Stem (#48093)

* Add theories that illustrate globbing Path/Stem behavior

* Update Path/Stem docs for FilePatternMatch

* Apply suggestions from code review

Co-authored-by: Carlos Sanchez <1175054+carlossanlop@users.noreply.github.com>
Co-authored-by: Carlos Sanchez <1175054+carlossanlop@users.noreply.github.com>
3 years agoAdd option to ignore reference cycles on serialization (#46101)
David Cantú [Thu, 18 Feb 2021 04:04:19 +0000 (20:04 -0800)]
Add option to ignore reference cycles on serialization (#46101)

* Add option to ignore reference cycles on serialization

* Fix whitespace

* Combine IsValueType check on WriteCoreAsObject

* Add missing null-forgiving operator

* Fix perf regression by using ReferenceHandlerStrategy field in JsonSerializerOptions

* Address suggestions

* Rename API to IgnoreCycles

* Add missing letter to test class name

* Fix CI issue with nullable annotation

3 years agoSetting Thread.Name sets the native thread name on MacOS. (#47465)
Eric Soros [Thu, 18 Feb 2021 03:24:18 +0000 (22:24 -0500)]
Setting Thread.Name sets the native thread name on MacOS. (#47465)

* Setting Thread.Name sets the native thread name on MacOS.

Setting Thread.Name will now set the name of the native thread
on MacOS - due to API constraints this will not work cross thread.

Fix #47087

* adding tests and formatting fixes

3 years agoAdjust logic in cor.h, corhdr.h, corinfo.h, corjit.h and such so that they may be...
David Wrighton [Thu, 18 Feb 2021 03:03:19 +0000 (19:03 -0800)]
Adjust logic in cor.h, corhdr.h, corinfo.h, corjit.h and such so that they may be included without the PAL (#46055)

This was done via simple substitution of the standard sized integer types, and dealing with the fallout. It is tested by using the ICorJitInfo interface directly within the crossgen2 jitinterface thunk library.

Effort was made to use a minimum number of casts, as casts in such a large change are likely to be wrong somewhere.
- Exceptions are the use of casting to handle some calls to the existing internal metadata api and around use of char16_t.

In addition, a small amount of logic from the PAL headers were pulled into the proper header to allow compilation in the presence of some SAL annotation, and other small details.

3 years agoAdd debug checking of the loop table (#48060)
SingleAccretion [Thu, 18 Feb 2021 02:45:35 +0000 (05:45 +0300)]
Add debug checking of the loop table (#48060)

* Add fgDebugCheckLoopTable()

* Minify some logging

* Add checking of the loop table to some phases

* Ignore removed loops in fgDebugCheckLoopTable

3 years agoRemove libraries test build from AndroidCrypto staging job (#48422)
Elinor Fung [Thu, 18 Feb 2021 00:39:43 +0000 (16:39 -0800)]
Remove libraries test build from AndroidCrypto staging job (#48422)

3 years agoUpdate SuperPMI license headers (#48417)
Bruce Forstall [Thu, 18 Feb 2021 00:38:26 +0000 (14:38 -1000)]
Update SuperPMI license headers (#48417)

3 years agoFix parsing of trailing zeros: TryParseNumber (#47666)
Prashanth Govindarajan [Thu, 18 Feb 2021 00:36:25 +0000 (16:36 -0800)]
Fix parsing of trailing zeros: TryParseNumber (#47666)

* Fix parsing of numbers to detect trailing zeros

* sq

* Typo

* Fix a bug

Calculate number of trailing zeros only when number.Digits is updated

* Test case should be culture agnostic

* Address feedback

* More unit tests

* Bug fix and unit tests

* sq

* Address feedback.

* Get more debug info to track down build break

* Turn off consistency check at 1 spot

* Skip consistency check for decimal

* Turn off trailing zero check

3 years agoFix various test crashes and asserts when running the Algorithms test suite with...
Jeremy Koritzinsky [Wed, 17 Feb 2021 22:58:12 +0000 (14:58 -0800)]
Fix various test crashes and asserts when running the Algorithms test suite with the Android Crypto PAL (#48370)

* Fix various test crashes and asserts when running the Algorithms test suite.

* Replace the key/iv if it is initialized multiple times.

3 years ago[RyuJIT] Track exact class for special intrinsics (EqulityComparer, Comparer) (#48279)
Egor Bogatov [Wed, 17 Feb 2021 21:17:20 +0000 (00:17 +0300)]
[RyuJIT] Track exact class for special intrinsics (EqulityComparer, Comparer) (#48279)

3 years agoAdd Eager Options Validation: ValidateOnStart API (#47821)
Maryam Ariyan [Wed, 17 Feb 2021 20:46:42 +0000 (12:46 -0800)]
Add Eager Options Validation: ValidateOnStart API (#47821)