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

nested signal handling #478

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

nested signal handling #478

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

Comments

@ramosian-glider
Copy link
Member

Originally reported on Google Code with ID 71

The reproducer is supplied offline. Nested signals are not handled in the reproducer.

Reported by dvyukov@google.com on 2014-08-06 07:37:22


- _Attachment: [nested_signals.cpp](https://storage.googleapis.com/google-code-attachments/thread-sanitizer/issue-71/comment-0/nested_signals.cpp)_
@ramosian-glider
Copy link
Member Author

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

Reported by igor@ikseek.com on 2014-08-09 16:07:42

@ramosian-glider
Copy link
Member Author

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.


Reported by dvyukov@google.com on 2014-09-02 12:39:26

@ramosian-glider
Copy link
Member Author

Awesome, thanks!

Reported by igor@unity3d.com on 2014-09-03 09:51:26

@ramosian-glider
Copy link
Member Author

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.

Reported by dvyukov@google.com on 2014-09-03 12:36:36

  • Status changed: Fixed

@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:32

  • Labels added: ProjectThreadSanitizer

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

1 participant