<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, Mar 30, 2020 at 8:43 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">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">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>><br>
Reviewed-by: Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com" target="_blank">xuyang2018.jy@cn.fujitsu.com</a>><br>
Reviewed-by: Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>><br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
 doc/test-writing-guidelines.txt                 |  4 ++--<br>
 include/tst_safe_macros.h                       |  5 ++---<br>
 testcases/kernel/syscalls/add_key/add_key05.c   | 14 ++++----------<br>
 testcases/kernel/syscalls/quotactl/quotactl01.c | 11 ++---------<br>
 testcases/kernel/syscalls/quotactl/quotactl06.c |  9 +--------<br>
 5 files changed, 11 insertions(+), 32 deletions(-)<br>
<br>
diff --git a/doc/test-writing-guidelines.txt b/doc/test-writing-guidelines.txt<br>
index 6f1eee490..75c2e517a 100644<br>
--- a/doc/test-writing-guidelines.txt<br>
+++ b/doc/test-writing-guidelines.txt<br>
@@ -1283,8 +1283,8 @@ return value is '255' if 'execvp()' failed with 'ENOENT' and '254' otherwise.<br>
 'stdout_path' and 'stderr_path' determine where to redirect the program<br>
 stdout and stderr I/O streams.<br>
<br>
-The 'SAFE_CMD()' macro can be used automatic handling non zero exits (exits<br>
-with 'TBROK') or 'ENOENT' (exits with 'TCONF').<br>
+The 'SAFE_CMD()' macro can be used automatic handling non-zero exits (exits<br>
+with 'TBROK') and 'ENOENT' (exits with 'TCONF').<br>
<br>
 .Example<br>
 [source,c]<br>
diff --git a/include/tst_safe_macros.h b/include/tst_safe_macros.h<br>
index 38eb67d1d..bcd4b9c9e 100644<br>
--- a/include/tst_safe_macros.h<br>
+++ b/include/tst_safe_macros.h<br>
@@ -540,9 +540,8 @@ void safe_cmd(const char *file, const int lineno, const char *const argv[],<br>
 {<br>
        int rval;<br>
<br>
-       switch ((rval = tst_run_cmd(argv, stdout_path, stderr_path,<br>
-                                   TST_RUN_CMD_PASS_EXIT_VAL |<br>
-                                   TST_RUN_CMD_CHECK_CMD))) {<br>
+       switch ((rval = tst_cmd(argv, stdout_path, stderr_path, TST_CMD_PASS_RETVAL<br>
+                                                       | TST_CMD_CHECK_CMD))) {<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Oh, you did the renaming in patch 5/5, but pity that the TST_CMD_TCONF_ON_MISSING still not being used:). I guess this modification should belong to patch4/5.</div></div><div><br></div></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>