From 8798372261ed1df999bc9f4f3f0be0a230480041 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Tue, 10 Sep 2019 12:53:31 -0500 Subject: Correct initial POSIX signals mask + Modify POSIX thread create extension to ensure expected initial signal mask is provided to system threads, initial tasks and threads, and inheritied by tasks and threads. + Adds psxsignal07 to verify functionality when using a POSIX Initialization thread and POSIX threads. + Adds psxsignal08 to verify functionality when using a Classic API Initialization task and Classic API tasks. Closes #3794. --- testsuites/psxtests/psxsignal08/main.c | 163 ++++++++++++++++++++++++ testsuites/psxtests/psxsignal08/psxsignal08.doc | 22 ++++ testsuites/psxtests/psxsignal08/psxsignal08.scn | 7 + testsuites/psxtests/psxsignal08/rtems_config.c | 45 +++++++ 4 files changed, 237 insertions(+) create mode 100644 testsuites/psxtests/psxsignal08/main.c create mode 100644 testsuites/psxtests/psxsignal08/psxsignal08.doc create mode 100644 testsuites/psxtests/psxsignal08/psxsignal08.scn create mode 100644 testsuites/psxtests/psxsignal08/rtems_config.c (limited to 'testsuites/psxtests/psxsignal08') diff --git a/testsuites/psxtests/psxsignal08/main.c b/testsuites/psxtests/psxsignal08/main.c new file mode 100644 index 0000000000..c55a7712b9 --- /dev/null +++ b/testsuites/psxtests/psxsignal08/main.c @@ -0,0 +1,163 @@ +/* + * @brief psxsignal08 - Ensure initial signal mask and Classic Task inheritance + */ + +/* + * COPYRIGHT (c) 2019. + * On-Line Applications Research Corporation (OAR). + * + * SPDX-License-Identifier: BSD-2-Clause + */ + + +#ifdef HAVE_CONFIG_H +#include "config.h" +#endif + +#include +#include "test_support.h" +#include +#include +#include + +#include + +const char rtems_test_name[] = "PSXSIGNAL 8"; + +static sigset_t main_sigmask; + +#define SIGSET_NBYTES (sizeof(sigset_t)) + +/* #define DEBUG_TEST */ + +static void fetch_sigmask(const char *name, sigset_t *sigset_p) +{ + int rc; + + rc = sigemptyset(sigset_p); + rtems_test_assert(rc == 0); + + rc = sigprocmask(SIG_SETMASK, NULL, sigset_p); + rtems_test_assert(rc == 0); + +#ifdef DEBUG_TEST + /* + * There is no assurance that sigset_t is a primitive type so + * we have to print it a long at a time. + */ + int i; + unsigned char *p = (unsigned char *) sigset_p; + + printf("%s signal mask (in hex):\n ", name); + for (i=0 ; i < SIGSET_NBYTES ; i++) { + printf("%02x%s", *p++, (i % 16 == 15) ? "\n " : " "); + } + printf("\n"); +#endif +} + +static void block_sigmask(int signo, sigset_t *sigset_p) +{ + sigset_t sigset; + int rc; + + /* + * Block the requested signal + */ + rc = sigemptyset(&sigset); + rtems_test_assert(rc == 0); + rc = sigaddset(&sigset, signo); + rtems_test_assert(rc == 0); + + rc = sigprocmask(SIG_BLOCK, &sigset, NULL); + rtems_test_assert(rc == 0); + + /* + * Fetch the current signal mask reflecting the requested update + */ + sigemptyset(sigset_p); + rc = sigprocmask(SIG_SETMASK, NULL, sigset_p); + rtems_test_assert(rc == 0); +} + +static rtems_task task_body(rtems_task_argument arg) +{ + sigset_t mask; + + /* + * There is no assurance that sigset_t is a primitive type so + * we have to use memcmp(). + */ + fetch_sigmask("main", &main_sigmask); + fetch_sigmask((void *)arg, &mask); + rtems_test_assert(main_sigmask == mask); + + (void) rtems_task_delete(RTEMS_SELF); +} + +int main(int argc, char **argv) +{ + sigset_t empty; + rtems_status_code status; + rtems_id id; + + TEST_BEGIN(); + + /* + * Verify first task has empty signal mask + */ + fetch_sigmask("main", &main_sigmask); + + sigemptyset(&empty); + fetch_sigmask("empty", &empty); + + puts("Ensure main task's mask equals sigemptyset"); + rtems_test_assert(main_sigmask == empty); + + /* + * Create a task and see if it inherits empty signal mask. + */ + puts("Ensure parent's empty mask is inherited by thread"); + status = rtems_task_create( + rtems_build_name( 'T', 'A', '1', ' ' ), + 1, + RTEMS_MINIMUM_STACK_SIZE, + RTEMS_DEFAULT_MODES, + RTEMS_DEFAULT_ATTRIBUTES, + &id + ); + directive_failed( status, "rtems_task_create" ); + + status = rtems_task_start( id, task_body, 1 ); + directive_failed( status, "rtems_task_start" ); + + sleep(1); + + /* + * Create a task and see if it inherits non-empty signal mask. + */ + puts("Ensure parent's mask with SIGUSR1 mask is inherited by thread"); + block_sigmask(SIGUSR1, &main_sigmask); + + status = rtems_task_create( + rtems_build_name( 'T', 'A', '1', ' ' ), + 1, + RTEMS_MINIMUM_STACK_SIZE, + RTEMS_DEFAULT_MODES, + RTEMS_DEFAULT_ATTRIBUTES, + &id + ); + directive_failed( status, "rtems_task_create" ); + + status = rtems_task_start( id, task_body, 1 ); + directive_failed( status, "rtems_task_start" ); + + sleep(1); + + + TEST_END(); + + rtems_test_exit(0); + return 0; +} + diff --git a/testsuites/psxtests/psxsignal08/psxsignal08.doc b/testsuites/psxtests/psxsignal08/psxsignal08.doc new file mode 100644 index 0000000000..f476996fad --- /dev/null +++ b/testsuites/psxtests/psxsignal08/psxsignal08.doc @@ -0,0 +1,22 @@ +# COPYRIGHT (c) 2019. +# On-Line Applications Research Corporation (OAR). +# +# SPDX-License-Identifier: BSD-2-Clause + +This file describes the directives and concepts tested by this test set. + +test set name: psxsignal07 + +directives: + + sigprocmask + sigemptyset + sigaddset + +concepts: + ++ Ensure that the initial signal mask is empty (all unmasked). + ++ Ensure that a Classic API task inherits an empty signal mask. + ++ Ensure that a Classic API task inherits an non-empty signal mask. diff --git a/testsuites/psxtests/psxsignal08/psxsignal08.scn b/testsuites/psxtests/psxsignal08/psxsignal08.scn new file mode 100644 index 0000000000..6aa8ce8e39 --- /dev/null +++ b/testsuites/psxtests/psxsignal08/psxsignal08.scn @@ -0,0 +1,7 @@ +** BEGIN OF TEST PSXSIGNAL 8 *** +Ensure main task's mask equals sigemptyset +Ensure parent's empty mask is inherited by thread +Ensure parent's mask with SIGUSR1 mask is inherited by thread + +*** END OF TEST PSXSIGNAL 8 *** + diff --git a/testsuites/psxtests/psxsignal08/rtems_config.c b/testsuites/psxtests/psxsignal08/rtems_config.c new file mode 100644 index 0000000000..f56e88c8be --- /dev/null +++ b/testsuites/psxtests/psxsignal08/rtems_config.c @@ -0,0 +1,45 @@ +/* + * COPYRIGHT (c) 2019. + * On-Line Applications Research Corporation (OAR). + * + * SPDX-License-Identifier: BSD-2-Clause + */ + +#include +#include + +int main(int argc, char **argv); + +static char *argv_list[] = { + "report", + "" +}; +static rtems_task Init(rtems_task_argument arg) +{ + (void) arg; /* deliberately ignored */ + + /* + * Initialize optional services + */ + + /* + * Could get arguments from command line or have a static set. + */ + (void) main(1, argv_list); + + exit(0); +} + +#include /* for device driver prototypes */ + +/* NOTICE: the clock driver is explicitly disabled */ +#define CONFIGURE_APPLICATION_NEEDS_CLOCK_DRIVER +#define CONFIGURE_APPLICATION_NEEDS_CONSOLE_DRIVER + +#define CONFIGURE_RTEMS_INIT_TASKS_TABLE + +#define CONFIGURE_UNLIMITED_OBJECTS +#define CONFIGURE_UNIFIED_WORK_AREAS + +#define CONFIGURE_INIT +#include -- cgit v1.2.3