Skip to content
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.

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

Add deadlock detector edges based on compiler annotations #457

Closed
ramosian-glider opened this issue Sep 1, 2015 · 5 comments
Closed

Add deadlock detector edges based on compiler annotations #457

ramosian-glider opened this issue Sep 1, 2015 · 5 comments

Comments

@ramosian-glider
Copy link
Member

Originally reported on Google Code with ID 50

Compiler annotations such as acquired_before and acquired_after under -Wthread-safety
in clang give information on expected lock order, but they aren't enforced in runtime.

We could enforce this in runtime by instrumenting code to add edges to the deadlock
detector (a class constructor for instance would be instrumented to add edges to annotated
member locks when created).

Reported by pbos@google.com on 2014-03-05 15:23:32

@ramosian-glider
Copy link
Member Author

We'll need a matching feature request for clang explaining how the static information
from thread-safety annotations can be transmitted into the dynamic tool at run-time.
+Delesley, in case he has any thoughts already. 

Reported by konstantin.s.serebryany on 2014-03-11 13:33:46

@ramosian-glider
Copy link
Member Author

Reported by dvyukov@google.com on 2014-04-25 10:18:27

  • Labels added: DeadlockDetector

@ramosian-glider
Copy link
Member Author

Adding Project:ThreadSanitizer as part of GitHub migration.

Reported by glider@google.com on 2015-07-30 09:21:31

  • Labels added: ProjectThreadSanitizer

@morehouse
Copy link
Contributor

@kcc: Do we still want to do this eventually?

@kcc
Copy link
Contributor

kcc commented Jun 6, 2018

nope

@kcc kcc closed this as completed Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants