[PATCH] generic/805: mask unmount failures on cleanup

[PATCH] generic/805: mask unmount failures on cleanup

[Darrick J. Wong]

From: Darrick J. Wong <djwong@kernel.org>

On (what's left of) my NFS-booted QA fleet I get test "failures" like
this:

 --- a/generic/805.out	2026-09-22 13:33:18.326820071 -0700
 +++ b/generic/805.out.bad	2026-10-14 18:58:34.438866321 -0700
 @@ -1,2 +1,4 @@
  QA output created by 805
  Silence is golden
 +umount.nfs: remote share not in 'host:dir' format
 +umount.nfs: /mnt/s: not mounted

This is caused by the _unmount call in _cleanup getting cranky about the
scratch mount not being mounted despite the -q flag.  I think this is
because umount(8) detects the underlying fs as NFS and delegates
entirely to umount.nfs, and that program whines.

So fix the test by redirecting the output.  We've already decided if the
test passes or not before we get to cleanup.

Signed-off-by: "Darrick J. Wong" <djwong@kernel.org>
Reviewed-by: Andrey Albershteyn <aalbersh@kernel.org>
---
 tests/generic/805 |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/generic/805 b/tests/generic/805
index 401829c4c6fcdf..fa454310137a29 100755
--- a/tests/generic/805
+++ b/tests/generic/805
@@ -25,7 +25,7 @@ _cleanup()
 {
 	xfs_freeze -u $SCRATCH_MNT 2>/dev/null
 	$DMSETUP_PROG resume $lvdev >> $seqres.full 2>&1
-	_unmount -q $SCRATCH_MNT
+	_unmount -q $SCRATCH_MNT &>> $seqres.full
 	_dmsetup_remove $node
 	cd /
 	rm -f $tmp.*

Re: [PATCH] generic/805: mask unmount failures on cleanup

[Christoph Hellwig]

On Wed, Oct 14, 2026 at 08:32:36PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@kernel.org>
> 
> On (what's left of) my NFS-booted QA fleet I get test "failures" like
> this:
> 
>  --- a/generic/805.out	2026-09-22 13:33:18.326820071 -0700
>  +++ b/generic/805.out.bad	2026-10-14 18:58:34.438866321 -0700
>  @@ -1,2 +1,4 @@
>   QA output created by 805
>   Silence is golden
>  +umount.nfs: remote share not in 'host:dir' format
>  +umount.nfs: /mnt/s: not mounted
> 
> This is caused by the _unmount call in _cleanup getting cranky about the
> scratch mount not being mounted despite the -q flag.  I think this is
> because umount(8) detects the underlying fs as NFS and delegates
> entirely to umount.nfs, and that program whines.
> 
> So fix the test by redirecting the output.  We've already decided if the
> test passes or not before we get to cleanup.

We really should not end up here with NFS.

Oh, this test is missing a

_require_block_device $SCRATCH_DEV

so that all the dm magic is skipped when on a non-bdev fs.