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

[Petr Vorel]

Hi,

> > I agree. Though I'd like to point out that tst_umount() currently loops
> > on any error, not just EBUSY.

> Right, I guess that we should change that to be on the safe side, it was
> never intended to retry anything else than EBUSY.

> What about?
Acked-by: Petr Vorel <pvorel@suse.cz>


> diff --git a/lib/tst_device.c b/lib/tst_device.c
> index 89b9c96de..52a5b37fd 100644
> --- a/lib/tst_device.c
> +++ b/lib/tst_device.c
> @@ -371,6 +371,9 @@ int tst_umount(const char *path)
>                                  "mounted fs, kill it to speed up tests.");
>                 }

> +               if (err != EBUSY) {
> +                       errno = err;
> +                       return ret;
> +               }
> +
>                 usleep(100000);
>         }

Kind regards,
Petr