J. R. Okajima <hooanon05g@xxxxxxxxx> wrote: > I've found some interesting cases. > > - impermissible.test, > open_file_as_bin -t -w $file -E EACCES > When $termslash is "/", a '/' is appended to the expanded $file, such > as "/path/fileA/". If fileA is a regular file, that path is obviously > wrong. Does UnionMount expect EACCES in this case too? > Should it be ENOTDIR? > It might be better to change > errcode=EACCES > test ! "$termslash" = "" && errcode=ENOTDIR > open_file_as_bin -t -w $file -E $errcode I'd never got to the end of the impermissible test because the utimes test fails on both unionmount and overlayfs. I'll have to address the termslash alterations at some point. > - readlink.test, > fs_op readlink $file -R $testdir/direct_dir_sym100 ${termslash:+-E EINVAL} > It expects "$testdir/direct_dir_sym100". Does it mean UnionMount > converts the target path? > For example, > - /u = /rw + /ro > - /rw/symlinkA doesn't exist > - /ro/symlinkA points to /ro/fileA No. Unionmount unions are constructed by mounting the lower layers all on the same mountpoint and then mounting the union over them, also on the same mountpoint. So their paths are coincident. > Does readlink(2) return "/u/fileA" instead of /ro/fileA?" No. The test suite sets the lower symlink to point to the union path for its target. [root@andromeda union-testsuite]# readlink /lower/a/indirect_dir_sym100 /mnt/a/direct_dir_sym100 > And all tests should be done by a superuser? Yes. It also will test things like mknod at some point and runs things as other users to test permissions. David -- 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