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

mds: don't modify inode that is not projected #13052

Merged
merged 1 commit into from Jan 27, 2017

Conversation

Projects
None yet
3 participants
@ukernel
Member

ukernel commented Jan 22, 2017

In the slave rename prep case (rename inode to different auth mds), the
rename inode is not projected. CDir::check_rstat() gets confused if
MDCache::_project_rstat_inode_to_frag() updates inode's accounted rstat
in that case.

Fixes: http://tracker.ceph.com/issues/16768
Signed-off-by: "Yan, Zheng" zyan@redhat.com

mds: don't modify inode that is not projected
In the slave rename prep case (rename inode to different auth mds), the
rename inode is not projected. CDir::check_rstat() gets confused if
MDCache::_project_rstat_inode_to_frag() updates inode's accounted rstat
in that case.

Fixes: http://tracker.ceph.com/issues/16768
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
@batrick

This comment has been minimized.

Member

batrick commented Jan 24, 2017

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

@jcsp jcsp merged commit b54707d into ceph:master Jan 27, 2017

2 of 3 checks passed

default Build finished.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details

@ukernel ukernel deleted the ukernel:wip-16768 branch Feb 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment