diff options
author | Matthias Maennich <maennich@google.com> | 2019-03-19 18:41:35 +0000 |
---|---|---|
committer | Petr Vorel <pvorel@suse.cz> | 2019-03-21 19:15:21 +0100 |
commit | b03885fffbd51bc61b7f4e0014774ed0f5dd6569 (patch) | |
tree | 3b8fd31924191be3a9a8c001b012c0352b69b12b /testcases/kernel | |
parent | 91242bbd2126d76bd3515163253d069ed7cd4fcd (diff) |
rt_sigpending02: reuse code from sigpending02
Rather than forking the code of sigpending02 completely, reuse its code
for rt_sigpending02 by conditionally compiling the syscalls accordingly.
That way we ensure tests written for either rt_sigpending or sigpending
are also considered for the respective other.
Signed-off-by: Matthias Maennich <maennich@google.com>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
Diffstat (limited to 'testcases/kernel')
4 files changed, 25 insertions, 52 deletions
diff --git a/testcases/kernel/syscalls/rt_sigpending/Makefile b/testcases/kernel/syscalls/rt_sigpending/Makefile index 60c2e0140..57ba3dac2 100644 --- a/testcases/kernel/syscalls/rt_sigpending/Makefile +++ b/testcases/kernel/syscalls/rt_sigpending/Makefile @@ -4,4 +4,12 @@ top_srcdir ?= ../../../.. include $(top_srcdir)/include/mk/testcases.mk + +CPPFLAGS += -DTEST_RT_SIGPENDING + +rt_sigpending02: $(abs_srcdir)/../sigpending/sigpending02.c + $(LINK.c) $^ $(LOADLIBES) $(LDLIBS) $(OUTPUT_OPTION) + +MAKE_TARGETS := rt_sigpending02 + include $(top_srcdir)/include/mk/generic_leaf_target.mk diff --git a/testcases/kernel/syscalls/rt_sigpending/rt_sigpending02.c b/testcases/kernel/syscalls/rt_sigpending/rt_sigpending02.c deleted file mode 100644 index 06d1c1578..000000000 --- a/testcases/kernel/syscalls/rt_sigpending/rt_sigpending02.c +++ /dev/null @@ -1,48 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 -/* - * Copyright (c) International Business Machines Corp., 2002 - * - * AUTHORS - * Paul Larson - * - * DESCRIPTION - * Test to see that the proper errors are returned by rt_sigpending - * - * Test 1: - * Call rt_sigpending(sigset_t*=-1, SIGSETSIZE), - * it should return -1 with errno EFAULT - */ - -#include <errno.h> -#include <signal.h> -#include <sys/types.h> - -#include "tst_test.h" -#include "ltp_signal.h" -#include "lapi/syscalls.h" - -static void run(void) -{ - /* set sigset to point to an invalid location */ - sigset_t *sigset = (sigset_t *) - 1; - - TEST(tst_syscall(__NR_rt_sigpending, sigset, SIGSETSIZE)); - - /* check return code */ - if (TST_RET == -1) { - if (TST_ERR != EFAULT) { - tst_res(TFAIL | TTERRNO, - "rt_sigpending() Failed with wrong errno, " - "expected errno=%d, got ", EFAULT); - } else { - tst_res(TPASS | TTERRNO, "expected failure"); - } - } else { - tst_res(TFAIL, - "rt_sigpending() Failed, expected return value=-1, got %ld", TST_RET); - } -} - -static struct tst_test test = { - .test_all = run -}; diff --git a/testcases/kernel/syscalls/sigpending/Makefile b/testcases/kernel/syscalls/sigpending/Makefile index bd617d806..00a7d5e2b 100644 --- a/testcases/kernel/syscalls/sigpending/Makefile +++ b/testcases/kernel/syscalls/sigpending/Makefile @@ -20,4 +20,6 @@ top_srcdir ?= ../../../.. include $(top_srcdir)/include/mk/testcases.mk +CPPFLAGS += -DTEST_SIGPENDING + include $(top_srcdir)/include/mk/generic_leaf_target.mk diff --git a/testcases/kernel/syscalls/sigpending/sigpending02.c b/testcases/kernel/syscalls/sigpending/sigpending02.c index e7ec0d681..70dda858d 100644 --- a/testcases/kernel/syscalls/sigpending/sigpending02.c +++ b/testcases/kernel/syscalls/sigpending/sigpending02.c @@ -6,7 +6,10 @@ * Paul Larson * * DESCRIPTION - * Test to see that the proper errors are returned by sigpending + * Test to see that the proper errors are returned by sigpending. All the + * tests can also be compiled to use the rt_sigpending syscall instead. To + * simplify the documentation, only sigpending() is usually mentioned + * below. * * Test 1: * Call sigpending(sigset_t*=-1), it should return -1 with errno EFAULT @@ -17,6 +20,7 @@ #include <sys/types.h> #include "tst_test.h" +#include "ltp_signal.h" #include "lapi/syscalls.h" static void run(void) @@ -24,20 +28,27 @@ static void run(void) /* set sigset to point to an invalid location */ sigset_t *sigset = (sigset_t *) - 1; +#if defined(TEST_SIGPENDING) TEST(tst_syscall(__NR_sigpending, sigset)); +#elif defined(TEST_RT_SIGPENDING) + TEST(tst_syscall(__NR_rt_sigpending, sigset, SIGSETSIZE)); +#else +#error Neither TEST_SIGPENDING nor TEST_RT_SIGPENDING is defined! +#endif /* check return code */ if (TST_RET == -1) { if (TST_ERR != EFAULT) { tst_res(TFAIL | TTERRNO, - "sigpending() Failed with wrong errno, " - "expected errno=%d, got ", EFAULT); + "syscall failed with wrong errno, expected errno=%d, got %d", + EFAULT, TST_ERR); } else { tst_res(TPASS | TTERRNO, "expected failure"); } } else { tst_res(TFAIL, - "sigpending() Failed, expected return value=-1, got %ld", TST_RET); + "syscall failed, expected return value=-1, got %ld", + TST_RET); } } |