github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-11-06 21:58:01 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

selftests/ovl: fix detached layer mountinfo expectations

Browse Source 
set_layers_via_detached_mount_fds was copied from the attached-fd test
and still looks for /tmp paths. Detached open_tree() layers are recorded
by d_path() as upperdir=/upper, workdir=/work, and lowerdir+=/ and
datadir+=/ for each cloned directory. The test has failed since it was
added.

Reported-by: Disha Goel <disgoel@linux.ibm.com>
Link: https://lore.kernel.org/linux-unionfs/20260924112858.51392-2-disgoel@linux.ibm.com
Fixes: ccc829b15d ("selftests: add tests for using detached mount with overlayfs")
Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Link: https://patch.msgid.link/20261017165103.2450042-1-amir73il@gmail.com
Signed-off-by: Christian Brauner (Amutable) <brauner@kernel.org>
This commit is contained in:
Amir Goldstein 2026-10-17 18:51:03 +02:00 committed by Christian Brauner
parent 3a00ecc890
commit 8457ca071f
No known key found for this signature in database
GPG Key ID: 91C61BC06578DCA2
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
tools/testing/selftests/filesystems/overlayfs/set_layers_via_fds.c
View File

@ -686,23 +686,23 @@ TEST_F(set_layers_via_fds, set_layers_via_detached_mount_fds)
while (getline(&line, &len, f_mountinfo) != -1) {
char *haystack = line;
if (strstr(haystack, "workdir=/tmp/w"))
if (strstr(haystack, "workdir=/work"))
layers_found[0] = true;
if (strstr(haystack, "upperdir=/tmp/u"))
if (strstr(haystack, "upperdir=/upper"))
layers_found[1] = true;
if (strstr(haystack, "lowerdir+=/tmp/l1"))
if (strstr(haystack, "lowerdir+=/"))
layers_found[2] = true;
if (strstr(haystack, "lowerdir+=/tmp/l2"))
if (strstr(haystack, "lowerdir+=/"))
layers_found[3] = true;
if (strstr(haystack, "lowerdir+=/tmp/l3"))
if (strstr(haystack, "lowerdir+=/"))
layers_found[4] = true;
if (strstr(haystack, "lowerdir+=/tmp/l4"))
if (strstr(haystack, "lowerdir+=/"))
layers_found[5] = true;
if (strstr(haystack, "datadir+=/tmp/d1"))
if (strstr(haystack, "datadir+=/"))
layers_found[6] = true;
if (strstr(haystack, "datadir+=/tmp/d2"))
if (strstr(haystack, "datadir+=/"))
layers_found[7] = true;
if (strstr(haystack, "datadir+=/tmp/d3"))
if (strstr(haystack, "datadir+=/"))
layers_found[8] = true;
}
free(line);