[LTP] [PATCH v2] lib/tst_test.c: Take account of tst_brk(TCONF/TFAIL) in summary output
Xiao Yang
yangx.jy@cn.fujitsu.com
Fri May 22 03:40:07 CEST 2020
Current summary output doesn't take account of tst_brk(TCONF/TFAIL),
for example:
-----------------------------------------------------
[root@Fedora-30 pidfd_send_signal]# ./pidfd_send_signal01
tst_test.c:1246: INFO: Timeout per run is 0h 05m 00s
../../../../include/lapi/pidfd_send_signal.h:16: CONF: syscall(424) __NR_pidfd_send_signal not supported
Summary:
passed 0
failed 0
skipped 0
warnings 0
----------------------------------------------------
1) Add update_result() in tst_vbrk_() to fix the issue.
2) Remove redundant update_result() in run_tcases_per_fs() after the fix.
Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
---
lib/tst_test.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/lib/tst_test.c b/lib/tst_test.c
index 0e58060e0..e93c88ba5 100644
--- a/lib/tst_test.c
+++ b/lib/tst_test.c
@@ -316,6 +316,7 @@ void tst_vbrk_(const char *file, const int lineno, int ttype,
const char *fmt, va_list va)
{
print_result(file, lineno, ttype, fmt, va);
+ update_results(TTYPE_RESULT(ttype));
/*
* The getpid implementation in some C library versions may cause cloned
@@ -1316,10 +1317,8 @@ static int run_tcases_per_fs(void)
mntpoint_mounted = 0;
}
- if (ret == TCONF) {
- update_results(ret);
+ if (ret == TCONF)
continue;
- }
if (ret == 0)
continue;
--
2.21.0
More information about the ltp
mailing list