From: Cyril Hrubis <chrubis@suse.cz>
To: Petr Vorel <pvorel@suse.cz>
Cc: Jeff Moyer <jmoyer@redhat.com>, ltp@lists.linux.it
Subject: Re: [LTP] [PATCH 0/3] tst_device: add support for overlayfs
Date: Fri, 7 Feb 2025 13:38:10 +0100 [thread overview]
Message-ID: <Z6X-skb0ce8MYUnb@yuki.lan> (raw)
In-Reply-To: <20250207113726.GB1739723@pevik>
Hi!
> > tst_tmpdir.c:316: TINFO: Using /mnt/fstests/TEST_DIR/ovl-mnt/ltp-hSHEHy5M0s/LTP_aio4q4GMW as tmpdir (overlayfs filesystem)
> > tst_test.c:1888: TINFO: LTP version: 20220121-2271-g91a10df22
> > tst_test.c:1892: TINFO: Tested kernel: vendor kernel
> > tst_test.c:1723: TINFO: Timeout per run is 0h 40m 00s
> > aiocp.c:211: TINFO: Maximum AIO blocks: 65536
> > tst_device.c:551: TINFO: Use BTRFS specific strategy
> > tst_device.c:569: TBROK: BTRFS ioctl failed. Is . on a tmpfs?: ENOTTY (25)
>
> > The issue is that stat(2) on an overlayfs mount point will return a
> > major device number of 0. The code assumes this is btrfs, and tries
> > to issue a btrfs-specific ioctl. When that fails, the final message is
> > printed that suggests this might be tmpfs.
>
> > I modified the code to use statfs(2) to get the file system type, and
> > use that to determine which file system specific code to call. Finally, I
> > added code to parse out the upper directory from the overlayfs mount options
> > using libmount. libmount is part of util-linux, so it should be fairly
>
> We try to avoid external libraries to help testing kernel on some minimal
> embedded devices. @Cyril: is it ok to drag libmount-devel?
For the base LTP library? I would say no.
--
Cyril Hrubis
chrubis@suse.cz
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next prev parent reply other threads:[~2025-02-07 12:38 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-03 22:05 [LTP] [PATCH 0/3] tst_device: add support for overlayfs Jeff Moyer
2025-02-03 22:05 ` [LTP] [PATCH 1/3] lib/tst_device.c: factor out btrfs-specific logic from tst_find_backing_dev Jeff Moyer
2025-02-03 22:05 ` [LTP] [PATCH 2/3] lib/tst_device.c: check for BTRFS_SUPER_MAGIC instead of device major of 0 Jeff Moyer
2025-02-03 22:06 ` [LTP] [PATCH 3/3] lib/tst_device.c: add support for overlayfs Jeff Moyer
2025-02-07 11:23 ` Petr Vorel
2025-02-07 11:37 ` [LTP] [PATCH 0/3] tst_device: " Petr Vorel
2025-02-07 12:38 ` Cyril Hrubis [this message]
2025-02-07 15:28 ` Jeff Moyer
2025-02-07 15:27 ` Jeff Moyer
2025-02-07 17:08 ` Jeff Moyer
2025-02-07 17:15 ` Petr Vorel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Z6X-skb0ce8MYUnb@yuki.lan \
--to=chrubis@suse.cz \
--cc=jmoyer@redhat.com \
--cc=ltp@lists.linux.it \
--cc=pvorel@suse.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.