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
os/bluestore/bluefs: Fix false collision with lockdep module #43589
Conversation
thanks @aclamk. i ran the same test with this fix, and now it's failing with |
4b23ecf
to
e292475
Compare
Usually sequence of locking is 1) FileWriter 2) File. In _compact_log_async_LD_NF_D it was in reversed order. No real deadlock was possible, but lockdep complained. Bonus: Improved lock dependency graph. Fixes: https://tracker.ceph.com/issues/52939 Signed-off-by: Adam Kupczyk <akupczyk@redhat.com>
e292475
to
21e5a0f
Compare
@cbodley Thanks for checking this so quickly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
works for me, thanks!
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
@aclamk pls rebase |
Usually sequence of locking is
In _compact_log_async_LD_NF_D it was in reversed order.
No real deadlock was possible, but lockdep complained.
Fixes: https://tracker.ceph.com/issues/52939
Signed-off-by: Adam Kupczyk akupczyk@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox