On Mon, Oct 30, 2023 at 2:15 PM Anand Jain <anand.jain@xxxxxxxxxx> wrote:
>
> This test case checks for failure of the cloned device mounts, which
> is no longer true after the commit a5b8a5f9f835 ("btrfs: support
> cloned-device mount capability"). So check for the non-presence the
> temp-fsid feature and do not test for the failure of the cloned device
> mount.
>
> Reported-by: kernel test robot <oliver.sang@xxxxxxxxx>
> Closes: https://lore.kernel.org/oe-lkp/202310251645.5fe5495a-oliver.sang@xxxxxxxxx
> Signed-off-by: Anand Jain <anand.jain@xxxxxxxxxx>
Reviewed-by: Filipe Manana <fdmanana@xxxxxxxx>
Looks good, thanks.
> ---
>
> v3: check only that mount of clone device fails if temp-fsid is not
> supported.
>
> tests/btrfs/219 | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/tests/btrfs/219 b/tests/btrfs/219
> index 44a4c79dc05d..3c414dd9c2e0 100755
> --- a/tests/btrfs/219
> +++ b/tests/btrfs/219
> @@ -51,6 +51,7 @@ loop_dev2=""
>
> _require_test
> _require_loop
> +_require_btrfs_fs_sysfs
> _require_btrfs_forget_or_module_loadable
> _fixed_by_kernel_commit 5f58d783fd78 \
> "btrfs: free device in btrfs_close_devices for a single device filesystem"
> @@ -88,14 +89,16 @@ _mount $loop_dev1 $loop_mnt1 > /dev/null 2>&1 || \
> _fail "Failed to mount the second time"
> $UMOUNT_PROG $loop_mnt1
>
> -# Now we definitely can't mount them at the same time, because we're still tied
> -# to the limitation of one fs_devices per fsid.
> +# Now try mount them at the same time, if kernel does not support
> +# temp-fsid feature then mount will fail.
> _btrfs_forget_or_module_reload
>
> _mount $loop_dev1 $loop_mnt1 > /dev/null 2>&1 || \
> _fail "Failed to mount the third time"
> -_mount $loop_dev2 $loop_mnt2 > /dev/null 2>&1 && \
> - _fail "We were allowed to mount when we should have failed"
> +if ! _has_btrfs_sysfs_feature_attr temp_fsid; then
> + _mount $loop_dev2 $loop_mnt2 > /dev/null 2>&1 && \
> + _fail "We were allowed to mount when we should have failed"
> +fi
>
> _btrfs_rescan_devices
> # success, all done
> --
> 2.39.3
>
