From: Brian Robbins <brianrob@microsoft.com>
Date: Thu, 22 Feb 2018 05:59:17 +0000 (-0800)
Subject: Fix EventPipe Test Failures Under GCStress (#16494)
X-Git-Tag: accepted/tizen/unified/20190422.045933~2915
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9f08404d97b0c1ee0d1bcabb8046e3efb49163c0;p=platform%2Fupstream%2Fcoreclr.git

Fix EventPipe Test Failures Under GCStress (#16494)

* Disable inducedgc test when GCStress is enabled.

* Suppress host contract violation in EventPipe::WalkManagedStackForThread.
---

diff --git a/src/vm/eventpipe.cpp b/src/vm/eventpipe.cpp
index 026a775..e413713 100644
--- a/src/vm/eventpipe.cpp
+++ b/src/vm/eventpipe.cpp
@@ -708,6 +708,10 @@ bool EventPipe::WalkManagedStackForThread(Thread *pThread, StackContents &stackC
     }
     CONTRACTL_END;
 
+    // Calling into StackWalkFrames in preemptive mode violates the host contract,
+    // but this contract is not used on CoreCLR.
+    CONTRACT_VIOLATION( HostViolation );
+
     stackContents.Reset();
 
     StackWalkAction swaRet = pThread->StackWalkFrames(
diff --git a/tests/src/tracing/tracevalidation/inducedgc/inducedgc.csproj b/tests/src/tracing/tracevalidation/inducedgc/inducedgc.csproj
index d4bb1f0..4da9f90 100644
--- a/tests/src/tracing/tracevalidation/inducedgc/inducedgc.csproj
+++ b/tests/src/tracing/tracevalidation/inducedgc/inducedgc.csproj
@@ -12,6 +12,7 @@
     <CLRTestKind>BuildAndRun</CLRTestKind>
     <DefineConstants>$(DefineConstants);STATIC</DefineConstants>
     <CLRTestPriority>0</CLRTestPriority>
+    <GCStressIncompatible>true</GCStressIncompatible>
   </PropertyGroup>
   <!-- Default configurations to help VS understand the configurations -->
   <PropertyGroup Condition="'$(Configuration)|$(Platform)' == 'Debug|x64'"></PropertyGroup>