lkml.org 
[lkml]   [2014]   [May]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: Unionmount and overlayfs testsuite
Date
J. R. Okajima <hooanon05g@gmail.com> 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


\
 
 \ /
  Last update: 2014-05-30 11:41    [W:0.264 / U:0.704 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site