sysfs: count subdirectories
authorMikulas Patocka <mpatocka@redhat.com>
Thu, 21 Jul 2011 23:59:22 +0000 (19:59 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 23 Aug 2011 00:43:30 +0000 (17:43 -0700)
sysfs: count subdirectories

This patch introduces a subdirectory counter for each sysfs directory.

Without the patch, sysfs_refresh_inode would walk all entries of the directory
to calculate the number of subdirectories.

This patch improves time of "ls -la /sys/block" when there are 10000 block
devices from 9 seconds to 0.19 seconds.

Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/sysfs/dir.c
fs/sysfs/inode.c
fs/sysfs/sysfs.h

index ea9120a830d824feb798db501a4b61fbd98f6071..7d240e6b7176432180585776ac0aaa354195643f 100644 (file)
@@ -47,6 +47,9 @@ static void sysfs_link_sibling(struct sysfs_dirent *sd)
 
        BUG_ON(sd->s_sibling);
 
+       if (sysfs_type(sd) == SYSFS_DIR)
+               parent_sd->s_dir.subdirs++;
+
        /* Store directory entries in order by ino.  This allows
         * readdir to properly restart without having to add a
         * cursor into the s_dir.children list.
@@ -73,6 +76,9 @@ static void sysfs_unlink_sibling(struct sysfs_dirent *sd)
 {
        struct sysfs_dirent **pos;
 
+       if (sysfs_type(sd) == SYSFS_DIR)
+               sd->s_parent->s_dir.subdirs--;
+
        for (pos = &sd->s_parent->s_dir.children; *pos;
             pos = &(*pos)->s_sibling) {
                if (*pos == sd) {
index e3f091a81c724bcf830184e1b8db4d66c40b4c5d..1ee18c81df7814949eae2d3d75ccea96cdcf2039 100644 (file)
@@ -202,18 +202,6 @@ static inline void set_inode_attr(struct inode * inode, struct iattr * iattr)
        inode->i_ctime = iattr->ia_ctime;
 }
 
-static int sysfs_count_nlink(struct sysfs_dirent *sd)
-{
-       struct sysfs_dirent *child;
-       int nr = 0;
-
-       for (child = sd->s_dir.children; child; child = child->s_sibling)
-               if (sysfs_type(child) == SYSFS_DIR)
-                       nr++;
-
-       return nr + 2;
-}
-
 static void sysfs_refresh_inode(struct sysfs_dirent *sd, struct inode *inode)
 {
        struct sysfs_inode_attrs *iattrs = sd->s_iattr;
@@ -230,7 +218,7 @@ static void sysfs_refresh_inode(struct sysfs_dirent *sd, struct inode *inode)
        }
 
        if (sysfs_type(sd) == SYSFS_DIR)
-               inode->i_nlink = sysfs_count_nlink(sd);
+               inode->i_nlink = sd->s_dir.subdirs + 2;
 }
 
 int sysfs_getattr(struct vfsmount *mnt, struct dentry *dentry, struct kstat *stat)
index 845ab3ad229d60616807bb39f0adaceb73a3ad8e..6348e2c753f65b232a092210bc61885f03b099a0 100644 (file)
@@ -19,6 +19,8 @@ struct sysfs_elem_dir {
        struct kobject          *kobj;
        /* children list starts here and goes through sd->s_sibling */
        struct sysfs_dirent     *children;
+
+       unsigned long           subdirs;
 };
 
 struct sysfs_elem_symlink {