diff options
author | Sage Weil <sage@newdream.net> | 2010-04-13 14:41:22 -0400 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2010-04-13 15:28:31 -0400 |
commit | fc837c8f0446b73a1661339db406c0238dd1d184 (patch) | |
tree | 5cc1e6b97a8f39f7979eb302ac8a090c812f2609 /fs/ceph/super.h | |
parent | f5b066287c74b624583b993395a65d03a6487b3a (diff) |
ceph: queue_cap_snap should always queue dirty context
This simplifies the calling convention, and fixes a bug where we queue a
capsnap with a context other than i_head_snapc (the one that matches the
dirty pages). The result was a BUG at fs/ceph/caps.c:2178 on writeback
completion when a capsnap matching the writeback snapc could not be found.
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/ceph/super.h')
-rw-r--r-- | fs/ceph/super.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/ceph/super.h b/fs/ceph/super.h index 65d12036b670..4c07acaf21e3 100644 --- a/fs/ceph/super.h +++ b/fs/ceph/super.h | |||
@@ -714,8 +714,7 @@ extern int ceph_update_snap_trace(struct ceph_mds_client *m, | |||
714 | extern void ceph_handle_snap(struct ceph_mds_client *mdsc, | 714 | extern void ceph_handle_snap(struct ceph_mds_client *mdsc, |
715 | struct ceph_mds_session *session, | 715 | struct ceph_mds_session *session, |
716 | struct ceph_msg *msg); | 716 | struct ceph_msg *msg); |
717 | extern void ceph_queue_cap_snap(struct ceph_inode_info *ci, | 717 | extern void ceph_queue_cap_snap(struct ceph_inode_info *ci); |
718 | struct ceph_snap_context *snapc); | ||
719 | extern int __ceph_finish_cap_snap(struct ceph_inode_info *ci, | 718 | extern int __ceph_finish_cap_snap(struct ceph_inode_info *ci, |
720 | struct ceph_cap_snap *capsnap); | 719 | struct ceph_cap_snap *capsnap); |
721 | extern void ceph_cleanup_empty_realms(struct ceph_mds_client *mdsc); | 720 | extern void ceph_cleanup_empty_realms(struct ceph_mds_client *mdsc); |