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

Thread deadlock in 9.83 #417

Closed
vgough opened this issue Nov 5, 2016 · 4 comments
Closed

Thread deadlock in 9.83 #417

vgough opened this issue Nov 5, 2016 · 4 comments
Labels

Comments

@vgough
Copy link

vgough commented Nov 5, 2016

Encfs recently changed to use easylogging++, and a thread deadlock has been reported in the logging library: vgough/encfs#244

The referenced issue has the relevant stack traces from ThreadSanitizer. Can you confirm that this is an issue w/ easylogging++?

WARNING: ThreadSanitizer: lock-order-inversion (potential deadlock) (pid=3933)
Cycle in lock order graph: M11 (0x7d480000be90) => M13 (0x7d4400009d90) => M17 (0x7d580000fc10) => M11

@abumq
Copy link
Owner

abumq commented Feb 1, 2017

This is possibly fixed in v9.91.

See dd12e89

Please re-open if it is not with updated ThreadSanitizer's output. Please update to latest version.

@abumq
Copy link
Owner

abumq commented Feb 1, 2017

Re-opening as i found an issue with this in the fix

@abumq abumq reopened this Feb 1, 2017
@abumq abumq added the bug label Feb 1, 2017
@abumq
Copy link
Owner

abumq commented Feb 3, 2017

@aparajita do you have time to look in to this? NOTE: Please use develop branch and use pthread.cpp sample for re-producing point (unless you want to write your own)

@aparajita
Copy link
Contributor

@mkhan3189 Without looking deeply at the code, I wonder if you might be a recursive lock on the mutex within the same thread, in which case you need to use a recursive mutex.

@abumq abumq closed this as completed in fddfa13 Feb 4, 2017
abumq added a commit that referenced this issue Feb 9, 2017
abumq added a commit that referenced this issue Feb 9, 2017
rschlaikjer added a commit to rschlaikjer/librealsense that referenced this issue Nov 5, 2018
This fixes a thread deadlock that could occur in some situations,
especially when running under Valgrind.

See this issue on the easyloggingpp repo:
abumq/easyloggingpp#417
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants