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: mds is crushed, after I set about 400 64KB xattr kv pairs to a file #16103

Merged
merged 1 commit into from Jul 19, 2017

Conversation

smithfarm
Copy link
Contributor

After I have set about 400 64KB xattr kv pair to a file,
mds is crashed. Every time I try to start mds, it will crash again.
The root reason is write_buf._len overflowed when doing
Journaler::append_entry().

This patch try to fix this problem through the following changes:

 1. limit file/dir's xattr size
 2. throttle journal entry append operations

Fixes: http://tracker.ceph.com/issues/19033
Signed-off-by: Yang Honggang joseph.yang@xtaotech.com
(cherry picked from commit eb915d0)
@smithfarm smithfarm self-assigned this Jul 4, 2017
@smithfarm smithfarm added this to the kraken milestone Jul 4, 2017
@smithfarm smithfarm added bug-fix core cephfs Ceph File System and removed core labels Jul 4, 2017
@smithfarm smithfarm requested a review from jcsp July 5, 2017 09:49
@smithfarm
Copy link
Contributor Author

@jcsp This PR passed an fs run at http://tracker.ceph.com/issues/19009#note-57 (there were some failures that required a fix - applied via #16114 and passed on rerun). Please review/approve/merge the PR if you agree it can be merged.

@smithfarm smithfarm requested a review from batrick July 18, 2017 22:12
@smithfarm
Copy link
Contributor Author

@batrick This PR passed another fs suite at http://tracker.ceph.com/issues/19009#note-67

Please review.

@smithfarm smithfarm merged commit 055319b into ceph:kraken Jul 19, 2017
@smithfarm smithfarm deleted the wip-19674-kraken branch July 19, 2017 20:56
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