<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 10, 2020 at 8:41 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Only on EBUSY makes sense to keep trying in a loop.<br>
This helps to use tst_umount() in safe_umount() (next commit).<br>
<br>
Suggested-by: Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>><br>
Reported-by: Jan Stancek <<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>><br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br></blockquote><div><span class="gmail_default" style="font-size:small">Acked-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></span></div><div><span class="gmail_default"></span> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 lib/tst_device.c | 7 +++++++<br>
 1 file changed, 7 insertions(+)<br>
<br>
diff --git a/lib/tst_device.c b/lib/tst_device.c<br>
index 89b9c96de..6cad9bd4d 100644<br>
--- a/lib/tst_device.c<br>
+++ b/lib/tst_device.c<br>
@@ -363,6 +363,13 @@ int tst_umount(const char *path)<br>
                if (!ret)<br>
                        return 0;<br>
<br>
+               if (err != EBUSY) {<br>
+                       tst_resm(TWARN, "umount('%s') failed with %s",<br>
+                        path, tst_strerrno(err));<br>
+                       errno = err;<br>
+                       return ret;<br>
+               }<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Since we have guaranteed there is no other error besides EBUSY, maybe we could cancel the 'err == EBUSY' in the next if() sentences. Otherweise patch 1/3, 2/3 looks good.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">if (i == 0 && err == EBUSY) {<br>    ...<br>         }</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>