ceph: remove ceph_lookup_inode()
authorYan, Zheng <zheng.z.yan@intel.com>
Mon, 2 Sep 2013 07:19:53 +0000 (15:19 +0800)
committerSage Weil <sage@inktank.com>
Fri, 6 Sep 2013 19:55:09 +0000 (12:55 -0700)
commit 6f60f889 (ceph: fix freeing inode vs removing session caps race)
introduced ceph_lookup_inode(). But there is already a ceph_find_inode()
which provides similar function. So remove ceph_lookup_inode(), use
ceph_find_inode() instead.

Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>
Reviewed-by: Alex Elder <alex.elder@linary.org>
Reviewed-by: Sage Weil <sage@inktank.com>
fs/ceph/inode.c
fs/ceph/mds_client.c
fs/ceph/super.h

index eae41cd..8549a48 100644 (file)
@@ -62,14 +62,6 @@ struct inode *ceph_get_inode(struct super_block *sb, struct ceph_vino vino)
        return inode;
 }
 
-struct inode *ceph_lookup_inode(struct super_block *sb, struct ceph_vino vino)
-{
-       struct inode *inode;
-       ino_t t = ceph_vino_to_ino(vino);
-       inode = ilookup5_nowait(sb, t, ceph_ino_compare, &vino);
-       return inode;
-}
-
 /*
  * get/constuct snapdir inode for a given directory
  */
index 603786b..b7bda5d 100644 (file)
@@ -1054,7 +1054,7 @@ static void remove_session_caps(struct ceph_mds_session *session)
                        vino = cap->ci->i_vino;
                        spin_unlock(&session->s_cap_lock);
 
-                       inode = ceph_lookup_inode(sb, vino);
+                       inode = ceph_find_inode(sb, vino);
                        iput(inode);
 
                        spin_lock(&session->s_cap_lock);
index bb23ef6..6014b0a 100644 (file)
@@ -694,8 +694,6 @@ extern void ceph_destroy_inode(struct inode *inode);
 
 extern struct inode *ceph_get_inode(struct super_block *sb,
                                    struct ceph_vino vino);
-extern struct inode *ceph_lookup_inode(struct super_block *sb,
-                                      struct ceph_vino vino);
 extern struct inode *ceph_get_snapdir(struct inode *parent);
 extern int ceph_fill_file_size(struct inode *inode, int issued,
                               u32 truncate_seq, u64 truncate_size, u64 size);