Re: [PATCH] generic/409-411: remove all after unbind mountpoints

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Tue, Mar 7, 2017 at 9:48 AM, Zorro Lang <zlang@xxxxxxxxxx> wrote:
> After some shared subtrees test (bind/slave/shared/private), maybe
> some dentries isn't freed. For example, EBUSY maybe returned due to
> some bugs. This patch tries to verify that.
>
> Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx>
> ---
>  tests/generic/409 | 1 +
>  tests/generic/410 | 1 +
>  tests/generic/411 | 1 +
>  3 files changed, 3 insertions(+)
>
> diff --git a/tests/generic/409 b/tests/generic/409
> index d7c39d6..76d657f 100755
> --- a/tests/generic/409
> +++ b/tests/generic/409
> @@ -48,6 +48,7 @@ _cleanup()
>         cd /
>         rm -f $tmp.*
>         _clear_mount_stack
> +       rm -rf $MNTHEAD $mpA $mpB $mpC $mpD

$mp* are under $MNTHEAD, so why bother listing them here?

It would be wise to also rm -rf $MNTHEAD before mkdir $MNTHEAD
or at least mkdir -p $MNTHEAD.
rm -rf is better because TEST_DIR may have a file by that name
(e.g. from failure to cleanup after generic/247).

>  }
>
>  # get standard environment, filters and checks
> diff --git a/tests/generic/410 b/tests/generic/410
> index e627cfe..f39b4fe 100755
> --- a/tests/generic/410
> +++ b/tests/generic/410
> @@ -56,6 +56,7 @@ _cleanup()
>         cd /
>         rm -f $tmp.*
>         _clear_mount_stack
> +       rm -rf $MNTHEAD $mpA $mpB $mpC
>  }
>
>  # get standard environment, filters and checks
> diff --git a/tests/generic/411 b/tests/generic/411
> index 414d3a5..e6afaf9 100755
> --- a/tests/generic/411
> +++ b/tests/generic/411
> @@ -37,6 +37,7 @@ _cleanup()
>         cd /
>         rm -f $tmp.*
>         _clear_mount_stack
> +       rm -rf $MNTHEAD $mpA $mpB $mpC
>  }
>
>  # get standard environment, filters and checks
> --
> 2.7.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux