Re: [LTP] [PATCH 1/3] nfs_lib.sh: Log NFS mount and unmount

Richard Palethorpe
2023-09-20T11:27:10Z

Reviewed-by: Richard Palethorpe <rpalethorpe@suse.com>

Martin Doucha <mdoucha@suse.cz> writes:

> Signed-off-by: Martin Doucha <mdoucha@suse.cz>
> ---
> testcases/network/nfs/nfs_stress/nfs_lib.sh | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/testcases/network/nfs/nfs_stress/nfs_lib.sh b/testcases/network/nfs/nfs_stress/nfs_lib.sh
> index abf7ba5a2..a996f7cc8 100644
> --- a/testcases/network/nfs/nfs_stress/nfs_lib.sh
> +++ b/testcases/network/nfs/nfs_stress/nfs_lib.sh
> @@ -191,7 +191,9 @@ nfs_setup()
>
> remote_dir="$(get_remote_dir $i $type)"
> nfs_setup_server "$remote_dir" "$(($$ + n))"
> - nfs_mount "$(get_local_dir $i $n)" "$remote_dir" "-o proto=$type,vers=$i"
> + local_dir="$(get_local_dir $i $n)"
> + tst_res TINFO "Mounting $local_dir"
> + nfs_mount "$local_dir" "$remote_dir" "-o proto=$type,vers=$i"
>
> n=$(( n + 1 ))
> done
> @@ -210,7 +212,10 @@ nfs_cleanup()
> local n=0
> for i in $VERSION; do
> local_dir="$(get_local_dir $i $n)"
> - grep -q "$local_dir" /proc/mounts && umount $local_dir
> + if grep -q "$local_dir" /proc/mounts; then
> + tst_res TINFO "Unmounting $local_dir"
> + umount $local_dir
> + fi
> n=$(( n + 1 ))
> done
> sleep 2
> --
> 2.42.0

--
Thank you,
Richard.