From 99cde3194cdd23ae5b317b10fd1b4667f62741ae Mon Sep 17 00:00:00 2001 From: Koundinya Veluri Date: Tue, 8 Dec 2015 19:57:47 -0800 Subject: Revert "Disable suspend on shutdown during shutdown finalization" Reverting commit b18d2a7e30a0d7066d2c09711de07488a7dec475 (PR #2207) due to issue dotnet/corefx#4899, while investigating the issue, to unblock the CI. --- src/vm/finalizerthread.cpp | 6 ------ 1 file changed, 6 deletions(-) (limited to 'src') diff --git a/src/vm/finalizerthread.cpp b/src/vm/finalizerthread.cpp index b9895742a0..aca97e3ff2 100644 --- a/src/vm/finalizerthread.cpp +++ b/src/vm/finalizerthread.cpp @@ -1207,11 +1207,6 @@ BOOL FinalizerThread::FinalizerThreadWatchDog() #endif //BACKGROUND_GC _ASSERTE ((g_fEEShutDown & ShutDown_Finalize1) || g_fFastExitProcess); - - // FinalizerThreadWatchDogHelper does not time out waiting for the finalizer thread in CoreCLR. If another thread blocks - // during GC for shutdown, a finalizer that allocates (or even jitting a finalizer, which does allocation) may block the - // finalizer thread indefinitely, which may in turn lead to a deadlock. -#ifndef FEATURE_CORECLR ThreadSuspend::SuspendEE(ThreadSuspend::SUSPEND_FOR_SHUTDOWN); g_fSuspendOnShutdown = TRUE; @@ -1222,7 +1217,6 @@ BOOL FinalizerThread::FinalizerThreadWatchDog() ThreadStore::TrapReturningThreads(TRUE); ThreadSuspend::RestartEE(FALSE, TRUE); -#endif if (g_fFastExitProcess) { -- cgit v1.2.3