[RFC 4/7] cramfs: allow rmdir

From: arnd
Date: Sat May 31 2008 - 11:41:03 EST


rmdir in cramfs works much like unlink, but we need to check
first that the directory is empty. Since the link count is
always 1, we have to look at the directory contents for that,
but fortunately, we can abuse the readdir logic for that,
to check if at least one entry exists.

Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
---
fs/cramfs/inode.c | 71 +++++++++++++++++++++++++++++++++++++++++++++-------
1 files changed, 61 insertions(+), 10 deletions(-)

diff --git a/fs/cramfs/inode.c b/fs/cramfs/inode.c
index 6b9f21f..8c3e8bb 100644
--- a/fs/cramfs/inode.c
+++ b/fs/cramfs/inode.c
@@ -326,18 +326,20 @@ static int cramfs_statfs(struct dentry *dentry, struct kstatfs *buf)
/*
* Read a cramfs directory entry.
*/
-static int cramfs_readdir(struct file *filp, void *dirent, filldir_t filldir)
+static int cramfs_readdir_ondisk(struct dentry *dentry, void *dirent,
+ filldir_t filldir, loff_t *pos)
{
- struct inode *inode = filp->f_path.dentry->d_inode;
+ struct inode *inode = dentry->d_inode;
struct super_block *sb = inode->i_sb;
char *buf;
unsigned int offset;
int copied;

/* Offset within the thing. */
- offset = filp->f_pos;
+ offset = *pos;
if (offset >= inode->i_size)
return 0;
+
/* Directory entries are always 4-byte aligned */
if (offset & 3)
return -EINVAL;
@@ -355,7 +357,7 @@ static int cramfs_readdir(struct file *filp, void *dirent, filldir_t filldir)
ino_t ino;
mode_t mode;
int namelen, error;
- struct dentry *dentry;
+ struct dentry *d;

mutex_lock(&read_mutex);
de = cramfs_read(sb, OFFSET(inode) + offset, sizeof(*de)+CRAMFS_MAXPATHLEN);
@@ -382,25 +384,32 @@ static int cramfs_readdir(struct file *filp, void *dirent, filldir_t filldir)
namelen--;
}
qstr.len = namelen;
- dentry = d_hash_and_lookup(filp->f_path.dentry, &qstr);
+ d = d_hash_and_lookup(dentry, &qstr);

error = 0;
- if (!dentry || (dentry->d_inode))
- error = filldir(dirent, buf, namelen, offset,
- ino, mode >> 12);
+ if (!d || (d->d_inode))
+ error = filldir(dirent, buf, namelen,
+ offset, ino, mode >> 12);

- dput(dentry);
+ dput(d);
if (error)
break;

offset = nextoffset;
- filp->f_pos = offset;
+ *pos = offset;
copied++;
}
kfree(buf);
+
return 0;
}

+static int cramfs_readdir(struct file *filp, void *dirent, filldir_t filldir)
+{
+ return cramfs_readdir_ondisk(filp->f_dentry, dirent,
+ filldir, &filp->f_pos);
+}
+
int cramfs_unlink(struct inode *dir, struct dentry *dentry)
{
struct inode *inode = dentry->d_inode;
@@ -414,6 +423,47 @@ int cramfs_unlink(struct inode *dir, struct dentry *dentry)
dget(dentry);
return 0;
}
+
+/*
+ * We use the readdir infrastructure to test whether a directory is
+ * empty or not. cramfs_test_filldir will get called for any on-disk
+ * dentries that have not been unlinked.
+ */
+static int cramfs_test_filldir(void * __buf, const char * name, int namlen,
+ loff_t offset, u64 ino, unsigned int d_type)
+{
+ int *buf = __buf;
+ *buf = 0;
+ return -ENOTEMPTY;
+}
+
+static int cramfs_empty(struct dentry *dentry)
+{
+ loff_t pos = 0;
+ int empty = 1;
+ int ret;
+ if (!dentry->d_inode || !S_ISDIR(dentry->d_inode->i_mode))
+ return 1;
+
+ if (!simple_empty(dentry))
+ return 0;
+
+ ret = cramfs_readdir_ondisk(dentry, &empty, cramfs_test_filldir, &pos);
+ if (ret)
+ return ret;
+
+ return empty;
+}
+
+int cramfs_rmdir(struct inode *dir, struct dentry *dentry)
+{
+ if (!cramfs_empty(dentry))
+ return -ENOTEMPTY;
+
+ cramfs_unlink(dir, dentry);
+ return 0;
+}
+
/*
* Lookup and fill in the inode data..
*/
@@ -536,6 +586,7 @@ static const struct file_operations cramfs_directory_operations = {

static const struct inode_operations cramfs_dir_inode_operations = {
.unlink = cramfs_unlink,
+ .rmdir = cramfs_rmdir,
.lookup = cramfs_lookup,
};

--
1.5.4.3

--

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/