* [LTP] [PATCH v3 1/3] remove old tst_flush()
@ 2018-03-09 14:16 Michael Moese
2018-03-09 14:16 ` [LTP] [PATCH v3 2/3] Add new tst_flush() library function Michael Moese
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Michael Moese @ 2018-03-09 14:16 UTC (permalink / raw)
To: ltp
The old tst_flush() was renamed to tst_old_flush(), and tst_flush()
was removed from tstapicmd.c.
Present calls to tst_flush() were changed to tst_old_flush().
Signed-off-by: Michael Moese <mmoese@suse.de>
---
doc/man3/tst_res.3 | 6 +++---
include/old/test.h | 6 +++---
lib/tests/trerrno.c | 2 +-
lib/tst_res.c | 8 ++++----
testcases/kernel/controllers/cgroup_xattr/cgroup_xattr.c | 4 ++--
testcases/kernel/syscalls/epoll/epoll-ltp.c | 4 ++--
testcases/kernel/syscalls/fork/fork07.c | 2 +-
testcases/kernel/syscalls/ipc/shmctl/shmctl01.c | 2 +-
testcases/kernel/syscalls/modify_ldt/modify_ldt02.c | 2 +-
tools/apicmds/ltpapicmd.c | 4 ----
10 files changed, 18 insertions(+), 22 deletions(-)
diff --git a/doc/man3/tst_res.3 b/doc/man3/tst_res.3
index e1f50c26b..56f722436 100644
--- a/doc/man3/tst_res.3
+++ b/doc/man3/tst_res.3
@@ -39,7 +39,7 @@ tst_resm_hexd \- Print result message, including specified buffer in hexadecimal
.sp
tst_brkm \- Print result message and break remaining test cases
.sp
-tst_flush \- Print any messages pending because of CONDENSE mode, and flush output stream
+tst_old_flush \- Print any messages pending because of CONDENSE mode, and flush output stream
.sp
tst_exit \- Exit test with a meaningful exit value
.sp
@@ -56,7 +56,7 @@ char *\fItmesg, [arg ...]\fR)
\fBvoid tst_brkm(int \fIttype\fB, void (*\fIfunc\fB)(), char *\fItmesg,
[arg ...]\fR)
.P
-\fBvoid tst_flush()
+\fBvoid tst_old_flush()
.P
\fBvoid tst_exit()
.P
@@ -180,7 +180,7 @@ after all results have been printed. If \fBtst_brk()\fR is called with a
first reported result. \fBtst_brk()\fR takes the \fIfname\fR argument
whereas \fBtst_brkm()\fR does not.
.P
-\fBtst_flush()\fR is used to print any results pending because of
+\fBtst_old_flush()\fR is used to print any results pending because of
\fBCONDENSE\fR or \fBNOPASS\fR modes (described below), and flushes the
output stream.
.P
diff --git a/include/old/test.h b/include/old/test.h
index 74d11a123..5eef304ab 100644
--- a/include/old/test.h
+++ b/include/old/test.h
@@ -102,7 +102,7 @@
#ifdef UCLINUX
# define FORK_OR_VFORK tst_vfork
# define MAP_PRIVATE_EXCEPT_UCLINUX 0
-/* tst_flush() + vfork() */
+/* tst_old_flush() + vfork() */
pid_t tst_vfork(void);
#else
# define FORK_OR_VFORK tst_fork
@@ -176,10 +176,10 @@ void tst_brkm_(const char *file, const int lineno, int ttype,
void tst_require_root(void);
void tst_exit(void) LTP_ATTRIBUTE_NORETURN;
-void tst_flush(void);
+void tst_old_flush(void);
/*
- * tst_flush() + fork
+ * tst_old_flush() + fork
* NOTE: tst_fork() will reset T_exitval to 0 for child process.
*/
pid_t tst_fork(void);
diff --git a/lib/tests/trerrno.c b/lib/tests/trerrno.c
index 49d50e9df..a160874de 100644
--- a/lib/tests/trerrno.c
+++ b/lib/tests/trerrno.c
@@ -57,7 +57,7 @@ int main(void)
TEST_ERRNO = EPERM;
TEST_RETURN = EINVAL;
tst_resm(TINFO | TRERRNO, "test");
- tst_flush();
+ tst_old_flush();
/* restore stdout */
TEST(dup2(stdout_fd, fileno(stdout)));
diff --git a/lib/tst_res.c b/lib/tst_res.c
index b56f37db0..5ea3531a4 100644
--- a/lib/tst_res.c
+++ b/lib/tst_res.c
@@ -245,7 +245,7 @@ static void tst_condense(int tnum, int ttype, const char *tmesg)
Buffered = TRUE;
}
-void tst_flush(void)
+void tst_old_flush(void)
{
NO_NEWLIB_ASSERT("Unknown", 0);
@@ -399,7 +399,7 @@ void tst_exit(void)
pthread_mutex_lock(&tmutex);
- tst_flush();
+ tst_old_flush();
T_exitval &= ~TINFO;
@@ -415,7 +415,7 @@ pid_t tst_fork(void)
NO_NEWLIB_ASSERT("Unknown", 0);
- tst_flush();
+ tst_old_flush();
child = fork();
if (child == 0)
@@ -460,7 +460,7 @@ pid_t tst_vfork(void)
{
NO_NEWLIB_ASSERT("Unknown", 0);
- tst_flush();
+ tst_old_flush();
return vfork();
}
diff --git a/testcases/kernel/controllers/cgroup_xattr/cgroup_xattr.c b/testcases/kernel/controllers/cgroup_xattr/cgroup_xattr.c
index ad291a1eb..71a03d12c 100644
--- a/testcases/kernel/controllers/cgroup_xattr/cgroup_xattr.c
+++ b/testcases/kernel/controllers/cgroup_xattr/cgroup_xattr.c
@@ -219,10 +219,10 @@ static void cleanup(void)
/*
* Kernels 3.7 can crash while unmounting cgroups with xattr,
- * call tst_flush() to make sure all buffered data written
+ * call tst_old_flush() to make sure all buffered data written
* before it happens
*/
- tst_flush();
+ tst_old_flush();
int i;
for (i = 0; i < odir_num; ++i) {
diff --git a/testcases/kernel/syscalls/epoll/epoll-ltp.c b/testcases/kernel/syscalls/epoll/epoll-ltp.c
index cb82fd865..12504ab95 100644
--- a/testcases/kernel/syscalls/epoll/epoll-ltp.c
+++ b/testcases/kernel/syscalls/epoll/epoll-ltp.c
@@ -142,7 +142,7 @@ do { \
pid_t kid_pid; \
int kid_status; \
\
- tst_flush(); \
+ tst_old_flush(); \
kid_pid = FORK_OR_VFORK(); \
if (kid_pid == 0) {
@@ -176,7 +176,7 @@ do { \
pid_t kid_pid; \
int kid_status; \
\
- tst_flush(); \
+ tst_old_flush(); \
kid_pid = FORK_OR_VFORK(); \
if (kid_pid == 0) { /* Run the function */ \
return fn(epoll_fd); \
diff --git a/testcases/kernel/syscalls/fork/fork07.c b/testcases/kernel/syscalls/fork/fork07.c
index 4344fdf4c..e596867c3 100644
--- a/testcases/kernel/syscalls/fork/fork07.c
+++ b/testcases/kernel/syscalls/fork/fork07.c
@@ -110,7 +110,7 @@ int main(int ac, char **av)
/* fork off the children */
tst_resm(TINFO, "Forking %d children", Nforks);
- tst_flush();
+ tst_old_flush();
for (forks = 0; forks < Nforks; forks++) {
pid1 = fork();
if (pid1 == 0) {
diff --git a/testcases/kernel/syscalls/ipc/shmctl/shmctl01.c b/testcases/kernel/syscalls/ipc/shmctl/shmctl01.c
index ddf6b86d1..1b4697755 100644
--- a/testcases/kernel/syscalls/ipc/shmctl/shmctl01.c
+++ b/testcases/kernel/syscalls/ipc/shmctl/shmctl01.c
@@ -228,7 +228,7 @@ void stat_setup(void)
*/
set_shared = set_shmat();
- tst_flush();
+ tst_old_flush();
for (stat_i = 0; stat_i < N_ATTACH; stat_i++) {
pid = FORK_OR_VFORK();
if (pid == -1)
diff --git a/testcases/kernel/syscalls/modify_ldt/modify_ldt02.c b/testcases/kernel/syscalls/modify_ldt/modify_ldt02.c
index b82d14ea3..c953ac420 100644
--- a/testcases/kernel/syscalls/modify_ldt/modify_ldt02.c
+++ b/testcases/kernel/syscalls/modify_ldt/modify_ldt02.c
@@ -140,7 +140,7 @@ int main(int ac, char **av)
tst_brkm(TINFO, cleanup, "Creation of segment failed");
}
- tst_flush();
+ tst_old_flush();
if ((pid = FORK_OR_VFORK()) == 0) {
val = read_segment(0);
exit(1);
diff --git a/tools/apicmds/ltpapicmd.c b/tools/apicmds/ltpapicmd.c
index 9ad68b589..4b66c4226 100644
--- a/tools/apicmds/ltpapicmd.c
+++ b/tools/apicmds/ltpapicmd.c
@@ -34,8 +34,6 @@
* break remaining test cases
* tst_res - Print result message, including file contents
* tst_resm - Print result message
- * tst_flush - Print any messages pending because of CONDENSE mode,
- * and flush output stream
* tst_exit - Exit test with a meaningful exit value
*
* These are the minimum set of functions or commands required to
@@ -332,8 +330,6 @@ int main(int argc, char *argv[])
apicmd_resm(argc, argv);
} else if (strcmp(cmd_name, "tst_exit") == 0) {
tst_exit();
- } else if (strcmp(cmd_name, "tst_flush") == 0) {
- tst_flush();
} else if (strcmp(cmd_name, "tst_ncpus") == 0) {
printf("%li\n", tst_ncpus());
} else if (strcmp(cmd_name, "tst_ncpus_conf") == 0) {
--
2.13.6
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [LTP] [PATCH v3 2/3] Add new tst_flush() library function
2018-03-09 14:16 [LTP] [PATCH v3 1/3] remove old tst_flush() Michael Moese
@ 2018-03-09 14:16 ` Michael Moese
2018-03-13 10:12 ` Cyril Hrubis
2018-03-09 14:16 ` [LTP] [PATCH v3 3/3] safe_fork() should use tst_flush() instead of fflush() Michael Moese
2018-03-13 10:11 ` [LTP] [PATCH v3 1/3] remove old tst_flush() Cyril Hrubis
2 siblings, 1 reply; 6+ messages in thread
From: Michael Moese @ 2018-03-09 14:16 UTC (permalink / raw)
To: ltp
Add a new library function to flush stderr and stdout streams:
void tst_flush(void)
This function flushes stderr and stdout streams. In case of an
error, the test is aborted with TBROK and an error message is
printed.
Signed-off-by: Michael Moese <mmoese@suse.de>
---
doc/test-writing-guidelines.txt | 8 ++++++++
include/tst_test.h | 3 +++
lib/tst_test.c | 15 +++++++++++++++
3 files changed, 26 insertions(+)
diff --git a/doc/test-writing-guidelines.txt b/doc/test-writing-guidelines.txt
index 739b295b8..4c60cd66b 100644
--- a/doc/test-writing-guidelines.txt
+++ b/doc/test-writing-guidelines.txt
@@ -465,6 +465,14 @@ Allows for setting timeout per test iteration dymanically in the test setup(),
the timeout is specified in seconds. There are a few testcases whose runtime
can vary arbitrarily, these can disable timeouts by setting it to -1.
+[source,c]
+------------------------------------------------------------------------------
+void tst_flush(void);
+-------------------------------------------------------------------------------
+
+Flush stderr and stdout streams, handling errors appropriately.
+You should not use fflush() for stderr or stdout.
+
2.2.3 Test temporary directory
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
diff --git a/include/tst_test.h b/include/tst_test.h
index af97b8983..54ff306d9 100644
--- a/include/tst_test.h
+++ b/include/tst_test.h
@@ -70,6 +70,9 @@ void tst_brk_(const char *file, const int lineno, int ttype,
#define tst_brk(ttype, arg_fmt, ...) \
tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__)
+/* flush stderr and stdout */
+void tst_flush(void);
+
pid_t safe_fork(const char *filename, unsigned int lineno);
#define SAFE_FORK() \
safe_fork(__FILE__, __LINE__)
diff --git a/lib/tst_test.c b/lib/tst_test.c
index 2cf35ed66..9b4f43828 100644
--- a/lib/tst_test.c
+++ b/lib/tst_test.c
@@ -1085,3 +1085,18 @@ void tst_run_tcases(int argc, char *argv[], struct tst_test *self)
do_exit(ret);
}
+
+
+void tst_flush(void)
+{
+ int rval;
+
+ rval = fflush(stderr);
+ if (rval != 0)
+ tst_brk(TBROK | TERRNO, "fflush(stderr) failed");
+
+ rval = fflush(stderr);
+ if (rval != 0)
+ tst_brk(TBROK | TERRNO, "fflush(stdout) failed");
+
+}
--
2.13.6
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [LTP] [PATCH v3 3/3] safe_fork() should use tst_flush() instead of fflush()
2018-03-09 14:16 [LTP] [PATCH v3 1/3] remove old tst_flush() Michael Moese
2018-03-09 14:16 ` [LTP] [PATCH v3 2/3] Add new tst_flush() library function Michael Moese
@ 2018-03-09 14:16 ` Michael Moese
2018-03-13 10:12 ` Cyril Hrubis
2018-03-13 10:11 ` [LTP] [PATCH v3 1/3] remove old tst_flush() Cyril Hrubis
2 siblings, 1 reply; 6+ messages in thread
From: Michael Moese @ 2018-03-09 14:16 UTC (permalink / raw)
To: ltp
safe_fork() should not use fflush(stdout). This commit makes use of
the the newly added tst_flush(), which flushes, in fact, both stdout
and stderr.
In addition, tests output their results to stderr, so flushing stdout
here is simply wrong.
Signed-off-by: Michael Moese <mmoese@suse.de>
---
lib/tst_test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/tst_test.c b/lib/tst_test.c
index 9b4f43828..7069bbc5e 100644
--- a/lib/tst_test.c
+++ b/lib/tst_test.c
@@ -380,7 +380,7 @@ pid_t safe_fork(const char *filename, unsigned int lineno)
if (!tst_test->forks_child)
tst_brk(TBROK, "test.forks_child must be set!");
- fflush(stdout);
+ tst_flush();
pid = fork();
if (pid < 0)
--
2.13.6
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [LTP] [PATCH v3 1/3] remove old tst_flush()
2018-03-09 14:16 [LTP] [PATCH v3 1/3] remove old tst_flush() Michael Moese
2018-03-09 14:16 ` [LTP] [PATCH v3 2/3] Add new tst_flush() library function Michael Moese
2018-03-09 14:16 ` [LTP] [PATCH v3 3/3] safe_fork() should use tst_flush() instead of fflush() Michael Moese
@ 2018-03-13 10:11 ` Cyril Hrubis
2 siblings, 0 replies; 6+ messages in thread
From: Cyril Hrubis @ 2018-03-13 10:11 UTC (permalink / raw)
To: ltp
Hi!
I've amended the patch with a change that removes the tst_flush from the
.gitingore as well and pushe, thanks.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 6+ messages in thread
* [LTP] [PATCH v3 2/3] Add new tst_flush() library function
2018-03-09 14:16 ` [LTP] [PATCH v3 2/3] Add new tst_flush() library function Michael Moese
@ 2018-03-13 10:12 ` Cyril Hrubis
0 siblings, 0 replies; 6+ messages in thread
From: Cyril Hrubis @ 2018-03-13 10:12 UTC (permalink / raw)
To: ltp
Hi!
I've changed the documentation to explain better why this call is needed
and pushed, thanks.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 6+ messages in thread
* [LTP] [PATCH v3 3/3] safe_fork() should use tst_flush() instead of fflush()
2018-03-09 14:16 ` [LTP] [PATCH v3 3/3] safe_fork() should use tst_flush() instead of fflush() Michael Moese
@ 2018-03-13 10:12 ` Cyril Hrubis
0 siblings, 0 replies; 6+ messages in thread
From: Cyril Hrubis @ 2018-03-13 10:12 UTC (permalink / raw)
To: ltp
Hi!
Pushed, thanks.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-03-13 10:12 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-09 14:16 [LTP] [PATCH v3 1/3] remove old tst_flush() Michael Moese
2018-03-09 14:16 ` [LTP] [PATCH v3 2/3] Add new tst_flush() library function Michael Moese
2018-03-13 10:12 ` Cyril Hrubis
2018-03-09 14:16 ` [LTP] [PATCH v3 3/3] safe_fork() should use tst_flush() instead of fflush() Michael Moese
2018-03-13 10:12 ` Cyril Hrubis
2018-03-13 10:11 ` [LTP] [PATCH v3 1/3] remove old tst_flush() Cyril Hrubis
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.