aboutsummaryrefslogtreecommitdiff
path: root/nptl/tst-cancel4.c
diff options
context:
space:
mode:
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>2016-06-14 12:18:32 -0300
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>2016-07-05 10:49:37 -0300
commit2918b0d0ecbf781400577d63aed7eaa4498dad91 (patch)
treed1921e8fa42044585bf7f7e0ca17f77c6129b8b6 /nptl/tst-cancel4.c
parent87523e9c3605037cee54bbc6bd7a040a15979cc1 (diff)
downloadglibc-2918b0d0ecbf781400577d63aed7eaa4498dad91.tar
glibc-2918b0d0ecbf781400577d63aed7eaa4498dad91.tar.gz
glibc-2918b0d0ecbf781400577d63aed7eaa4498dad91.tar.bz2
glibc-2918b0d0ecbf781400577d63aed7eaa4498dad91.zip
nptl: Add more coverage in tst-cancel4
This patch adds early cancel test for open syscall through a FIFO (thus makign subsequent call to open block until the other end is also opened). It also cleanup the sigpause tests by using sigpause along with SIGINT instead of __xpg_sigpause and SIGCANCEL. Since the idea is just to test the cancellation handling there is no need to expose internal glibc implementation details to the test through pthreadP.h inclusion. Tested x86_64. * nptl/tst-cancel4-common.c (do_test): Add temporary fifo creation. * nptl/tst-cancel4-common.h (fifoname): New variable. (fifofd): Likewise. (cl_fifo): New function. * nptl/tst-cancel4.c (tf_sigpause): Replace SIGCANCEL usage by SIGINT. (tf_open): Add early cancel test.
Diffstat (limited to 'nptl/tst-cancel4.c')
-rw-r--r--nptl/tst-cancel4.c42
1 files changed, 21 insertions, 21 deletions
diff --git a/nptl/tst-cancel4.c b/nptl/tst-cancel4.c
index 4221af4bdf..fd0ab4070e 100644
--- a/nptl/tst-cancel4.c
+++ b/nptl/tst-cancel4.c
@@ -36,8 +36,6 @@
#include <sys/poll.h>
#include <sys/wait.h>
-#include "pthreadP.h"
-
/* Since STREAMS are not supported in the standard Linux kernel and
there we don't advertise STREAMS as supported is no need to test
@@ -67,6 +65,7 @@
#include "tst-cancel4-common.h"
+
#ifndef IPC_ADDVAL
# define IPC_ADDVAL 0
#endif
@@ -734,13 +733,7 @@ tf_sigpause (void *arg)
pthread_cleanup_push (cl, NULL);
-#ifdef SIGCANCEL
- /* Just for fun block the cancellation signal. We need to use
- __xpg_sigpause since otherwise we will get the BSD version. */
- __xpg_sigpause (SIGCANCEL);
-#else
- pause ();
-#endif
+ sigpause (sigmask (SIGINT));
pthread_cleanup_pop (0);
@@ -1348,27 +1341,34 @@ static void *
tf_open (void *arg)
{
if (arg == NULL)
- // XXX If somebody can provide a portable test case in which open()
- // blocks we can enable this test to run in both rounds.
- abort ();
-
- int r = pthread_barrier_wait (&b2);
- if (r != 0 && r != PTHREAD_BARRIER_SERIAL_THREAD)
{
- printf ("%s: barrier_wait failed\n", __FUNCTION__);
- exit (1);
+ fifofd = mkfifo (fifoname, S_IWUSR | S_IRUSR);
+ if (fifofd == -1)
+ {
+ printf ("%s: mkfifo failed: %m\n", __func__);
+ exit (1);
+ }
+ }
+ else
+ {
+ int r = pthread_barrier_wait (&b2);
+ if (r != 0 && r != PTHREAD_BARRIER_SERIAL_THREAD)
+ {
+ printf ("%s: barrier_wait failed: %m\n", __func__);
+ exit (1);
+ }
}
- r = pthread_barrier_wait (&b2);
+ int r = pthread_barrier_wait (&b2);
if (r != 0 && r != PTHREAD_BARRIER_SERIAL_THREAD)
{
- printf ("%s: 2nd barrier_wait failed\n", __FUNCTION__);
+ printf ("%s: 2nd barrier_wait failed: %m\n", __func__);
exit (1);
}
- pthread_cleanup_push (cl, NULL);
+ pthread_cleanup_push (cl_fifo, NULL);
- open ("Makefile", O_RDONLY);
+ open (arg ? "Makefile" : fifoname, O_RDONLY);
pthread_cleanup_pop (0);