[LTP] [PATCH v6 1/2] safe_macros: Use tst_umount() in safe_umount()

Cyril Hrubis chrubis@suse.cz
Fri Feb 7 16:24:07 CET 2020


Hi!
> To get retry workaround for gvfsd-trash background daemon.
> 
> Suggested-by: Cyril Hrubis <chrubis@suse.cz>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> Hi,
> 
> this should be ok, but better to double check.
> Used in fsmount01.

My expectation is that retrying on EBUSY will not break anything. Jan
what do you think?

>  lib/safe_macros.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/safe_macros.c b/lib/safe_macros.c
> index 41fa4ca83..deb19d2df 100644
> --- a/lib/safe_macros.c
> +++ b/lib/safe_macros.c
> @@ -772,7 +772,7 @@ int safe_umount(const char *file, const int lineno, void (*cleanup_fn)(void),
>  {
>  	int rval;
>  
> -	rval = umount(target);
> +	rval = tst_umount(target);
>  
>  	if (rval == -1) {
>  		tst_brkm(TBROK | TERRNO, cleanup_fn,
> -- 
> 2.24.1
> 

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list