diff options
| author | Sage Weil <sage@newdream.net> | 2010-04-13 11:41:22 -0700 |
|---|---|---|
| committer | Sage Weil <sage@newdream.net> | 2010-04-13 12:28:31 -0700 |
| commit | fc837c8f0446b73a1661339db406c0238dd1d184 (patch) | |
| tree | 5cc1e6b97a8f39f7979eb302ac8a090c812f2609 /fs/ceph/super.h | |
| parent | f5b066287c74b624583b993395a65d03a6487b3a (diff) | |
| download | blackbird-op-linux-fc837c8f0446b73a1661339db406c0238dd1d184.tar.gz blackbird-op-linux-fc837c8f0446b73a1661339db406c0238dd1d184.zip | |
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, extern void ceph_handle_snap(struct ceph_mds_client *mdsc, struct ceph_mds_session *session, struct ceph_msg *msg); -extern void ceph_queue_cap_snap(struct ceph_inode_info *ci, - struct ceph_snap_context *snapc); +extern void ceph_queue_cap_snap(struct ceph_inode_info *ci); extern int __ceph_finish_cap_snap(struct ceph_inode_info *ci, struct ceph_cap_snap *capsnap); extern void ceph_cleanup_empty_realms(struct ceph_mds_client *mdsc); |
