mir/criu
2
0
Fork 0
mirror of https://github.com/checkpoint-restore/criu synced 2025-08-22 09:58:09 +00:00
Code Issues Releases Wiki Activity

zdtm: fifo-rowo-pair test update

Browse Source 
Address Andrew's comments

 - use TEST_VALUE macro instead of open coded value
 - use WIFx macros for child status testing
 - kill child/parent on error

Note that it was not clear from an initiall patch
change log -- the difference between fifo_wronly
test case and this one is that we use fork() call
here which allow us to make sure that fifo restoration
code logic will be tested more widely.

Basically the flowchart of the test case:

parent				child
------				-----
open RW fifo (slave)
open RO fifo (master)
				open WO fifo (master)
				dup parent slave to slave1

				... waiting c/r to happen ...
				... waking up from c/r ...

				send bytes to master
				send bytes to slave1

				finish
... initiating c/r ...
... waking up from c/r ...

read bytes from master
read bytes from slave

PASS :D

Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Acked-by: Andrew Vagin <avagin@parallels.com>
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
This commit is contained in:
Cyrill Gorcunov 2012-07-01 22:38:00 +04:00 committed by Pavel Emelyanov
parent d249c293f7
commit 7ff9efda4a
1 changed files with 28 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
test/zdtm/live/static/fifo-rowo-pair.c
View File

@ -26,6 +26,14 @@ TEST_OPTION(name_master, string, "master fifo name", 1);
char *name_slave;
TEST_OPTION(name_slave, string, "slave fifo name", 1);
#define TEST_VALUE (00100)
#define exit_shot(pid, code) \
do { kill(pid, SIGKILL); exit(code); } while (0)
#define exit_shot_parent(code) \
exit_shot(getppid(), 1)
int main(int argc, char **argv)
{
task_waiter_t t;
@ -60,29 +68,29 @@ int main(int argc, char **argv)
fd_master = open(name_master, O_WRONLY);
if (fd_master < 0) {
err("can't open %s: %m\n", name_master);
exit(1);
exit_shot_parent(1);
}
new_slave = dup2(fd_slave, 64);
if (new_slave < 0) {
err("can't dup %s: %m\n", name_slave);
exit(1);
exit_shot_parent(1);
}
close(fd_slave);
task_waiter_complete_current(&t);
v = 00100;
v = TEST_VALUE;
if (write(new_slave, &v, sizeof(v)) != sizeof(v)) {
err("write failed: %m\n");
exit(1);
exit_shot_parent(1);
}
v = 00100;
v = TEST_VALUE;
if (write(fd_master, &v, sizeof(v)) != sizeof(v)) {
err("write failed: %m\n");
exit(1);
exit_shot_parent(1);
}
/* Don't exit until explicitly asked */
@ -97,7 +105,7 @@ int main(int argc, char **argv)
fd_master = open(name_master, O_RDONLY);
if (fd_master < 0) {
err("can't open %s: %m\n", name_master);
exit(1);
exit_shot(pid, 1);
}
/* Wait until data appear in kernel fifo buffer */
@ -108,38 +116,39 @@ int main(int argc, char **argv)
if (read(fd_master, &v, sizeof(v)) != sizeof(v)) {
err("read failed: %m\n");
exit(1);
exit_shot(pid, 1);
}
task_waiter_complete_current(&t);
if (v != 00100) {
if (v != TEST_VALUE) {
fail("read data mismatch\n");
exit(1);
exit_shot(pid, 1);
}
if (read(fd_slave, &v, sizeof(v)) != sizeof(v)) {
err("read failed: %m\n");
exit(1);
exit_shot(pid, 1);
}
if (v != 00100) {
if (v != TEST_VALUE) {
fail("read data mismatch\n");
exit(1);
exit_shot(pid, 1);
}
waitpid(pid, &status, P_ALL);
if (unlink(name_master) < 0) {
if (unlink(name_master) < 0)
err("can't unlink %s: %m", name_master);
exit(1);
}
if (unlink(name_slave) < 0) {
if (unlink(name_slave) < 0)
err("can't unlink %s: %m", name_slave);
exit(1);
if (!WIFEXITED(status)) {
err("child %d is still running\n", pid);
exit_shot(pid, 1);
}
errno = (status >> 8) & 0x7f;
errno = WEXITSTATUS(status);
if (errno) {
fail("Child exited with error %m");
exit(errno);