Commit 9d7459d8 authored by Miklos Szeredi's avatar Miklos Szeredi

ovl: multi-layer readdir

If multiple lower layers exist, merge them as well in readdir according to
the same rules as merging upper with lower.  I.e. take whiteouts and opaque
directories into account on all but the lowers layer.
Signed-off-by: default avatarMiklos Szeredi <mszeredi@suse.cz>
parent 5ef88da5
...@@ -261,35 +261,34 @@ static void ovl_dir_reset(struct file *file) ...@@ -261,35 +261,34 @@ static void ovl_dir_reset(struct file *file)
static int ovl_dir_read_merged(struct dentry *dentry, struct list_head *list) static int ovl_dir_read_merged(struct dentry *dentry, struct list_head *list)
{ {
int err; int err;
struct path lowerpath; struct path realpath;
struct path upperpath;
struct ovl_readdir_data rdd = { struct ovl_readdir_data rdd = {
.ctx.actor = ovl_fill_merge, .ctx.actor = ovl_fill_merge,
.list = list, .list = list,
.root = RB_ROOT, .root = RB_ROOT,
.is_merge = false, .is_merge = false,
}; };
int idx, next;
ovl_path_lower(dentry, &lowerpath);
ovl_path_upper(dentry, &upperpath); for (idx = 0; idx != -1; idx = next) {
next = ovl_path_next(idx, dentry, &realpath);
if (upperpath.dentry) {
rdd.dir = upperpath.dentry; if (next != -1) {
err = ovl_dir_read(&upperpath, &rdd); rdd.dir = realpath.dentry;
if (err) err = ovl_dir_read(&realpath, &rdd);
goto out; if (err)
} break;
if (lowerpath.dentry) { } else {
/* /*
* Insert lowerpath entries before upperpath ones, this allows * Insert lowest layer entries before upper ones, this
* offsets to be reasonably constant * allows offsets to be reasonably constant
*/ */
list_add(&rdd.middle, rdd.list); list_add(&rdd.middle, rdd.list);
rdd.is_merge = true; rdd.is_merge = true;
err = ovl_dir_read(&lowerpath, &rdd); err = ovl_dir_read(&realpath, &rdd);
list_del(&rdd.middle); list_del(&rdd.middle);
}
} }
out:
return err; return err;
} }
......
...@@ -81,6 +81,9 @@ enum ovl_path_type ovl_path_type(struct dentry *dentry) ...@@ -81,6 +81,9 @@ enum ovl_path_type ovl_path_type(struct dentry *dentry)
} else if (!oe->opaque) { } else if (!oe->opaque) {
type |= __OVL_PATH_PURE; type |= __OVL_PATH_PURE;
} }
} else {
if (oe->numlower > 1)
type |= __OVL_PATH_MERGE;
} }
return type; return type;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment