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: cephfs: segfault in handle_client_caps #14566

Merged
merged 1 commit into from Apr 20, 2017

Conversation

smithfarm
Copy link
Contributor

Locker::handle_client_caps delays processing cap message if the
corresponding inode is freezing or frozen. When the message gets
processed, client can have already closed the session.

Fixes: http://tracker.ceph.com/issues/18306
Signed-off-by: Yan, Zheng <zyan@redhat.com>
(cherry picked from commit e281a0b)
@smithfarm smithfarm self-assigned this Apr 14, 2017
@smithfarm smithfarm added this to the kraken milestone Apr 14, 2017
@smithfarm smithfarm changed the title kraken: segfault in handle_client_caps kraken: cephfs: segfault in handle_client_caps Apr 14, 2017
@smithfarm smithfarm added cephfs Ceph File System and removed core labels Apr 14, 2017
@smithfarm smithfarm requested a review from jcsp April 20, 2017 08:32
@smithfarm
Copy link
Contributor Author

@jcsp This passed a rados suite at http://tracker.ceph.com/issues/19009#note-22 and an fs suite at http://tracker.ceph.com/issues/19009#note-27

There are some failures in the upgrade/jewel-x and upgrade/client-upgrade suites, but the only unexplained one is http://tracker.ceph.com/issues/19637

Please review/approve if you agree its OK to merge under those conditions?

@smithfarm smithfarm merged commit f1affa9 into ceph:kraken Apr 20, 2017
@smithfarm smithfarm deleted the wip-18616-kraken branch April 20, 2017 16:58
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