From ef05780692200613f2de4828e2e27172600ca54c Mon Sep 17 00:00:00 2001 From: Taylor R Campbell Date: Sat, 18 Nov 2023 21:16:40 +0000 Subject: [PATCH] Revert "WIP: add test for interrupting eventfd read/write by signal" This reverts commit f71c52d125f8376c3e99f1fda37f1f466c97e199. --- tests/lib/libc/sys/t_eventfd.c | 71 +--------------------------------- 1 file changed, 2 insertions(+), 69 deletions(-) diff --git a/tests/lib/libc/sys/t_eventfd.c b/tests/lib/libc/sys/t_eventfd.c index 4dc2fe3e4baf..4eaea2aff3ba 100644 --- a/tests/lib/libc/sys/t_eventfd.c +++ b/tests/lib/libc/sys/t_eventfd.c @@ -31,27 +31,23 @@ __COPYRIGHT("@(#) Copyright (c) 2020\ The NetBSD Foundation, inc. All rights reserved."); __RCSID("$NetBSD: t_eventfd.c,v 1.3 2022/02/20 15:21:14 thorpej Exp $"); +#include #include #include #include #include #include #include -#include - #include #include #include -#include -#include #include +#include #include #include #include -#include "h_macros.h" - struct helper_context { int efd; @@ -819,67 +815,6 @@ ATF_TC_BODY(eventfd_fcntl, tc) /*****************************************************************************/ -static pthread_key_t eventfd_signal_key; - -static void -eventfd_read_signal_handler(int signo) -{ - volatile sig_atomic_t *const flag = - pthread_getspecific(eventfd_signal_key); - - *flag = 1; -} - -static void * -eventfd_read_signal_helper(void * const v) -{ - struct helper_context * const ctx = v; - eventfd_t efd_value; - sig_atomic_t flag = 0; - int error; - - RZ(pthread_setspecific(eventfd_signal_key, &flag)); - if (signal(SIGUSR1, &eventfd_read_signal_handler) == SIG_ERR) - atf_tc_fail("signal(SIGUSR1): %s", strerror(errno)); - - ATF_REQUIRE(wait_barrier(ctx)); - ATF_REQUIRE(eventfd_read(ctx->efd, &efd_value) == -1); - error = errno; - ATF_REQUIRE_MSG(error == EINTR, "errno=%d (%s)", error, - strerror(error)); - ATF_REQUIRE_MSG(flag, "signal not delivered"); - - return NULL; -} - -ATF_TC(eventfd_read_signal); -ATF_TC_HEAD(eventfd_read_signal, tc) -{ - atf_tc_set_md_var(tc, "descr", - "Tests eventfd reads can be interrupted by signal"); -} -ATF_TC_BODY(eventfd_read_signal, tc) -{ - struct helper_context ctx; - pthread_t helper; - - RZ(pthread_key_create(&eventfd_signal_key, NULL)); - - init_helper_context(&ctx); - - RL(ctx.efd = eventfd(0, 0)); - RZ(pthread_create(&helper, NULL, &eventfd_read_signal_helper, &ctx)); - - ATF_REQUIRE(wait_barrier(&ctx)); /* wait for helper to start */ - (void)sleep(1); /* wait for the read to block */ - (void)alarm(1); /* set a deadline */ - RZ(pthread_kill(helper, SIGUSR1)); /* wake helper */ - RZ(pthread_join(helper, NULL)); /* wait for helper to wake and fail */ - (void)alarm(0); /* clear deadline */ -} - -/*****************************************************************************/ - ATF_TP_ADD_TCS(tp) { ATF_TP_ADD_TC(tp, eventfd_normal); @@ -890,8 +825,6 @@ ATF_TP_ADD_TCS(tp) ATF_TP_ADD_TC(tp, eventfd_select_poll_kevent_block); ATF_TP_ADD_TC(tp, eventfd_restart); ATF_TP_ADD_TC(tp, eventfd_fcntl); - ATF_TP_ADD_TC(tp, eventfd_read_signal); -// ATF_TP_ADD_TC(tp, eventfd_write_signal); return atf_no_error(); }