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

kraken: mds: failed filelock.can_read(-1) assertion in Server::_dir_is_nonempty #13555

Merged
merged 1 commit into from Apr 12, 2017

Conversation

smithfarm
Copy link
Contributor

when filelock is in XLOCKDONE state. client of xlocker can rdlock
the filelock. In that case, only client of xlocker can read the lock.

Fixes: http://tracker.ceph.com/issues/18578
Signed-off-by: Yan, Zheng <zyan@redhat.com>
(cherry picked from commit fe4ab52)
@smithfarm smithfarm self-assigned this Feb 21, 2017
@smithfarm smithfarm added this to the kraken milestone Feb 21, 2017
@smithfarm smithfarm added bug-fix core cephfs Ceph File System and removed core labels Feb 21, 2017
@smithfarm smithfarm removed their assignment Feb 21, 2017
@smithfarm smithfarm changed the title kraken: failed filelock.can_read(-1) assertion in Server::_dir_is_nonempty kraken: mds: failed filelock.can_read(-1) assertion in Server::_dir_is_nonempty Feb 21, 2017
@smithfarm
Copy link
Contributor Author

@jcsp This passed an fs suite at http://tracker.ceph.com/issues/19009#note-7 so I guess it's OK to merge?

@smithfarm smithfarm requested a review from jcsp April 10, 2017 21:01
@jcsp
Copy link
Contributor

jcsp commented Apr 10, 2017

👍 (if @ukernel has already reviewed something you can merge it)

@smithfarm smithfarm merged commit 5abdffe into ceph:kraken Apr 12, 2017
@smithfarm smithfarm deleted the wip-18707-kraken branch April 12, 2017 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix cephfs Ceph File System
Projects
None yet
3 participants