diff options
author | Xiaoguang Wang <wangxg.fnst@cn.fujitsu.com> | 2014-04-25 18:54:07 +0800 |
---|---|---|
committer | Wanlong Gao <gaowanlong@cn.fujitsu.com> | 2014-04-26 19:25:47 +0800 |
commit | e3c09dcd8a640a4a82aec6968395c05a8ec95067 (patch) | |
tree | 5454eedf0efbdf248117094302931311e36c7292 /scenario_groups | |
parent | f3e448f8bc5e50d3fcd75cf82042b9515cd4e651 (diff) |
pipeio/pipeio.c: cleanup and add it to run default
Delete some useless comments and fix some.
Use SAFE_* macros.
Make a re-arrangement on original code: let parent process
work in do_parent(), child process work in do_child(), and
arguments parse and initialization work in setup().
Delete several useless or pointless options:
Option C: this option is not supported and has no effect to this test.
Option B: if this option is specified, the parent process will always
return 0, which won't reflect the correct test results. I think it does
not make much sense and remove it.
Option d: this option is used to specify a directory, where named pipe
is created, other than the current directory. Since tst_tmpdir() will
create a temporary directory, so I think "-d" option can be removed.
printf() is not async-signal-safe, so use write(2) in signal handler directly.
Some cleanup.
Signed-off-by: Xiaoguang Wang <wangxg.fnst@cn.fujitsu.com>
Acked-by: Jan Stancek <jstancek@redhat.com>
Signed-off-by: Wanlong Gao <gaowanlong@cn.fujitsu.com>
Diffstat (limited to 'scenario_groups')
-rw-r--r-- | scenario_groups/default | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/scenario_groups/default b/scenario_groups/default index 196e63671..bf7ab79ea 100644 --- a/scenario_groups/default +++ b/scenario_groups/default @@ -27,3 +27,4 @@ hyperthreading kernel_misc modules fs_ext4 +pipes |