theRockLiu / thread-sanitizer

Automatically exported from code.google.com/p/thread-sanitizer
0 stars 0 forks source link

nested signal handling #71

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The reproducer is supplied offline. Nested signals are not handled in the 
reproducer.

Original issue reported on code.google.com by dvyu...@google.com on 6 Aug 2014 at 7:37

Attachments:

GoogleCodeExporter commented 9 years ago
Also please note magic sleeps placed in both threads! Without them signals sent 
from main thread are lost.

Original comment by i...@ikseek.com on 9 Aug 2014 at 4:07

GoogleCodeExporter commented 9 years ago
Commit 216903
http://llvm.org/viewvc/llvm-project?view=revision&revision=216903
allows recursive interceptors.

I will also try to eliminate the need in sleep(1) in main.

The sleep in BusyThread still will be required. But it does not need to be a 
sleep, it can be any function intercepted by tsan (e.g. malloc or free), so I 
would expect it to not be a problem.

Original comment by dvyu...@google.com on 2 Sep 2014 at 12:39

GoogleCodeExporter commented 9 years ago
Awesome, thanks!

Original comment by i...@unity3d.com on 3 Sep 2014 at 9:51

GoogleCodeExporter commented 9 years ago
I've submitted
http://llvm.org/viewvc/llvm-project?view=revision&revision=217031
which should eliminate the need in sleep in the main thread.

Igor, I hope that this part of tasn now works for you.
If you have any other issues with tsan, feel free to file bug reports.

Original comment by dvyu...@google.com on 3 Sep 2014 at 12:36