From: Valerie Aurora <vaurora@xxxxxxxxxx> d_free_unions() frees the union stack associated with a directory. --- fs/union.c | 24 ++++++++++++++++++++++++ fs/union.h | 7 +++++++ 2 files changed, 31 insertions(+), 0 deletions(-) diff --git a/fs/union.c b/fs/union.c index c8d7766..77d6a74 100644 --- a/fs/union.c +++ b/fs/union.c @@ -19,6 +19,7 @@ #include <linux/mount.h> #include <linux/fs_struct.h> #include <linux/slab.h> +#include <linux/namei.h> #include "union.h" @@ -36,3 +37,26 @@ static struct union_stack *union_alloc(struct path *topmost) return kcalloc(sizeof(struct path), layers, GFP_KERNEL); } + +/** + * d_free_unions - free all unions for this dentry + * @dentry: topmost dentry in the union stack to remove + * + * This must be called when freeing a dentry. + */ +void d_free_unions(struct dentry *topmost) +{ + struct path *path; + unsigned int i, layers = topmost->d_sb->s_union_count; + + if (!IS_DIR_UNIONED(topmost)) + return; + + for (i = 0; i < layers; i++) { + path = union_find_dir(topmost, i); + if (path->mnt) + path_put(path); + } + kfree(topmost->d_union_stack); + topmost->d_union_stack = NULL; +} diff --git a/fs/union.h b/fs/union.h index bd9c2b0..5b79131 100644 --- a/fs/union.h +++ b/fs/union.h @@ -49,6 +49,10 @@ struct union_stack { struct path u_dirs[0]; }; +#define IS_DIR_UNIONED(dentry) ((dentry)->d_union_stack) + +extern void d_free_unions(struct dentry *); + static inline struct path *union_find_dir(struct dentry *dentry, unsigned int layer) { @@ -58,6 +62,9 @@ struct path *union_find_dir(struct dentry *dentry, unsigned int layer) #else /* CONFIG_UNION_MOUNT */ +#define IS_DIR_UNIONED(x) (0) + +#define d_free_unions(x) do { } while (0) #define union_find_dir(x, y) ({ BUG(); (NULL); }) #endif /* CONFIG_UNION_MOUNT */ -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html