From: Sage Weil <sage@newdream.net>
Date: Wed, 10 Feb 2010 00:43:11 +0000 (-0800)
Subject: ceph: remove bogus invalidate_mapping_pages
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=8031049147c58d9d8b6226c3ac31a9d72d053e25;p=deliverable%2Flinux.git

ceph: remove bogus invalidate_mapping_pages

We were invalidating mapping pages when dropping FILE_CACHE in
__send_cap().  But ceph_check_caps attempts to invalidate already, and
also checks for success, so we should never get to this point.

Signed-off-by: Sage Weil <sage@newdream.net>
---

diff --git a/fs/ceph/caps.c b/fs/ceph/caps.c
index 20b28dc0c97c..ab9b571dda11 100644
--- a/fs/ceph/caps.c
+++ b/fs/ceph/caps.c
@@ -1137,12 +1137,6 @@ static int __send_cap(struct ceph_mds_client *mdsc, struct ceph_cap *cap,
 
 	spin_unlock(&inode->i_lock);
 
-	if (dropping & CEPH_CAP_FILE_CACHE) {
-		/* invalidate what we can */
-		dout("invalidating pages on %p\n", inode);
-		invalidate_mapping_pages(&inode->i_data, 0, -1);
-	}
-
 	ret = send_cap_msg(session, ceph_vino(inode).ino, cap_id,
 		op, keep, want, flushing, seq, flush_tid, issue_seq, mseq,
 		size, max_size, &mtime, &atime, time_warp_seq,