Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Merge nmirror to master #3622

Merged
merged 8 commits into from May 15, 2017
Merged

Merge nmirror to master #3622

merged 8 commits into from May 15, 2017

Conversation

jkotas
Copy link
Member

@jkotas jkotas commented May 15, 2017

No description provided.

AntonLapounov and others added 8 commits May 12, 2017 10:25
…eads methods of ThreadPool class.

At present worker threads and completion port threads share the same Windows default thread pool, so we cannot properly implement these methods.

CR: JKotas, SergeyK

[tfs-changeset: 1658202]
[tfs-changeset: 1658215]
     ENABLE_MIN_WINRT doesn't provide enough support for dynamic interop

[tfs-changeset: 1658228]
Hopefully, this will fix the other Etw-related test failures
I've seen among various projects. ETW propagation to EventListeners
was dying internally with a NullReferenceException on
EventMetadata.Parameters.

EventMetadata.Parameters and EventMetadata.ParameterTypes
represent the same data with EventMetadata.Parameters always
being null on the toolchain-based path and a comment attached to
it saying "Would you hurry up and die already?"

The EventSource rules of the code seems to be to check
before using Parameters and fallback to using ParameterTypes
if necessary but it was forgetting that in a few places.
@jkotas jkotas merged commit b0398c9 into master May 15, 2017
dotnet-bot pushed a commit to dotnet/coreclr that referenced this pull request May 16, 2017
Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas added a commit to dotnet/coreclr that referenced this pull request May 16, 2017
Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants