ceph: hide /.ceph from readdir results
authorSage Weil <sage@newdream.net>
Wed, 2 Dec 2009 19:54:25 +0000 (11:54 -0800)
committerSage Weil <sage@newdream.net>
Thu, 3 Dec 2009 22:59:48 +0000 (14:59 -0800)
We need to skip /.ceph in (cached) readdir results, and exclude "/.ceph"
from the cached ENOENT lookup check.

Signed-off-by: Sage Weil <sage@newdream.net>
fs/ceph/ceph_fs.h
fs/ceph/dir.c

index 4e5f49c738d83b22abdf6b94c32891d18590cf2b..699196a10c6662119d1ce628eadf1ab79833bbdd 100644 (file)
@@ -44,6 +44,7 @@
 
 
 #define CEPH_INO_ROOT  1
+#define CEPH_INO_CEPH  2        /* hidden .ceph dir */
 
 /* arbitrary limit on max # of monitors (cluster of 3 is typical) */
 #define CEPH_MAX_MON   31
index 32ef543672247127186ff4e9a7b34e3ac36ab1fd..89ce3ba4a614104c100bcb9eb6e343c5bf15ad30 100644 (file)
@@ -132,6 +132,7 @@ more:
                }
                if (!d_unhashed(dentry) && dentry->d_inode &&
                    ceph_snap(dentry->d_inode) != CEPH_SNAPDIR &&
+                   ceph_ino(dentry->d_inode) != CEPH_INO_CEPH &&
                    filp->f_pos <= di->offset)
                        break;
                dout(" skipping %p %.*s at %llu (%llu)%s%s\n", dentry,
@@ -512,6 +513,12 @@ struct dentry *ceph_finish_lookup(struct ceph_mds_request *req,
        return dentry;
 }
 
+static int is_root_ceph_dentry(struct inode *inode, struct dentry *dentry)
+{
+       return ceph_ino(inode) == CEPH_INO_ROOT &&
+               strncmp(dentry->d_name.name, ".ceph", 5) == 0;
+}
+
 /*
  * Look up a single dir entry.  If there is a lookup intent, inform
  * the MDS so that it gets our 'caps wanted' value in a single op.
@@ -554,6 +561,7 @@ static struct dentry *ceph_lookup(struct inode *dir, struct dentry *dentry,
                if (strncmp(dentry->d_name.name,
                            client->mount_args->snapdir_name,
                            dentry->d_name.len) &&
+                   !is_root_ceph_dentry(dir, dentry) &&
                    (ci->i_ceph_flags & CEPH_I_COMPLETE) &&
                    (__ceph_caps_issued_mask(ci, CEPH_CAP_FILE_SHARED, 1))) {
                        di->offset = ci->i_max_offset++;