iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ovl: relax requirement for non null uuid of lower fs

Browse Source 
We use uuid to associate an overlay lower file handle with a lower layer,
so we can accept lower fs with null uuid as long as all lower layers with
null uuid are on the same fs.

This change allows enabling index and nfs_export features for the setup of
single lower fs of type squashfs - squashfs supports file handles, but has
a null uuid. This change also allows enabling index and nfs_export features
for nested overlayfs, where the lower overlay has nfs_export enabled.

Enabling the index feature with single lower squashfs fixes the
unionmount-testsuite test:
  ./run --ov --squashfs --verify

As a by-product, if, like the lower squashfs, upper fs also uses the
generic export_encode_fh() implementation to export 32bit inode file
handles (e.g. ext4), then the xino_auto config/module/mount option will
enable unique overlay inode numbers.

Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
This commit is contained in:
Amir Goldstein 2018-09-03 09:12:09 +03:00 committed by Miklos Szeredi
parent 6b52243f63
commit 9df085f3c9
2 changed files with 32 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
fs/overlayfs/super.c
View File

@ -1175,10 +1175,30 @@ out:
return err; return err;
} }
/* Get a unique fsid for the layer */ static bool ovl_lower_uuid_ok(struct ovl_fs *ofs, const uuid_t *uuid)
static int ovl_get_fsid(struct ovl_fs *ofs, struct super_block *sb)
{ {
unsigned int i; unsigned int i;
if (!ofs->config.nfs_export && !(ofs->config.index && ofs->upper_mnt))
return true;
for (i = 0; i < ofs->numlowerfs; i++) {
/*
* We use uuid to associate an overlay lower file handle with a
* lower layer, so we can accept lower fs with null uuid as long
* as all lower layers with null uuid are on the same fs.
*/
if (uuid_equal(&ofs->lower_fs[i].sb->s_uuid, uuid))
return false;
}
return true;
}
/* Get a unique fsid for the layer */
static int ovl_get_fsid(struct ovl_fs *ofs, const struct path *path)
{
struct super_block *sb = path->mnt->mnt_sb;
unsigned int i;
dev_t dev; dev_t dev;
int err; int err;
@ -1191,6 +1211,14 @@ static int ovl_get_fsid(struct ovl_fs *ofs, struct super_block *sb)
return i + 1; return i + 1;
} }
if (!ovl_lower_uuid_ok(ofs, &sb->s_uuid)) {
ofs->config.index = false;
ofs->config.nfs_export = false;
pr_warn("overlayfs: %s uuid detected in lower fs '%pd2', falling back to index=off,nfs_export=off.\n",
uuid_is_null(&sb->s_uuid) ? "null" : "conflicting",
path->dentry);
}
err = get_anon_bdev(&dev); err = get_anon_bdev(&dev);
if (err) { if (err) {
pr_err("overlayfs: failed to get anonymous bdev for lowerpath\n"); pr_err("overlayfs: failed to get anonymous bdev for lowerpath\n");
@ -1225,7 +1253,7 @@ static int ovl_get_lower_layers(struct ovl_fs *ofs, struct path *stack,
struct vfsmount *mnt; struct vfsmount *mnt;
int fsid; int fsid;
err = fsid = ovl_get_fsid(ofs, stack[i].mnt->mnt_sb); err = fsid = ovl_get_fsid(ofs, &stack[i]);
if (err < 0) if (err < 0)
goto out; goto out;

3
fs/overlayfs/util.c
View File

@ -65,8 +65,7 @@ struct super_block *ovl_same_sb(struct super_block *sb)
*/ */
int ovl_can_decode_fh(struct super_block *sb) int ovl_can_decode_fh(struct super_block *sb)
{ {
if (!sb->s_export_op || !sb->s_export_op->fh_to_dentry || if (!sb->s_export_op || !sb->s_export_op->fh_to_dentry)
uuid_is_null(&sb->s_uuid))
return 0; return 0;
return sb->s_export_op->encode_fh ? -1 : FILEID_INO32_GEN; return sb->s_export_op->encode_fh ? -1 : FILEID_INO32_GEN;