CAS Security cleanup (#9355)

Merge pull request #9367 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-25006-02 (master)

Update CoreClr to beta-25006-02

Delete dead code from ExecutionContext (#9359)

Merge pull request #9358 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-25006-01 (master)

[x86/Linux] Add Assertion on pvControlPc (#9356)

Update CoreClr to beta-25006-01

Update CoreClr, CoreFx to beta-25005-03, beta-25004-01, respectively (#9350)

Port 121d095ed0b0076fb1c7ff59e6446fd19d506b32 from Windows to Linux (#9346)

Port #5821 to Linux (#9347)

Port #5339 to Linux (#9348)

Merge pull request #9342 from stephentoub/tasks_inlining

Remove NoInlining/StackCrawlMarks from Tasks

Address PR feedback and other cleanup

Addressed PR comment about removing ExecutionContext.Capture(ref StackCrawlMark, ...), which led me to remove NoInlining/StackCrawlMarks from some other places in Timer, Thread, ThreadPool, and Overlapped.

In doing so, I found several other unnecessary members on ExecutionContext that could be removed with minor tweaks elsewhere in the source, e.g.
- Dispose is a nop, so we can remove explicit try/finally's to clean it up
- ExecutionContext.Run(..., bool) just ignores the bool arg, so it can be removed and all call sites redirected (but I've left it in EC, as it appears it's used via internals visible from a library in corefx)
- FastCapture() just calls Capture, so the call sites can be changed (but I've left it in EC for a similar reason)
- PreAllocatedDefault can be removed in favor of Default; etc.
- ExecutionContext.Capture itself checks whether flow is suppressed; doing a check before it adds a TLS lookup and in doing so optimizes for an uncommon case in exchange for making the common case more expensive.  I've removed those checks.

And in the process, I also noticed that several lazily initialized delegates that no longer need to be, and cleaned those up.  These were lazy due to needing to be for security checks that aren't relevant in coreclr.

Remove NoInlining/StackCrawlMarks from Tasks

These crawl marks in Tasks aren't relevant in coreclr (they're passed down to ExecutionContext, which in coreclr just ignores them), yet they required NoInlining, which does impact coreclr.  Removing them all from tasks.

Merge pull request #9343 from stephentoub/tasks_selfreplicating

Remove internal self-replicating support from Tasks

Remove internal self-replicating support from Tasks

In the initial TPL prototype, Tasks had a feature that allowed them to "replicate", meaning to schedule a copy of themselves.  Prior to shipping the initial version, this feature was made internal, as it was deemed unnecessary, but by then the Parallel loops constructs had already been built on top of them.  Now that Parallel no longer depends on this feature (and isn't in CoreLib), we can entirely rip it out.

Rename internal CoreLib Console to LowLevelConsole (#9341)

Merge pull request #9338 from jkotas/against-packages

Add mscorlib.dll back to Microsoft.TargetingPack.Private.CoreCLR

Use StringBuilderCache for Enum.InternalFlagsFormat (#9340)

Remove stripped code (#9269)

Delete essentially all the code that the BCL rewriter is removing.
Added back some code it shouldn't have been removing, eg., serialization code, debugger visualizers, and some changing of 'protected' to 'protected internal'

Merge pull request #9240 from BruceForstall/FixSimdFieldMorphingDeadCode

Remove SIMD dead code

Add mscorlib.dll back to Microsoft.TargetingPack.Private.CoreCLR

Removing it broke BuildFromPackages flavor of test build

Revert "Update CoreClr, CoreFx to beta-25004-01, beta-25004-01, respectively (#9330)"

This reverts commit e8dff5550c022d2854914dc534f9ec3d7fe623ac.

Merge pull request #9324 from gkhanna79/Fix9030

Use fallback load context as binding context for dynamic parent assembly

Remove duplicate attributes from CoreLib (#9327)

Update CoreClr, CoreFx to beta-25004-01, beta-25004-01, respectively (#9330)

Remove cross appdomains cultures checks (#9328)

* Remove cross appdomains cultures checks

In the ful framework, CultureInfo and other globalization objects are marked as agile which means it can cross the appdomains. In coreclr we don't have multiple appdomains anymore so these checks are not really needed.
removing these checks will make coreclr consistent when running on Windows and other OS's

* Remove un-needed resource entry

Fix legacy Globalization tests to work on Linux (#9322)

* Fix legacy Globalization tests to work on Linux

* Delete globalization tests

Use fallback load context as binding context for dynamic parent assembly

Delete mscorlib from packages (#9313)

Switch netcoreapp test reference to netcoreapp2.0 (#9310)

Update glossary.md

Merge pull request #9315 from wtgodbe/disableBenchmarkGames

Disable failing Helix tests until we fix underlying issues

Disable failing Helix tests until we fix underlying issues

Update glossary.md

Merge pull request #9294 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-25003-02, beta-25003-02, respectively (master)

[x86/Linux] Clean Up #ifdef for VerifyValidTransitionFromManagedCode (#9304)

[x86/Linux] Clean up non-WIN64EXCEPTIONS code: (#9305)

[x86/Linux] Clean Up JIT_EndCatch (#9302)

Merge pull request #9256 from CarolEidt/TreeNodeInfoSplit

Refactor TreeNodeInfoInit

Merge pull request #9279 from stephentoub/more_tp_perf

Some more perf tweaks to ThreadPool

Update CoreClr, CoreFx to beta-25003-02, beta-25003-02, respectively

Reduce cost of LocalPop when local queue empty

- Change LocalPop to be parameterless and return the IThreadPoolWorkItem rather than returning a bool and having the IThreadPoolWorkItem in an out arg.
- Refactor LocalPop slightly to have an inlineable upfront check for whether the local queue is empty, and only call the non-inlineable method if there's something that could potentially be popped

Allow Dequeue to avoid TrySteal if there's nothing to steal

Address PR feedback

Enable inlining of EnsureVMInitialized

Replace usage of Random

We need some randomness, but it doesn't need to be particularly good, just fast.  A simple xorshift prng is 3-4x faster than Random, in particular without multiple virtual method calls that don't get inlined.  It's also a bit lighter in terms of memory (though there's only one of these per thread, so it doesn't impact much).

Remove StackCrawlMark goo from {Unsafe}QueueUserWorkItem

Allow more methods to be inlined

Remove unnecessary try/finally blocks

And clean up the code around it.

Rework work-stealing queue list

The current data structure used to store the list of local work-stealing queues is maintains a sparse array, where entries can be null, and where removals null out entries in an active array that could be being read by another thread concurrently.  This necessitates that threads looking for work need to use volatile reads and null checks on each element.  Further, because the array doubles in size when it grows, and never shrinks, we often end up having many empty slots that threads need to look at as they're looking for work.

It's actually relatively rare for threads to come and go.  While the thread pool does take threads in and out of service, it only rarely actually terminates a thread or asks the OS for a new one, so it's relatively rare that threads are added/removed from the list.  Given that, we can simply use immutable arrays, creating a new array of the exact right size whenever a thread is added or removed.  Then iteration can be done without volatile reads and without null checks, because the contents of the array being read through won't change and won't ever be null.

Reorder usings per previous PR feedback

[x86/Linux] Do NOT use QuickUnwindStackFrame (#9299)

* [x86/Linux] Do NOT use QuickUnwindStackFrame

* Fix x64 build error

[x86/Linux] Convert Fake DBZ into OVF (#9295)

Merge pull request #9283 from adiaaida/devdiv377155

Add a test for #9282

Clean up GetAddrOfSecurityObjectFromCachedInfo (#9291)

GetAddrOfSecurityObjectFromCachedInfo seems to be X86-specific, but has
ifdef for other architectures.

This commit declares GetAddrOfSecurityObjectFromCachedInfo only for X86
to simplify its implementation.

[x86/Linux] Do NOT use GSCookie on Funclet (#9290)

Refactor TreeNodeInfoInit

Move all the register requirements (TreeNodeInfoInit) code into lsra{arch}.cpp.
Split methods that currently do both tree transformation and register requirements.

Merge pull request #9280 from wtgodbe/disableLargeMemoryTests

Disable GC LargeMemory tests that were failing in Helix

Add a test for #9282

Merge pull request #9229 from sandreenko/increase_gtCallMoreFlags_size

increase gtCallMoreFlags size

Disable GC LargeMemory tests that were failing in Helix

Merge pull request #9258 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-25002-03 (master)

Merge pull request #9275 from RussKeldorph/results_moved

Adapt to corefx moving test results.

Make ValueTuple types serializable (#9271)

Update CoreClr to beta-25002-03

Adapt to corefx moving test results.

[x86/Linux] Set Establisher Frame Pointer as Caller SP (#9264)

Delete remaining legacy ComInterfaces (#9270)

Add ThreadPoolBoundHandle to model.xml (#9268)

This type was moved from corefx. This will allow the redundant copy in corefx to be deleted.

[x86/Linux] Do NOT use pXXX fields (#9121)

* [x86/Linux] Do NOT use pXXX fields

* Fix x86/Windows build error

* Fix another x86/Windows build error

* Fix typo

* Do NOT trash EBP

* Reflect the original semantics of EHContext::UpdateFrame

* Unify ReadXXX/LocateXXX into GetXXXLocation, and RestoreXXX/TrashXXX into SetXXXLocation

* Revert the order of pXXX fields

* Revise cgenx86.cpp

* Revert unnecessary changes

* Remove direct accesses to CALLEE_SAVED_REGISTERS_TAG

* Do NOT update pCurrentContext inside SetXXXLocation

* Update RegPtr via offset

* Unify REG_METHODS (and revise UpdateRegDisplay methods accordingly)

* Revise per feedback

* Fix x86/Windows build error

Delete LocalDataStore (#9266)

This lives in CoreFX now

[x86/Linux] Process Explicit Frame before Managed Frame (#9261)

This commit revises stack walker to process explicit frames before
managed frame as in other architectures that use Win64 EH model.

This commit fixes #9260.

Synchronize part of System/* with CoreRT (#9259)

Merge pull request #9234 from stephentoub/tp_perf

Some ThreadPool performance and maintainability improvements

Adding some missing APIs on core that where added to N.S 2.0 (#9203)

Adding missing apis on .NET Core.

Fix segment map modification in OOM scenarios (#9241)

* Fix an issue where the segment mapping table is not reverted
back to its original state when failing to commit the mark array
in an OOM scenario. Fixes #9064

* Code review feedback

* Update seg_mapping_table before setting g_gc_*_address

* Add a barrier between the seg_mapping_table write and updates to g_gc_*_address

Merge pull request #9254 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-25001-07 (master)

Update CoreClr to beta-25001-07

Address PR feedback

- Fix ArgumentNullException parameter name

Merge pull request #9252 from wtgodbe/fixRuntimes

Use full set of runtimes for all test project.json files

Use full set of runtimes for all test project.json files

Synchronize src\mscorlib\corefx with CoreRT (#9251)

Miscellaneous code cleanup

- Remove some explicit ctors
- Remove some unnecessary casts
- Mark some fields readonly
- Follow style guidelines for visibility/static ordering in signatures
- Move usings to top of file
- Delete some stale comments
- Added names for bool args at call sites
- Use expression-bodied members in a few places
- Pass lower bounds to Array.Copy
- Remove unnecessary "success" local in QueueUserworkItemHelper

Merge pull request #9242 from dotnet-bot/master-UpdateDependencies

Update CoreFx to beta-25001-02 (master)

Simplify Dequeue method

- Consolidate branches
- Pass resulting work item as a return value rather than an out
- Avoid an unnecessary write to the bool passed by-ref

Remove unnecessary explicit static cctors

Let these types be marked beforefieldinit

Simplify TrySteal

- Use the newly updated SpinLock.TryEnter(ref bool) instead of TryEnter(int, ref bool), as the former is ~3x faster when the lock is already held.
- Change the method to avoid the out reference argument and instead just return it
- Simplify the logic to consolidate the setting of missedSteal

Update CoreFx to beta-25001-02

Use ConcurrentQueue<T> in ThreadPool

Use the same ConcurrentQueue<T> now being used in corefx in ThreadPool instead of ThreadPool's custom queue implementation (which was close in design to the old ConcurrentQueue<T> implementation).

Port ConcurrentQueue<T> back from corefx

It's not exposed from CoreLib, but a) as long as the code is here it's good to have it in sync, and b) we can use it in ThreadPool rather than having a separate concurrent queue implementation (which is very similar to the old ConcurrentQueue design).

Fix GC reporting for Span<T> passed in registers on Unix (#9247)

Part of fix for #9033

Test coverage is already there in CoreFX's span tests.

Synchronize SpinLock with CoreRT (#9244)

Merge pull request #9211 from hqueue/arm/clang_O3_issue

Use -O1 optimization for arm and armel

ReflectionOnlyLoad should throw NotSupportedException (#9238)

Renaming Span/ReadOnlySpan Extension classes to avoid conflict with CoreFX. (#9182)

* Moving all extension methods to a class called Span
* Adding known workaround for JIT bug.

[x86/Linux] Fix LLDB Plugin Build Error (#9239)

increase gtCallMoreFlags size

gtCallMoreFlags was shart and all 16 values are already used. We need
one more for CoreRT fat pointers and it is likely that we wll need more
in the future.

Merge pull request #9233 from adiaaida/disableDeleteNextCardTable

Disable delete_next_card_table

Remove SIMD dead code

Fixes #9089

[x86/Linux] Get Frame Pointer from CallerSp (#9235)

GetGSCookieAddress uses pEbp to get the current frame pointer, but pEbp
is not properly initialized as discussed in #8980.

This commit revises GetGSCookieAddress to use CallerSp (as in other
architectures) to get Frame Pointer in order to fix #8980.