Skip to content

Commit 9a9927c

Browse files
authored
Merge pull request #356 from github/jsinglet/release-mutex
Fixes a locking order issue in the Matrix testing
2 parents e47e0ae + f905b78 commit 9a9927c

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

scripts/matrix_testing/CreateMatrixTestReport.ps1

+5-3
Original file line numberDiff line numberDiff line change
@@ -417,16 +417,18 @@ $jobRows = $queriesToCheck | ForEach-Object -ThrottleLimit $NumThreads -Parallel
417417
# output current row state
418418
$row
419419

420-
# release any held mutexes
421-
$Mutex.ReleaseMutex();
422-
420+
423421
###########################################################
424422
###########################################################
425423
# Context is restored here
426424
###########################################################
427425
if($context){
428426
Pop-CompilerSpecificFiles -Context $context
429427
}
428+
429+
# release any held mutexes
430+
$Mutex.ReleaseMutex();
431+
Write-Host "Released $testDirectory"
430432
}
431433
}
432434
# go to next row

0 commit comments

Comments
 (0)