* [LTP] [PATCH 1/2] epoll_pwait/epoll_pwait01.c: add new testcase
@ 2016-01-20 11:09 Guangwen Feng
2016-01-20 11:10 ` [LTP] [PATCH 2/2] epoll_pwait/epoll_pwait02.c: " Guangwen Feng
0 siblings, 1 reply; 8+ messages in thread
From: Guangwen Feng @ 2016-01-20 11:09 UTC (permalink / raw)
To: ltp
Check that epoll_pwait(2) with sigmask argument allows the caller
to safely wait until either a file descriptor becomes ready or
the timeout expires.
Signed-off-by: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
---
configure.ac | 1 +
m4/ltp-epoll_pwait.m4 | 25 +++
runtest/ltplite | 2 +
runtest/stress.part3 | 2 +
runtest/syscalls | 1 +
testcases/kernel/syscalls/.gitignore | 1 +
testcases/kernel/syscalls/epoll_pwait/Makefile | 23 +++
.../kernel/syscalls/epoll_pwait/epoll_pwait.h | 35 +++++
.../kernel/syscalls/epoll_pwait/epoll_pwait01.c | 174 +++++++++++++++++++++
9 files changed, 264 insertions(+)
create mode 100644 m4/ltp-epoll_pwait.m4
create mode 100644 testcases/kernel/syscalls/epoll_pwait/Makefile
create mode 100644 testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
create mode 100644 testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
diff --git a/configure.ac b/configure.ac
index cc50397..b065fe6 100644
--- a/configure.ac
+++ b/configure.ac
@@ -183,5 +183,6 @@ LTP_CHECK_KCMP
LTP_CHECK_KCMP_TYPE
LTP_CHECK_PREADV
LTP_CHECK_PWRITEV
+LTP_CHECK_EPOLL_PWAIT
AC_OUTPUT
diff --git a/m4/ltp-epoll_pwait.m4 b/m4/ltp-epoll_pwait.m4
new file mode 100644
index 0000000..e918110
--- /dev/null
+++ b/m4/ltp-epoll_pwait.m4
@@ -0,0 +1,25 @@
+dnl
+dnl Copyright (c) 2016 Fujitsu Ltd.
+dnl Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+dnl
+dnl This program is free software; you can redistribute it and/or modify
+dnl it under the terms of the GNU General Public License as published by
+dnl the Free Software Foundation; either version 2 of the License, or
+dnl (at your option) any later version.
+dnl
+dnl This program is distributed in the hope that it will be useful,
+dnl but WITHOUT ANY WARRANTY; without even the implied warranty of
+dnl MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+dnl the GNU General Public License for more details.
+dnl
+dnl You should have received a copy of the GNU General Public License
+dnl along with this program.
+dnl
+
+dnl
+dnl LTP_CHECK_EPOLL_PWAIT
+dnl ----------------------------
+dnl
+AC_DEFUN([LTP_CHECK_EPOLL_PWAIT],[
+AC_CHECK_FUNCS(epoll_pwait,,)
+])
diff --git a/runtest/ltplite b/runtest/ltplite
index 1aa93e6..ac876a4 100644
--- a/runtest/ltplite
+++ b/runtest/ltplite
@@ -160,6 +160,8 @@ dup203 dup203
dup204 dup204
dup205 dup205
+epoll_pwait01 epoll_pwait01
+
execl01 execl01
execle01 execle01
execlp01 execlp01
diff --git a/runtest/stress.part3 b/runtest/stress.part3
index 87f42ea..c76e025 100644
--- a/runtest/stress.part3
+++ b/runtest/stress.part3
@@ -101,6 +101,8 @@ dup203 dup203
dup204 dup204
dup205 dup205
+epoll_pwait01 epoll_pwait01
+
execl01 execl01
execle01 execle01
execlp01 execlp01
diff --git a/runtest/syscalls b/runtest/syscalls
index 7173f22..26cb48f 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -123,6 +123,7 @@ dup3_02 dup3_02
epoll_create1_01 epoll_create1_01
epoll01 epoll-ltp
+epoll_pwait01 epoll_pwait01
eventfd01 eventfd01
diff --git a/testcases/kernel/syscalls/.gitignore b/testcases/kernel/syscalls/.gitignore
index d5f21ef..c9d8aaa 100644
--- a/testcases/kernel/syscalls/.gitignore
+++ b/testcases/kernel/syscalls/.gitignore
@@ -98,6 +98,7 @@
/dup3/dup3_02
/epoll/epoll-ltp
/epoll_create1/epoll_create1_01
+/epoll_pwait/epoll_pwait01
/eventfd/eventfd01
/eventfd2/eventfd2_01
/eventfd2/eventfd2_02
diff --git a/testcases/kernel/syscalls/epoll_pwait/Makefile b/testcases/kernel/syscalls/epoll_pwait/Makefile
new file mode 100644
index 0000000..289aaaf
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/Makefile
@@ -0,0 +1,23 @@
+#
+# Copyright (c) 2016 Fujitsu Ltd.
+# Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+# the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.
+#
+
+top_srcdir ?= ../../../..
+
+include $(top_srcdir)/include/mk/testcases.mk
+
+include $(top_srcdir)/include/mk/generic_leaf_target.mk
diff --git a/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
new file mode 100644
index 0000000..694cd46
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
@@ -0,0 +1,35 @@
+/*
+ * Copyright (c) 2016 Fujitsu Ltd.
+ * Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.
+ */
+
+#ifndef EPOLL_PWAIT_H
+#define EPOLL_PWAIT_H
+
+#include <sys/types.h>
+#include "config.h"
+#include "linux_syscall_numbers.h"
+
+#if !defined(HAVE_EPOLL_PWAIT)
+int epoll_pwait(int epfd, struct epoll_event *events, int maxevents,
+ int timeout, const sigset_t *sigmask)
+{
+ return ltp_syscall(__NR_epoll_pwait, epfd, events, maxevents,
+ timeout, sigmask);
+}
+#endif
+
+#endif /* EPOLL_PWAIT_H */
diff --git a/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
new file mode 100644
index 0000000..e3d5bfb
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
@@ -0,0 +1,174 @@
+/*
+ * Copyright (c) 2016 Fujitsu Ltd.
+ * Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.
+ */
+
+/*
+ * Description:
+ * Basic test for epoll_pwait(2).
+ * epoll_pwait(2) with sigmask argument allows the caller to
+ * safely wait until either a file descriptor becomes ready
+ * or the timeout expires.
+ */
+
+#include <sys/epoll.h>
+#include <sys/types.h>
+#include <unistd.h>
+#include <string.h>
+#include <errno.h>
+
+#include "test.h"
+#include "epoll_pwait.h"
+#include "safe_macros.h"
+
+char *TCID = "epoll_pwait01";
+int TST_TOTAL = 1;
+
+static int epfd, fds[2];
+static const char buf[] = "Testing";
+static sigset_t sigset;
+static struct epoll_event epevs;
+static struct sigaction sa;
+
+static void setup(void);
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED);
+static void do_test(void);
+static void do_child(void);
+static void cleanup(void);
+
+int main(int ac, char **av)
+{
+ int lc;
+
+ tst_parse_opts(ac, av, NULL, NULL);
+
+ setup();
+
+ for (lc = 0; TEST_LOOPING(lc); lc++) {
+ tst_count = 0;
+
+ do_test();
+ }
+
+ cleanup();
+ tst_exit();
+}
+
+static void setup(void)
+{
+ if ((tst_kvercmp(2, 6, 19)) < 0) {
+ tst_brkm(TCONF, NULL, "This test can only run on kernels "
+ "that are 2.6.19 or higher");
+ }
+
+ tst_sig(FORK, DEF_HANDLER, cleanup);
+
+ TEST_PAUSE;
+
+ if (sigemptyset(&sigset) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
+
+ if (sigaddset(&sigset, SIGUSR1) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigaddset() failed");
+
+ sa.sa_flags = 0;
+ sa.sa_handler = sighandler;
+ if (sigemptyset(&sa.sa_mask) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
+
+ if (sigaction(SIGUSR1, &sa, NULL) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigaction() failed");
+
+ SAFE_PIPE(NULL, fds);
+
+ epfd = epoll_create(1);
+ if (epfd == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to create epoll instance");
+ }
+
+ epevs.events = EPOLLIN;
+ epevs.data.fd = fds[0];
+
+ if (epoll_ctl(epfd, EPOLL_CTL_ADD, fds[0], &epevs) == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to register epoll target");
+ }
+}
+
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED)
+{
+
+}
+
+static void do_test(void)
+{
+ char read_buf[sizeof(buf)];
+ pid_t cpid;
+
+ cpid = tst_fork();
+ if (cpid < 0)
+ tst_brkm(TBROK | TERRNO, cleanup, "fork() failed");
+
+ if (cpid == 0)
+ do_child();
+
+ TEST(epoll_pwait(epfd, &epevs, 1, -1, &sigset));
+
+ if (TEST_RETURN == -1) {
+ tst_resm(TFAIL | TTERRNO, "epoll_pwait() failed");
+ goto end;
+ }
+
+ if (TEST_RETURN != 1) {
+ tst_resm(TFAIL, "epoll_pwait() failed to get one fd ready");
+ goto end;
+ }
+
+ tst_resm(TPASS, "epoll_pwait() get one fd ready");
+
+end:
+ tst_record_childstatus(cleanup, cpid);
+
+ SAFE_READ(cleanup, 1, fds[0], read_buf, sizeof(buf));
+}
+
+static void do_child(void)
+{
+ if (tst_process_state_wait2(getppid(), 'S') != 0) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to wait for parent process's state");
+ }
+
+ SAFE_KILL(cleanup, getppid(), SIGUSR1);
+
+ SAFE_WRITE(cleanup, 1, fds[1], buf, sizeof(buf));
+
+ cleanup();
+ tst_exit();
+}
+
+static void cleanup(void)
+{
+ if (epfd > 0 && close(epfd))
+ tst_resm(TWARN | TERRNO, "failed to close epfd");
+
+ if (close(fds[0]))
+ tst_resm(TWARN | TERRNO, "close(fds[0]) failed");
+
+ if (close(fds[1]))
+ tst_resm(TWARN | TERRNO, "close(fds[1]) failed");
+}
--
1.8.4.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [LTP] [PATCH 2/2] epoll_pwait/epoll_pwait02.c: add new testcase
2016-01-20 11:09 [LTP] [PATCH 1/2] epoll_pwait/epoll_pwait01.c: add new testcase Guangwen Feng
@ 2016-01-20 11:10 ` Guangwen Feng
2016-02-09 14:02 ` Cyril Hrubis
0 siblings, 1 reply; 8+ messages in thread
From: Guangwen Feng @ 2016-01-20 11:10 UTC (permalink / raw)
To: ltp
Verify that:
epoll_pwait(2) with NULL sigmask argument fails
if interrupted by a signal handler.
Expected result:
epoll_pwait(2) should return -1 and set errno to EINTR.
Signed-off-by: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
---
runtest/ltplite | 1 +
runtest/stress.part3 | 1 +
runtest/syscalls | 1 +
testcases/kernel/syscalls/.gitignore | 1 +
.../kernel/syscalls/epoll_pwait/epoll_pwait02.c | 161 +++++++++++++++++++++
5 files changed, 165 insertions(+)
create mode 100644 testcases/kernel/syscalls/epoll_pwait/epoll_pwait02.c
diff --git a/runtest/ltplite b/runtest/ltplite
index ac876a4..3daaac4 100644
--- a/runtest/ltplite
+++ b/runtest/ltplite
@@ -161,6 +161,7 @@ dup204 dup204
dup205 dup205
epoll_pwait01 epoll_pwait01
+epoll_pwait02 epoll_pwait02
execl01 execl01
execle01 execle01
diff --git a/runtest/stress.part3 b/runtest/stress.part3
index c76e025..47809ea 100644
--- a/runtest/stress.part3
+++ b/runtest/stress.part3
@@ -102,6 +102,7 @@ dup204 dup204
dup205 dup205
epoll_pwait01 epoll_pwait01
+epoll_pwait02 epoll_pwait02
execl01 execl01
execle01 execle01
diff --git a/runtest/syscalls b/runtest/syscalls
index 26cb48f..e20884a 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -124,6 +124,7 @@ dup3_02 dup3_02
epoll_create1_01 epoll_create1_01
epoll01 epoll-ltp
epoll_pwait01 epoll_pwait01
+epoll_pwait02 epoll_pwait02
eventfd01 eventfd01
diff --git a/testcases/kernel/syscalls/.gitignore b/testcases/kernel/syscalls/.gitignore
index c9d8aaa..4dd1103 100644
--- a/testcases/kernel/syscalls/.gitignore
+++ b/testcases/kernel/syscalls/.gitignore
@@ -99,6 +99,7 @@
/epoll/epoll-ltp
/epoll_create1/epoll_create1_01
/epoll_pwait/epoll_pwait01
+/epoll_pwait/epoll_pwait02
/eventfd/eventfd01
/eventfd2/eventfd2_01
/eventfd2/eventfd2_02
diff --git a/testcases/kernel/syscalls/epoll_pwait/epoll_pwait02.c b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait02.c
new file mode 100644
index 0000000..315e9c8
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait02.c
@@ -0,0 +1,161 @@
+/*
+ * Copyright (c) 2016 Fujitsu Ltd.
+ * Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.
+ */
+
+/*
+ * Description:
+ * epoll_pwait(2) with NULL sigmask argument fails
+ * if interrupted by a signal handler.
+ *
+ * Expected Result:
+ * epoll_pwait(2) should return -1 and set errno to EINTR.
+ */
+
+#include <sys/epoll.h>
+#include <sys/types.h>
+#include <unistd.h>
+#include <string.h>
+#include <errno.h>
+
+#include "test.h"
+#include "epoll_pwait.h"
+#include "safe_macros.h"
+
+char *TCID = "epoll_pwait02";
+int TST_TOTAL = 1;
+
+static int epfd, fds[2];
+static struct epoll_event epevs;
+static struct sigaction sa;
+
+static void setup(void);
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED);
+static void do_test(void);
+static void do_child(void);
+static void cleanup(void);
+
+int main(int ac, char **av)
+{
+ int lc;
+
+ tst_parse_opts(ac, av, NULL, NULL);
+
+ setup();
+
+ for (lc = 0; TEST_LOOPING(lc); lc++) {
+ tst_count = 0;
+
+ do_test();
+ }
+
+ cleanup();
+ tst_exit();
+}
+
+static void setup(void)
+{
+ if ((tst_kvercmp(2, 6, 19)) < 0) {
+ tst_brkm(TCONF, NULL, "This test can only run on kernels "
+ "that are 2.6.19 or higher");
+ }
+
+ tst_sig(FORK, DEF_HANDLER, cleanup);
+
+ TEST_PAUSE;
+
+ sa.sa_flags = 0;
+ sa.sa_handler = sighandler;
+ if (sigemptyset(&sa.sa_mask) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
+
+ if (sigaction(SIGUSR1, &sa, NULL) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigaction() failed");
+
+ SAFE_PIPE(NULL, fds);
+
+ epfd = epoll_create(1);
+ if (epfd == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to create epoll instance");
+ }
+
+ epevs.events = EPOLLIN;
+ epevs.data.fd = fds[0];
+
+ if (epoll_ctl(epfd, EPOLL_CTL_ADD, fds[0], &epevs) == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to register epoll target");
+ }
+}
+
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED)
+{
+
+}
+
+static void do_test(void)
+{
+ pid_t cpid;
+
+ cpid = tst_fork();
+ if (cpid < 0)
+ tst_brkm(TBROK | TERRNO, cleanup, "fork() failed");
+
+ if (cpid == 0)
+ do_child();
+
+ TEST(epoll_wait(epfd, &epevs, 1, 1000));
+ if (TEST_RETURN != -1) {
+ tst_resm(TFAIL, "epoll_wait() succeed unexpectedly");
+ } else {
+ if (TEST_ERRNO == EINTR) {
+ tst_resm(TPASS | TTERRNO,
+ "epoll_wait() fails as expected");
+ } else {
+ tst_resm(TFAIL | TTERRNO,
+ "epoll_wait() fails unexpectedly, "
+ "expected errno is EINTR");
+ }
+ }
+
+ tst_record_childstatus(cleanup, cpid);
+}
+
+static void do_child(void)
+{
+ if (tst_process_state_wait2(getppid(), 'S') != 0) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to wait for parent process's state");
+ }
+
+ SAFE_KILL(cleanup, getppid(), SIGUSR1);
+
+ cleanup();
+ tst_exit();
+}
+
+static void cleanup(void)
+{
+ if (epfd > 0 && close(epfd))
+ tst_resm(TWARN | TERRNO, "failed to close epfd");
+
+ if (close(fds[0]))
+ tst_resm(TWARN | TERRNO, "close(fds[0]) failed");
+
+ if (close(fds[1]))
+ tst_resm(TWARN | TERRNO, "close(fds[1]) failed");
+}
--
1.8.4.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [LTP] [PATCH 2/2] epoll_pwait/epoll_pwait02.c: add new testcase
2016-01-20 11:10 ` [LTP] [PATCH 2/2] epoll_pwait/epoll_pwait02.c: " Guangwen Feng
@ 2016-02-09 14:02 ` Cyril Hrubis
2016-02-19 8:33 ` [LTP] [PATCH v2] epoll_pwait/epoll_pwait01.c: " Guangwen Feng
0 siblings, 1 reply; 8+ messages in thread
From: Cyril Hrubis @ 2016-02-09 14:02 UTC (permalink / raw)
To: ltp
Hi!
> Verify that:
> epoll_pwait(2) with NULL sigmask argument fails
> if interrupted by a signal handler.
These two testcases are nearly identical, the difference is sinal mask
for epoll_pwait() and function that child executes, right?. Why can't we
implement both of them in one test?
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 8+ messages in thread
* [LTP] [PATCH v2] epoll_pwait/epoll_pwait01.c: add new testcase
2016-02-09 14:02 ` Cyril Hrubis
@ 2016-02-19 8:33 ` Guangwen Feng
2016-03-02 15:32 ` Cyril Hrubis
0 siblings, 1 reply; 8+ messages in thread
From: Guangwen Feng @ 2016-02-19 8:33 UTC (permalink / raw)
To: ltp
Check that:
1) epoll_pwait(2) with sigmask argument allows the caller
to safely wait until either a file descriptor becomes
ready or the timeout expires.
2) epoll_pwait(2) with NULL sigmask argument fails if
interrupted by a signal handler, epoll_pwait(2) should
return -1 and set errno to EINTR.
Signed-off-by: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
---
configure.ac | 1 +
m4/ltp-epoll_pwait.m4 | 25 +++
runtest/ltplite | 2 +
runtest/stress.part3 | 2 +
runtest/syscalls | 1 +
testcases/kernel/syscalls/.gitignore | 1 +
testcases/kernel/syscalls/epoll_pwait/Makefile | 23 +++
.../kernel/syscalls/epoll_pwait/epoll_pwait.h | 35 ++++
.../kernel/syscalls/epoll_pwait/epoll_pwait01.c | 191 +++++++++++++++++++++
9 files changed, 281 insertions(+)
create mode 100644 m4/ltp-epoll_pwait.m4
create mode 100644 testcases/kernel/syscalls/epoll_pwait/Makefile
create mode 100644 testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
create mode 100644 testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
diff --git a/configure.ac b/configure.ac
index cc50397..b065fe6 100644
--- a/configure.ac
+++ b/configure.ac
@@ -183,5 +183,6 @@ LTP_CHECK_KCMP
LTP_CHECK_KCMP_TYPE
LTP_CHECK_PREADV
LTP_CHECK_PWRITEV
+LTP_CHECK_EPOLL_PWAIT
AC_OUTPUT
diff --git a/m4/ltp-epoll_pwait.m4 b/m4/ltp-epoll_pwait.m4
new file mode 100644
index 0000000..e918110
--- /dev/null
+++ b/m4/ltp-epoll_pwait.m4
@@ -0,0 +1,25 @@
+dnl
+dnl Copyright (c) 2016 Fujitsu Ltd.
+dnl Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+dnl
+dnl This program is free software; you can redistribute it and/or modify
+dnl it under the terms of the GNU General Public License as published by
+dnl the Free Software Foundation; either version 2 of the License, or
+dnl (at your option) any later version.
+dnl
+dnl This program is distributed in the hope that it will be useful,
+dnl but WITHOUT ANY WARRANTY; without even the implied warranty of
+dnl MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+dnl the GNU General Public License for more details.
+dnl
+dnl You should have received a copy of the GNU General Public License
+dnl along with this program.
+dnl
+
+dnl
+dnl LTP_CHECK_EPOLL_PWAIT
+dnl ----------------------------
+dnl
+AC_DEFUN([LTP_CHECK_EPOLL_PWAIT],[
+AC_CHECK_FUNCS(epoll_pwait,,)
+])
diff --git a/runtest/ltplite b/runtest/ltplite
index 89accc4..fd496cd 100644
--- a/runtest/ltplite
+++ b/runtest/ltplite
@@ -159,6 +159,8 @@ dup203 dup203
dup204 dup204
dup205 dup205
+epoll_pwait01 epoll_pwait01
+
execl01 execl01
execle01 execle01
execlp01 execlp01
diff --git a/runtest/stress.part3 b/runtest/stress.part3
index 85950bf..f371f9f 100644
--- a/runtest/stress.part3
+++ b/runtest/stress.part3
@@ -100,6 +100,8 @@ dup203 dup203
dup204 dup204
dup205 dup205
+epoll_pwait01 epoll_pwait01
+
execl01 execl01
execle01 execle01
execlp01 execlp01
diff --git a/runtest/syscalls b/runtest/syscalls
index 5f169a3..23ace15 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -122,6 +122,7 @@ dup3_02 dup3_02
epoll_create1_01 epoll_create1_01
epoll01 epoll-ltp
+epoll_pwait01 epoll_pwait01
eventfd01 eventfd01
diff --git a/testcases/kernel/syscalls/.gitignore b/testcases/kernel/syscalls/.gitignore
index de403e4..d9f2889 100644
--- a/testcases/kernel/syscalls/.gitignore
+++ b/testcases/kernel/syscalls/.gitignore
@@ -97,6 +97,7 @@
/dup3/dup3_02
/epoll/epoll-ltp
/epoll_create1/epoll_create1_01
+/epoll_pwait/epoll_pwait01
/eventfd/eventfd01
/eventfd2/eventfd2_01
/eventfd2/eventfd2_02
diff --git a/testcases/kernel/syscalls/epoll_pwait/Makefile b/testcases/kernel/syscalls/epoll_pwait/Makefile
new file mode 100644
index 0000000..289aaaf
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/Makefile
@@ -0,0 +1,23 @@
+#
+# Copyright (c) 2016 Fujitsu Ltd.
+# Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+# the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.
+#
+
+top_srcdir ?= ../../../..
+
+include $(top_srcdir)/include/mk/testcases.mk
+
+include $(top_srcdir)/include/mk/generic_leaf_target.mk
diff --git a/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
new file mode 100644
index 0000000..694cd46
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
@@ -0,0 +1,35 @@
+/*
+ * Copyright (c) 2016 Fujitsu Ltd.
+ * Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.
+ */
+
+#ifndef EPOLL_PWAIT_H
+#define EPOLL_PWAIT_H
+
+#include <sys/types.h>
+#include "config.h"
+#include "linux_syscall_numbers.h"
+
+#if !defined(HAVE_EPOLL_PWAIT)
+int epoll_pwait(int epfd, struct epoll_event *events, int maxevents,
+ int timeout, const sigset_t *sigmask)
+{
+ return ltp_syscall(__NR_epoll_pwait, epfd, events, maxevents,
+ timeout, sigmask);
+}
+#endif
+
+#endif /* EPOLL_PWAIT_H */
diff --git a/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
new file mode 100644
index 0000000..6a290dd
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
@@ -0,0 +1,191 @@
+/*
+ * Copyright (c) 2016 Fujitsu Ltd.
+ * Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.
+ */
+
+/*
+ * Description:
+ * Basic test for epoll_pwait(2).
+ * 1) epoll_pwait(2) with sigmask argument allows the caller to
+ * safely wait until either a file descriptor becomes ready
+ * or the timeout expires.
+ * 2) epoll_pwait(2) with NULL sigmask argument fails if
+ * interrupted by a signal handler, epoll_pwait(2) should
+ * return -1 and set errno to EINTR.
+ */
+
+#include <sys/epoll.h>
+#include <sys/types.h>
+#include <unistd.h>
+#include <string.h>
+#include <errno.h>
+
+#include "test.h"
+#include "epoll_pwait.h"
+#include "safe_macros.h"
+
+char *TCID = "epoll_pwait01";
+int TST_TOTAL = 2;
+
+static int epfd, fds[2];
+static const char buf[] = "Testing";
+static sigset_t sigset;
+static struct epoll_event epevs;
+static struct sigaction sa;
+
+static void setup(void);
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED);
+static void do_test(sigset_t *);
+static void do_child(void);
+static void cleanup(void);
+
+int main(int ac, char **av)
+{
+ int lc;
+
+ tst_parse_opts(ac, av, NULL, NULL);
+
+ setup();
+
+ for (lc = 0; TEST_LOOPING(lc); lc++) {
+ tst_count = 0;
+
+ do_test(&sigset);
+ do_test(NULL);
+ }
+
+ cleanup();
+ tst_exit();
+}
+
+static void setup(void)
+{
+ if ((tst_kvercmp(2, 6, 19)) < 0) {
+ tst_brkm(TCONF, NULL, "This test can only run on kernels "
+ "that are 2.6.19 or higher");
+ }
+
+ tst_sig(FORK, DEF_HANDLER, cleanup);
+
+ TEST_PAUSE;
+
+ if (sigemptyset(&sigset) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
+
+ if (sigaddset(&sigset, SIGUSR1) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigaddset() failed");
+
+ sa.sa_flags = 0;
+ sa.sa_handler = sighandler;
+ if (sigemptyset(&sa.sa_mask) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
+
+ if (sigaction(SIGUSR1, &sa, NULL) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigaction() failed");
+
+ SAFE_PIPE(NULL, fds);
+
+ epfd = epoll_create(1);
+ if (epfd == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to create epoll instance");
+ }
+
+ epevs.events = EPOLLIN;
+ epevs.data.fd = fds[0];
+
+ if (epoll_ctl(epfd, EPOLL_CTL_ADD, fds[0], &epevs) == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to register epoll target");
+ }
+}
+
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED)
+{
+
+}
+
+static void do_test(sigset_t *sigmask)
+{
+ char read_buf[sizeof(buf)];
+ pid_t cpid;
+
+ cpid = tst_fork();
+ if (cpid < 0)
+ tst_brkm(TBROK | TERRNO, cleanup, "fork() failed");
+
+ if (cpid == 0)
+ do_child();
+
+ TEST(epoll_pwait(epfd, &epevs, 1, -1, sigmask));
+
+ if (sigmask != NULL) {
+ if (TEST_RETURN == -1) {
+ tst_resm(TFAIL | TTERRNO, "epoll_pwait() failed");
+ } else if (TEST_RETURN != 1) {
+ tst_resm(TFAIL, "epoll_pwait() returned %li, "
+ "expected 1", TEST_RETURN);
+ } else {
+ tst_resm(TPASS, "epoll_pwait(sigmask) blocked signal");
+ }
+ } else {
+ if (TEST_RETURN != -1) {
+ tst_resm(TFAIL, "epoll_wait() succeeded unexpectedly");
+ } else {
+ if (TEST_ERRNO == EINTR) {
+ tst_resm(TPASS | TTERRNO,
+ "epoll_wait() failed as expected");
+ } else {
+ tst_resm(TFAIL | TTERRNO,
+ "epoll_wait() failed unexpectedly, "
+ "expected errno is EINTR");
+ }
+ }
+ }
+
+ tst_record_childstatus(cleanup, cpid);
+
+ SAFE_READ(cleanup, 1, fds[0], read_buf, sizeof(buf));
+}
+
+static void do_child(void)
+{
+ if (tst_process_state_wait2(getppid(), 'S') != 0) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to wait for parent process's state");
+ }
+
+ SAFE_KILL(cleanup, getppid(), SIGUSR1);
+
+ sleep(1);
+
+ SAFE_WRITE(cleanup, 1, fds[1], buf, sizeof(buf));
+
+ cleanup();
+ tst_exit();
+}
+
+static void cleanup(void)
+{
+ if (epfd > 0 && close(epfd))
+ tst_resm(TWARN | TERRNO, "failed to close epfd");
+
+ if (close(fds[0]))
+ tst_resm(TWARN | TERRNO, "close(fds[0]) failed");
+
+ if (close(fds[1]))
+ tst_resm(TWARN | TERRNO, "close(fds[1]) failed");
+}
--
1.8.4.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [LTP] [PATCH v2] epoll_pwait/epoll_pwait01.c: add new testcase
2016-02-19 8:33 ` [LTP] [PATCH v2] epoll_pwait/epoll_pwait01.c: " Guangwen Feng
@ 2016-03-02 15:32 ` Cyril Hrubis
2016-03-03 7:37 ` Guangwen Feng
0 siblings, 1 reply; 8+ messages in thread
From: Cyril Hrubis @ 2016-03-02 15:32 UTC (permalink / raw)
To: ltp
Hi!
> +/*
> + * Description:
> + * Basic test for epoll_pwait(2).
> + * 1) epoll_pwait(2) with sigmask argument allows the caller to
> + * safely wait until either a file descriptor becomes ready
> + * or the timeout expires.
> + * 2) epoll_pwait(2) with NULL sigmask argument fails if
> + * interrupted by a signal handler, epoll_pwait(2) should
> + * return -1 and set errno to EINTR.
> + */
> +
> +#include <sys/epoll.h>
> +#include <sys/types.h>
> +#include <unistd.h>
> +#include <string.h>
> +#include <errno.h>
> +
> +#include "test.h"
> +#include "epoll_pwait.h"
> +#include "safe_macros.h"
> +
> +char *TCID = "epoll_pwait01";
> +int TST_TOTAL = 2;
> +
> +static int epfd, fds[2];
> +static const char buf[] = "Testing";
> +static sigset_t sigset;
> +static struct epoll_event epevs;
> +static struct sigaction sa;
> +
> +static void setup(void);
> +static void sighandler(int sig LTP_ATTRIBUTE_UNUSED);
> +static void do_test(sigset_t *);
> +static void do_child(void);
> +static void cleanup(void);
> +
> +int main(int ac, char **av)
> +{
> + int lc;
> +
> + tst_parse_opts(ac, av, NULL, NULL);
> +
> + setup();
> +
> + for (lc = 0; TEST_LOOPING(lc); lc++) {
> + tst_count = 0;
> +
> + do_test(&sigset);
> + do_test(NULL);
> + }
> +
> + cleanup();
> + tst_exit();
> +}
> +
> +static void setup(void)
> +{
> + if ((tst_kvercmp(2, 6, 19)) < 0) {
> + tst_brkm(TCONF, NULL, "This test can only run on kernels "
> + "that are 2.6.19 or higher");
> + }
> +
> + tst_sig(FORK, DEF_HANDLER, cleanup);
> +
> + TEST_PAUSE;
> +
> + if (sigemptyset(&sigset) == -1)
> + tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
> +
> + if (sigaddset(&sigset, SIGUSR1) == -1)
> + tst_brkm(TFAIL | TERRNO, NULL, "sigaddset() failed");
> +
> + sa.sa_flags = 0;
> + sa.sa_handler = sighandler;
> + if (sigemptyset(&sa.sa_mask) == -1)
> + tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
> +
> + if (sigaction(SIGUSR1, &sa, NULL) == -1)
> + tst_brkm(TFAIL | TERRNO, NULL, "sigaction() failed");
> +
> + SAFE_PIPE(NULL, fds);
> +
> + epfd = epoll_create(1);
> + if (epfd == -1) {
> + tst_brkm(TBROK | TERRNO, cleanup,
> + "failed to create epoll instance");
> + }
> +
> + epevs.events = EPOLLIN;
> + epevs.data.fd = fds[0];
> +
> + if (epoll_ctl(epfd, EPOLL_CTL_ADD, fds[0], &epevs) == -1) {
> + tst_brkm(TBROK | TERRNO, cleanup,
> + "failed to register epoll target");
> + }
> +}
> +
> +static void sighandler(int sig LTP_ATTRIBUTE_UNUSED)
> +{
> +
> +}
> +
> +static void do_test(sigset_t *sigmask)
> +{
> + char read_buf[sizeof(buf)];
> + pid_t cpid;
> +
> + cpid = tst_fork();
> + if (cpid < 0)
> + tst_brkm(TBROK | TERRNO, cleanup, "fork() failed");
> +
> + if (cpid == 0)
> + do_child();
> +
> + TEST(epoll_pwait(epfd, &epevs, 1, -1, sigmask));
> +
> + if (sigmask != NULL) {
> + if (TEST_RETURN == -1) {
> + tst_resm(TFAIL | TTERRNO, "epoll_pwait() failed");
> + } else if (TEST_RETURN != 1) {
> + tst_resm(TFAIL, "epoll_pwait() returned %li, "
> + "expected 1", TEST_RETURN);
> + } else {
> + tst_resm(TPASS, "epoll_pwait(sigmask) blocked signal");
> + }
> + } else {
> + if (TEST_RETURN != -1) {
> + tst_resm(TFAIL, "epoll_wait() succeeded unexpectedly");
> + } else {
> + if (TEST_ERRNO == EINTR) {
> + tst_resm(TPASS | TTERRNO,
> + "epoll_wait() failed as expected");
> + } else {
> + tst_resm(TFAIL | TTERRNO,
> + "epoll_wait() failed unexpectedly, "
> + "expected errno is EINTR");
> + }
> + }
> + }
Can we put the checks in a separate functions and call them from here?
> + tst_record_childstatus(cleanup, cpid);
> +
> + SAFE_READ(cleanup, 1, fds[0], read_buf, sizeof(buf));
> +}
> +
> +static void do_child(void)
> +{
> + if (tst_process_state_wait2(getppid(), 'S') != 0) {
> + tst_brkm(TBROK | TERRNO, cleanup,
> + "failed to wait for parent process's state");
> + }
> +
> + SAFE_KILL(cleanup, getppid(), SIGUSR1);
> +
> + sleep(1);
One second is too much, 0.1 second should be more than enough for the
test.
> + SAFE_WRITE(cleanup, 1, fds[1], buf, sizeof(buf));
Technically we don't have to write anything here (and we could exit the
child without sleeping as well) if we passed a timeout to the
epoll_pwait() instead in the parent (we would have to check for timeout
instead of success but that shouldn't matter in this test).
> + cleanup();
> + tst_exit();
> +}
Otherwise it looks good.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 8+ messages in thread
* [LTP] [PATCH v2] epoll_pwait/epoll_pwait01.c: add new testcase
2016-03-02 15:32 ` Cyril Hrubis
@ 2016-03-03 7:37 ` Guangwen Feng
2016-03-03 8:24 ` [LTP] [PATCH v3] " Guangwen Feng
0 siblings, 1 reply; 8+ messages in thread
From: Guangwen Feng @ 2016-03-03 7:37 UTC (permalink / raw)
To: ltp
Hi!
Thanks for your review!
OK, I will modify the test case according to your suggestion.
Best Regards,
Guangwen Feng
On 03/02/2016 11:32 PM, Cyril Hrubis wrote:
> Hi!
>> +/*
>> + * Description:
>> + * Basic test for epoll_pwait(2).
>> + * 1) epoll_pwait(2) with sigmask argument allows the caller to
>> + * safely wait until either a file descriptor becomes ready
>> + * or the timeout expires.
>> + * 2) epoll_pwait(2) with NULL sigmask argument fails if
>> + * interrupted by a signal handler, epoll_pwait(2) should
>> + * return -1 and set errno to EINTR.
>> + */
>> +
>> +#include <sys/epoll.h>
>> +#include <sys/types.h>
>> +#include <unistd.h>
>> +#include <string.h>
>> +#include <errno.h>
>> +
>> +#include "test.h"
>> +#include "epoll_pwait.h"
>> +#include "safe_macros.h"
>> +
>> +char *TCID = "epoll_pwait01";
>> +int TST_TOTAL = 2;
>> +
>> +static int epfd, fds[2];
>> +static const char buf[] = "Testing";
>> +static sigset_t sigset;
>> +static struct epoll_event epevs;
>> +static struct sigaction sa;
>> +
>> +static void setup(void);
>> +static void sighandler(int sig LTP_ATTRIBUTE_UNUSED);
>> +static void do_test(sigset_t *);
>> +static void do_child(void);
>> +static void cleanup(void);
>> +
>> +int main(int ac, char **av)
>> +{
>> + int lc;
>> +
>> + tst_parse_opts(ac, av, NULL, NULL);
>> +
>> + setup();
>> +
>> + for (lc = 0; TEST_LOOPING(lc); lc++) {
>> + tst_count = 0;
>> +
>> + do_test(&sigset);
>> + do_test(NULL);
>> + }
>> +
>> + cleanup();
>> + tst_exit();
>> +}
>> +
>> +static void setup(void)
>> +{
>> + if ((tst_kvercmp(2, 6, 19)) < 0) {
>> + tst_brkm(TCONF, NULL, "This test can only run on kernels "
>> + "that are 2.6.19 or higher");
>> + }
>> +
>> + tst_sig(FORK, DEF_HANDLER, cleanup);
>> +
>> + TEST_PAUSE;
>> +
>> + if (sigemptyset(&sigset) == -1)
>> + tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
>> +
>> + if (sigaddset(&sigset, SIGUSR1) == -1)
>> + tst_brkm(TFAIL | TERRNO, NULL, "sigaddset() failed");
>> +
>> + sa.sa_flags = 0;
>> + sa.sa_handler = sighandler;
>> + if (sigemptyset(&sa.sa_mask) == -1)
>> + tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
>> +
>> + if (sigaction(SIGUSR1, &sa, NULL) == -1)
>> + tst_brkm(TFAIL | TERRNO, NULL, "sigaction() failed");
>> +
>> + SAFE_PIPE(NULL, fds);
>> +
>> + epfd = epoll_create(1);
>> + if (epfd == -1) {
>> + tst_brkm(TBROK | TERRNO, cleanup,
>> + "failed to create epoll instance");
>> + }
>> +
>> + epevs.events = EPOLLIN;
>> + epevs.data.fd = fds[0];
>> +
>> + if (epoll_ctl(epfd, EPOLL_CTL_ADD, fds[0], &epevs) == -1) {
>> + tst_brkm(TBROK | TERRNO, cleanup,
>> + "failed to register epoll target");
>> + }
>> +}
>> +
>> +static void sighandler(int sig LTP_ATTRIBUTE_UNUSED)
>> +{
>> +
>> +}
>> +
>> +static void do_test(sigset_t *sigmask)
>> +{
>> + char read_buf[sizeof(buf)];
>> + pid_t cpid;
>> +
>> + cpid = tst_fork();
>> + if (cpid < 0)
>> + tst_brkm(TBROK | TERRNO, cleanup, "fork() failed");
>> +
>> + if (cpid == 0)
>> + do_child();
>> +
>> + TEST(epoll_pwait(epfd, &epevs, 1, -1, sigmask));
>> +
>> + if (sigmask != NULL) {
>> + if (TEST_RETURN == -1) {
>> + tst_resm(TFAIL | TTERRNO, "epoll_pwait() failed");
>> + } else if (TEST_RETURN != 1) {
>> + tst_resm(TFAIL, "epoll_pwait() returned %li, "
>> + "expected 1", TEST_RETURN);
>> + } else {
>> + tst_resm(TPASS, "epoll_pwait(sigmask) blocked signal");
>> + }
>> + } else {
>> + if (TEST_RETURN != -1) {
>> + tst_resm(TFAIL, "epoll_wait() succeeded unexpectedly");
>> + } else {
>> + if (TEST_ERRNO == EINTR) {
>> + tst_resm(TPASS | TTERRNO,
>> + "epoll_wait() failed as expected");
>> + } else {
>> + tst_resm(TFAIL | TTERRNO,
>> + "epoll_wait() failed unexpectedly, "
>> + "expected errno is EINTR");
>> + }
>> + }
>> + }
>
>
> Can we put the checks in a separate functions and call them from here?
>
>> + tst_record_childstatus(cleanup, cpid);
>> +
>> + SAFE_READ(cleanup, 1, fds[0], read_buf, sizeof(buf));
>> +}
>> +
>> +static void do_child(void)
>> +{
>> + if (tst_process_state_wait2(getppid(), 'S') != 0) {
>> + tst_brkm(TBROK | TERRNO, cleanup,
>> + "failed to wait for parent process's state");
>> + }
>> +
>> + SAFE_KILL(cleanup, getppid(), SIGUSR1);
>> +
>> + sleep(1);
>
> One second is too much, 0.1 second should be more than enough for the
> test.
>
>> + SAFE_WRITE(cleanup, 1, fds[1], buf, sizeof(buf));
>
> Technically we don't have to write anything here (and we could exit the
> child without sleeping as well) if we passed a timeout to the
> epoll_pwait() instead in the parent (we would have to check for timeout
> instead of success but that shouldn't matter in this test).
>
>> + cleanup();
>> + tst_exit();
>> +}
>
> Otherwise it looks good.
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [LTP] [PATCH v3] epoll_pwait/epoll_pwait01.c: add new testcase
2016-03-03 7:37 ` Guangwen Feng
@ 2016-03-03 8:24 ` Guangwen Feng
2016-03-03 15:13 ` Cyril Hrubis
0 siblings, 1 reply; 8+ messages in thread
From: Guangwen Feng @ 2016-03-03 8:24 UTC (permalink / raw)
To: ltp
Check that:
1) epoll_pwait(2) with sigmask argument allows the caller
to safely wait until either a file descriptor becomes
ready or the timeout expires.
2) epoll_pwait(2) with NULL sigmask argument fails if
interrupted by a signal handler, epoll_pwait(2) should
return -1 and set errno to EINTR.
Signed-off-by: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
---
configure.ac | 1 +
m4/ltp-epoll_pwait.m4 | 25 +++
runtest/ltplite | 2 +
runtest/stress.part3 | 2 +
runtest/syscalls | 1 +
testcases/kernel/syscalls/.gitignore | 1 +
testcases/kernel/syscalls/epoll_pwait/Makefile | 23 +++
.../kernel/syscalls/epoll_pwait/epoll_pwait.h | 35 ++++
.../kernel/syscalls/epoll_pwait/epoll_pwait01.c | 194 +++++++++++++++++++++
9 files changed, 284 insertions(+)
create mode 100644 m4/ltp-epoll_pwait.m4
create mode 100644 testcases/kernel/syscalls/epoll_pwait/Makefile
create mode 100644 testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
create mode 100644 testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
diff --git a/configure.ac b/configure.ac
index cc50397..b065fe6 100644
--- a/configure.ac
+++ b/configure.ac
@@ -183,5 +183,6 @@ LTP_CHECK_KCMP
LTP_CHECK_KCMP_TYPE
LTP_CHECK_PREADV
LTP_CHECK_PWRITEV
+LTP_CHECK_EPOLL_PWAIT
AC_OUTPUT
diff --git a/m4/ltp-epoll_pwait.m4 b/m4/ltp-epoll_pwait.m4
new file mode 100644
index 0000000..e918110
--- /dev/null
+++ b/m4/ltp-epoll_pwait.m4
@@ -0,0 +1,25 @@
+dnl
+dnl Copyright (c) 2016 Fujitsu Ltd.
+dnl Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+dnl
+dnl This program is free software; you can redistribute it and/or modify
+dnl it under the terms of the GNU General Public License as published by
+dnl the Free Software Foundation; either version 2 of the License, or
+dnl (at your option) any later version.
+dnl
+dnl This program is distributed in the hope that it will be useful,
+dnl but WITHOUT ANY WARRANTY; without even the implied warranty of
+dnl MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+dnl the GNU General Public License for more details.
+dnl
+dnl You should have received a copy of the GNU General Public License
+dnl along with this program.
+dnl
+
+dnl
+dnl LTP_CHECK_EPOLL_PWAIT
+dnl ----------------------------
+dnl
+AC_DEFUN([LTP_CHECK_EPOLL_PWAIT],[
+AC_CHECK_FUNCS(epoll_pwait,,)
+])
diff --git a/runtest/ltplite b/runtest/ltplite
index f1795ed..d762ac2 100644
--- a/runtest/ltplite
+++ b/runtest/ltplite
@@ -163,6 +163,8 @@ epoll_wait01 epoll_wait01
epoll_wait02 epoll_wait02
epoll_wait03 epoll_wait03
+epoll_pwait01 epoll_pwait01
+
execl01 execl01
execle01 execle01
execlp01 execlp01
diff --git a/runtest/stress.part3 b/runtest/stress.part3
index ab2c531..ec1de78 100644
--- a/runtest/stress.part3
+++ b/runtest/stress.part3
@@ -104,6 +104,8 @@ epoll_wait01 epoll_wait01
epoll_wait02 epoll_wait02
epoll_wait03 epoll_wait03
+epoll_pwait01 epoll_pwait01
+
execl01 execl01
execle01 execle01
execlp01 execlp01
diff --git a/runtest/syscalls b/runtest/syscalls
index a51ac19..2d981ed 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -125,6 +125,7 @@ epoll01 epoll-ltp
epoll_wait01 epoll_wait01
epoll_wait02 epoll_wait02
epoll_wait03 epoll_wait03
+epoll_pwait01 epoll_pwait01
eventfd01 eventfd01
diff --git a/testcases/kernel/syscalls/.gitignore b/testcases/kernel/syscalls/.gitignore
index 1b10650..f54e065 100644
--- a/testcases/kernel/syscalls/.gitignore
+++ b/testcases/kernel/syscalls/.gitignore
@@ -100,6 +100,7 @@
/epoll_wait/epoll_wait01
/epoll_wait/epoll_wait02
/epoll_wait/epoll_wait03
+/epoll_pwait/epoll_pwait01
/eventfd/eventfd01
/eventfd2/eventfd2_01
/eventfd2/eventfd2_02
diff --git a/testcases/kernel/syscalls/epoll_pwait/Makefile b/testcases/kernel/syscalls/epoll_pwait/Makefile
new file mode 100644
index 0000000..289aaaf
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/Makefile
@@ -0,0 +1,23 @@
+#
+# Copyright (c) 2016 Fujitsu Ltd.
+# Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+# the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.
+#
+
+top_srcdir ?= ../../../..
+
+include $(top_srcdir)/include/mk/testcases.mk
+
+include $(top_srcdir)/include/mk/generic_leaf_target.mk
diff --git a/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
new file mode 100644
index 0000000..694cd46
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait.h
@@ -0,0 +1,35 @@
+/*
+ * Copyright (c) 2016 Fujitsu Ltd.
+ * Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.
+ */
+
+#ifndef EPOLL_PWAIT_H
+#define EPOLL_PWAIT_H
+
+#include <sys/types.h>
+#include "config.h"
+#include "linux_syscall_numbers.h"
+
+#if !defined(HAVE_EPOLL_PWAIT)
+int epoll_pwait(int epfd, struct epoll_event *events, int maxevents,
+ int timeout, const sigset_t *sigmask)
+{
+ return ltp_syscall(__NR_epoll_pwait, epfd, events, maxevents,
+ timeout, sigmask);
+}
+#endif
+
+#endif /* EPOLL_PWAIT_H */
diff --git a/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
new file mode 100644
index 0000000..30d1746
--- /dev/null
+++ b/testcases/kernel/syscalls/epoll_pwait/epoll_pwait01.c
@@ -0,0 +1,194 @@
+/*
+ * Copyright (c) 2016 Fujitsu Ltd.
+ * Author: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+ * the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.
+ */
+
+/*
+ * Description:
+ * Basic test for epoll_pwait(2).
+ * 1) epoll_pwait(2) with sigmask argument allows the caller to
+ * safely wait until either a file descriptor becomes ready
+ * or the timeout expires.
+ * 2) epoll_pwait(2) with NULL sigmask argument fails if
+ * interrupted by a signal handler, epoll_pwait(2) should
+ * return -1 and set errno to EINTR.
+ */
+
+#include <sys/epoll.h>
+#include <sys/types.h>
+#include <unistd.h>
+#include <string.h>
+#include <errno.h>
+
+#include "test.h"
+#include "epoll_pwait.h"
+#include "safe_macros.h"
+
+char *TCID = "epoll_pwait01";
+int TST_TOTAL = 2;
+
+static int epfd, fds[2];
+static sigset_t sigset;
+static struct epoll_event epevs;
+static struct sigaction sa;
+
+static void setup(void);
+static void verify_sigmask(void);
+static void verify_nonsigmask(void);
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED);
+static void do_test(sigset_t *);
+static void do_child(void);
+static void cleanup(void);
+
+int main(int ac, char **av)
+{
+ int lc;
+
+ tst_parse_opts(ac, av, NULL, NULL);
+
+ setup();
+
+ for (lc = 0; TEST_LOOPING(lc); lc++) {
+ tst_count = 0;
+
+ do_test(&sigset);
+ do_test(NULL);
+ }
+
+ cleanup();
+ tst_exit();
+}
+
+static void setup(void)
+{
+ if ((tst_kvercmp(2, 6, 19)) < 0) {
+ tst_brkm(TCONF, NULL, "This test can only run on kernels "
+ "that are 2.6.19 or higher");
+ }
+
+ tst_sig(FORK, DEF_HANDLER, cleanup);
+
+ TEST_PAUSE;
+
+ if (sigemptyset(&sigset) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
+
+ if (sigaddset(&sigset, SIGUSR1) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigaddset() failed");
+
+ sa.sa_flags = 0;
+ sa.sa_handler = sighandler;
+ if (sigemptyset(&sa.sa_mask) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigemptyset() failed");
+
+ if (sigaction(SIGUSR1, &sa, NULL) == -1)
+ tst_brkm(TFAIL | TERRNO, NULL, "sigaction() failed");
+
+ SAFE_PIPE(NULL, fds);
+
+ epfd = epoll_create(1);
+ if (epfd == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to create epoll instance");
+ }
+
+ epevs.events = EPOLLIN;
+ epevs.data.fd = fds[0];
+
+ if (epoll_ctl(epfd, EPOLL_CTL_ADD, fds[0], &epevs) == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to register epoll target");
+ }
+}
+
+static void verify_sigmask(void)
+{
+ if (TEST_RETURN == -1) {
+ tst_resm(TFAIL | TTERRNO, "epoll_pwait() failed");
+ } else if (TEST_RETURN != 0) {
+ tst_resm(TFAIL, "epoll_pwait() returned %li, expected 0",
+ TEST_RETURN);
+ } else {
+ tst_resm(TPASS, "epoll_pwait(sigmask) blocked signal");
+ }
+}
+
+static void verify_nonsigmask(void)
+{
+ if (TEST_RETURN != -1) {
+ tst_resm(TFAIL, "epoll_wait() succeeded unexpectedly");
+ } else {
+ if (TEST_ERRNO == EINTR) {
+ tst_resm(TPASS | TTERRNO,
+ "epoll_wait() failed as expected");
+ } else {
+ tst_resm(TFAIL | TTERRNO,
+ "epoll_wait() failed unexpectedly, "
+ "expected errno is EINTR");
+ }
+ }
+}
+
+static void sighandler(int sig LTP_ATTRIBUTE_UNUSED)
+{
+
+}
+
+static void do_test(sigset_t *sigmask)
+{
+ pid_t cpid;
+
+ cpid = tst_fork();
+ if (cpid < 0)
+ tst_brkm(TBROK | TERRNO, cleanup, "fork() failed");
+
+ if (cpid == 0)
+ do_child();
+
+ TEST(epoll_pwait(epfd, &epevs, 1, 100, sigmask));
+
+ if (sigmask != NULL)
+ verify_sigmask();
+ else
+ verify_nonsigmask();
+
+ tst_record_childstatus(cleanup, cpid);
+}
+
+static void do_child(void)
+{
+ if (tst_process_state_wait2(getppid(), 'S') != 0) {
+ tst_brkm(TBROK | TERRNO, cleanup,
+ "failed to wait for parent process's state");
+ }
+
+ SAFE_KILL(cleanup, getppid(), SIGUSR1);
+
+ cleanup();
+ tst_exit();
+}
+
+static void cleanup(void)
+{
+ if (epfd > 0 && close(epfd))
+ tst_resm(TWARN | TERRNO, "failed to close epfd");
+
+ if (close(fds[0]))
+ tst_resm(TWARN | TERRNO, "close(fds[0]) failed");
+
+ if (close(fds[1]))
+ tst_resm(TWARN | TERRNO, "close(fds[1]) failed");
+}
--
1.8.4.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [LTP] [PATCH v3] epoll_pwait/epoll_pwait01.c: add new testcase
2016-03-03 8:24 ` [LTP] [PATCH v3] " Guangwen Feng
@ 2016-03-03 15:13 ` Cyril Hrubis
0 siblings, 0 replies; 8+ messages in thread
From: Cyril Hrubis @ 2016-03-03 15:13 UTC (permalink / raw)
To: ltp
Hi!
Pushed with two changes, thanks.
> +#ifndef EPOLL_PWAIT_H
> +#define EPOLL_PWAIT_H
> +
> +#include <sys/types.h>
> +#include "config.h"
> +#include "linux_syscall_numbers.h"
> +
> +#if !defined(HAVE_EPOLL_PWAIT)
> +int epoll_pwait(int epfd, struct epoll_event *events, int maxevents,
> + int timeout, const sigset_t *sigmask)
> +{
> + return ltp_syscall(__NR_epoll_pwait, epfd, events, maxevents,
> + timeout, sigmask);
The syscall takes one more parameter than the glibc wrapper. I had to
add the size of the sigmask i.e. _NSIG/8 here otherwise we will get
EINVAL from kernel.
> +}
> +#endif
...
> +static void verify_nonsigmask(void)
> +{
> + if (TEST_RETURN != -1) {
> + tst_resm(TFAIL, "epoll_wait() succeeded unexpectedly");
> + } else {
> + if (TEST_ERRNO == EINTR) {
> + tst_resm(TPASS | TTERRNO,
> + "epoll_wait() failed as expected");
> + } else {
> + tst_resm(TFAIL | TTERRNO,
> + "epoll_wait() failed unexpectedly, "
> + "expected errno is EINTR");
> + }
> + }
> +}
And changed these two verify function to use return after tst_resm() so
that we don't have to do the else-if spagetti code.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-03-03 15:13 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-20 11:09 [LTP] [PATCH 1/2] epoll_pwait/epoll_pwait01.c: add new testcase Guangwen Feng
2016-01-20 11:10 ` [LTP] [PATCH 2/2] epoll_pwait/epoll_pwait02.c: " Guangwen Feng
2016-02-09 14:02 ` Cyril Hrubis
2016-02-19 8:33 ` [LTP] [PATCH v2] epoll_pwait/epoll_pwait01.c: " Guangwen Feng
2016-03-02 15:32 ` Cyril Hrubis
2016-03-03 7:37 ` Guangwen Feng
2016-03-03 8:24 ` [LTP] [PATCH v3] " Guangwen Feng
2016-03-03 15:13 ` 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.