Skip to content

Assemblymanager thread safety #277

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
filmor merged 14 commits into from
Nov 17, 2016
Merged

Assemblymanager thread safety #277

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
89a5504
Use ConcurrentDictionary for thread safety of import clr
Oct 26, 2016
cd1b9b9
Add thread safety test, use ConcurrentDictionary, lock for assemblies
Oct 27, 2016
596c210
Use atomic TryAdd call here as well
Oct 27, 2016
b9e532d
Print System.__file__ if assertion fails
Oct 27, 2016
2742b12
Wrap assemblies in thread-safe class, make sure forked thread joins i…
Oct 27, 2016
6838be8
No need to support recursion, also lock when we get _list.Count
Oct 28, 2016
59b9ea1
Import some .NET types as well and use them
Oct 28, 2016
2f416c4
Part of the System namespace is implemented in mscorlib.dll
Oct 28, 2016
3844d19
Missed an asterisk
Oct 28, 2016
c8ddb63
Merge branch 'master' into assemblymanager_thread_safety
Oct 30, 2016
bce86dc
Grab lock *before* getting enumerator
Nov 4, 2016
f3f7151
Merge branch 'assemblymanager_thread_safety' of https://github.com/Ar…
Nov 4, 2016
abb5eff
Merge branch 'master' into assemblymanager_thread_safety
ArvidJB Nov 14, 2016
cee2eb6
Merge branch 'master' into assemblymanager_thread_safety
filmor Nov 17, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
No need to support recursion, also lock when we get _list.Count
  • Loading branch information
abessen committed Oct 28, 2016
commit 6838be81c7beebed564c02f277e6ec33cbae697c
16 changes: 14 additions & 2 deletions src/runtime/assemblymanager.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -488,10 +488,22 @@ private class AssemblyList : IEnumerable<Assembly>{

public AssemblyList(int capacity) {
_list = new List<Assembly>(capacity);
_lock = new ReaderWriterLockSlim(LockRecursionPolicy.SupportsRecursion);
_lock = new ReaderWriterLockSlim();
}

public int Count { get { return _list.Count; } }
public int Count
{
get
{
_lock.EnterReadLock();
try {
return _list.Count;
}
finally {
_lock.ExitReadLock();
}
}
}

public void Add(Assembly assembly) {
_lock.EnterWriteLock();
Expand Down