On 3/18/25 6:51 PM, Theodore Ts'o wrote: > On Tue, Mar 18, 2025 at 03:39:33PM -0500, Eric Sandeen wrote: >> generic/699 fails because _overlay_mount_dirs() uses _common_dev_mount_options(), >> and when FSTYP is ext4, that yields "-o acl,user_xattr" and acl is not a valid >> option for overlayfs. > > Hmm, I hadn't noticed because the test is getting skipped for me: > > BEGIN TEST 4k (1 test): Ext4 4k block Tue Mar 18 19:44:51 EDT 2025 > DEVICE: /dev/vdb > EXT_MKFS_OPTIONS: -b 4096 > EXT_MOUNT_OPTIONS: -o block_validity > FSTYP -- ext4 > PLATFORM -- Linux/x86_64 kvm-xfstests 6.14.0-rc2-xfstests-00063-g6630cf085eb0 #14 SMP PREEMPT_DYNAMIC Mon Mar 17 11:19:56 EDT 2025 > MKFS_OPTIONS -- -F -q -b 4096 /dev/vdc > MOUNT_OPTIONS -- -o acl,user_xattr -o block_validity /dev/vdc /vdc > > generic/699 [19:44:51][ 12.536740] run fstests generic/699 at 2025-03-18 19:44:51 > [ 12.794084] overlay: Unknown parameter 'acl' > [19:44:52] [not run] > generic/699 -- overlayfs doesn't support idmappped layers > Ran: generic/699 > > I haven't really played with idmapped mount. Is there some CONFIG > option I need to enable this test to run? > > - Ted I shouldn't have said "failed" - you're seeing what I'm seeing, it's skipped (not failed) because it tries to test overlayfs idmapped layers using the ext4 default options, and acl mounts fail, skipping the test. -Eric