From: Viresh Kumar <viresh.kumar@linaro.org>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH V2 2/3] syscalls: sigwaitinfo: Remove SUCCEED_OR_DIE() macro
Date: Tue, 21 Jul 2020 14:22:03 +0530 [thread overview]
Message-ID: <8fbd46e33184029f89829af3cd471c05ee6ce496.1595321086.git.viresh.kumar@linaro.org> (raw)
In-Reply-To: <cover.1595321086.git.viresh.kumar@linaro.org>
The macro makes it less readable and there isn't a great deal of code it
is trying to remove. Get rid of it.
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
.../kernel/syscalls/sigwaitinfo/sigwaitinfo01.c | 156 ++++++++++++++-------
1 file changed, 109 insertions(+), 47 deletions(-)
diff --git a/testcases/kernel/syscalls/sigwaitinfo/sigwaitinfo01.c b/testcases/kernel/syscalls/sigwaitinfo/sigwaitinfo01.c
index 86fce4427a6d..5e1baf9df774 100644
--- a/testcases/kernel/syscalls/sigwaitinfo/sigwaitinfo01.c
+++ b/testcases/kernel/syscalls/sigwaitinfo/sigwaitinfo01.c
@@ -10,13 +10,6 @@
#include "lapi/syscalls.h"
#include "tst_sig_proc.h"
-#define SUCCEED_OR_DIE(syscall, message, ...) \
- (errno = 0, \
- ({int ret=syscall(__VA_ARGS__); \
- if (ret==-1) \
- tst_brk(TBROK|TERRNO, message);\
- ret;}))
-
/* Report success iff TST_RET and TST_ERR are equal to
exp_return and exp_errno, resp., and cond is true. If cond is not
true, report condition_errmsg
@@ -113,7 +106,10 @@ void test_empty_set(swi_func sigwaitinfo, int signo,
siginfo_t si;
pid_t child;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
+
/* Run a child that will wake us up */
child = create_sig_proc(signo, INT_MAX, 100000);
@@ -135,7 +131,9 @@ void test_timeout(swi_func sigwaitinfo, int signo, enum tst_ts_type type)
tst_ts_set_sec(&ts, 1);
tst_ts_set_nsec(&ts, 0);
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
/* Run a child that will wake us up */
child = create_sig_proc(signo, INT_MAX, 100000);
@@ -157,8 +155,13 @@ void test_unmasked_matching(swi_func sigwaitinfo, int signo,
siginfo_t si;
pid_t child;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, signo);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
+
+ TEST(sigaddset(&sigs, signo));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
/* Run a child that will wake us up */
child = create_sig_proc(signo, INT_MAX, 100000);
@@ -178,8 +181,14 @@ void test_unmasked_matching_noinfo(swi_func sigwaitinfo, int signo,
sigset_t sigs;
pid_t child;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, signo);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
+
+ TEST(sigaddset(&sigs, signo));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
+
/* Run a child that will wake us up */
child = create_sig_proc(signo, INT_MAX, 100000);
@@ -197,16 +206,27 @@ void test_masked_matching(swi_func sigwaitinfo, int signo,
siginfo_t si;
pid_t child;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, signo);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
+
+ TEST(sigaddset(&sigs, signo));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
+
/* let's not get interrupted by our dying child */
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, SIGCHLD);
+ TEST(sigaddset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
- SUCCEED_OR_DIE(sigprocmask, "sigprocmask failed", SIG_SETMASK, &sigs,
- &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &sigs, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigprocmask() failed");
/* don't wait on a SIGCHLD */
- SUCCEED_OR_DIE(sigdelset, "sigaddset failed", &sigs, SIGCHLD);
+ TEST(sigdelset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigdelset() failed");
/* Run a child that will wake us up */
child = create_sig_proc(signo, 1, 0);
@@ -216,8 +236,9 @@ void test_masked_matching(swi_func sigwaitinfo, int signo,
&& si.si_code == SI_USER
&& si.si_signo == signo, "Struct siginfo mismatch");
- SUCCEED_OR_DIE(sigprocmask, "restoring original signal mask failed",
- SIG_SETMASK, &oldmask, &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &oldmask, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "restoring original signal mask failed");
if (sigismember(&oldmask, signo))
tst_res(TPASS, "sigwaitinfo restored the original mask");
@@ -239,17 +260,31 @@ void test_masked_matching_rt(swi_func sigwaitinfo, int signo,
signo = SIGRTMIN + 1;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, signo);
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, signo + 1);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
+
+ TEST(sigaddset(&sigs, signo));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
+
+ TEST(sigaddset(&sigs, signo + 1));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
+
/* let's not get interrupted by our dying child */
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, SIGCHLD);
+ TEST(sigaddset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
- SUCCEED_OR_DIE(sigprocmask, "sigprocmask failed", SIG_SETMASK, &sigs,
- &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &sigs, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigprocmask() failed");
/* don't wait on a SIGCHLD */
- SUCCEED_OR_DIE(sigdelset, "sigdelset failed", &sigs, SIGCHLD);
+ TEST(sigdelset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigdelset() failed");
/* Run a child that will wake us up */
child[0] = create_sig_proc(signo, 1, 0);
@@ -271,8 +306,9 @@ void test_masked_matching_rt(swi_func sigwaitinfo, int signo,
&& si.si_signo == signo + 1,
"Struct siginfo mismatch");
- SUCCEED_OR_DIE(sigprocmask, "restoring original signal mask failed",
- SIG_SETMASK, &oldmask, &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &oldmask, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "restoring original signal mask failed");
if (sigismember(&oldmask, signo))
tst_res(TPASS, "sigwaitinfo restored the original mask");
@@ -287,16 +323,27 @@ void test_masked_matching_noinfo(swi_func sigwaitinfo, int signo,
sigset_t sigs, oldmask;
pid_t child;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, signo);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
+
+ TEST(sigaddset(&sigs, signo));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
+
/* let's not get interrupted by our dying child */
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, SIGCHLD);
+ TEST(sigaddset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
- SUCCEED_OR_DIE(sigprocmask, "sigprocmask failed", SIG_SETMASK, &sigs,
- &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &sigs, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigprocmask() failed");
/* don't wait on a SIGCHLD */
- SUCCEED_OR_DIE(sigdelset, "sigaddset failed", &sigs, SIGCHLD);
+ TEST(sigdelset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigdelset() failed");
/* Run a child that will wake us up */
child = create_sig_proc(signo, 1, 0);
@@ -304,8 +351,9 @@ void test_masked_matching_noinfo(swi_func sigwaitinfo, int signo,
TEST(sigwaitinfo(&sigs, NULL, NULL));
REPORT_SUCCESS(signo, 0);
- SUCCEED_OR_DIE(sigprocmask, "restoring original signal mask failed",
- SIG_SETMASK, &oldmask, &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &oldmask, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "restoring original signal mask failed");
if (sigismember(&oldmask, signo))
tst_res(TPASS, "sigwaitinfo restored the original mask");
@@ -323,16 +371,27 @@ void test_bad_address(swi_func sigwaitinfo, int signo,
sigset_t sigs, oldmask;
pid_t child;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, signo);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
+
+ TEST(sigaddset(&sigs, signo));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
+
/* let's not get interrupted by our dying child */
- SUCCEED_OR_DIE(sigaddset, "sigaddset failed", &sigs, SIGCHLD);
+ TEST(sigaddset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigaddset() failed");
- SUCCEED_OR_DIE(sigprocmask, "sigprocmask failed", SIG_SETMASK, &sigs,
- &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &sigs, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigprocmask() failed");
/* don't wait on a SIGCHLD */
- SUCCEED_OR_DIE(sigdelset, "sigaddset failed", &sigs, SIGCHLD);
+ TEST(sigdelset(&sigs, SIGCHLD));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigdelset() failed");
/* Run a child that will wake us up */
child = create_sig_proc(signo, 1, 0);
@@ -340,8 +399,9 @@ void test_bad_address(swi_func sigwaitinfo, int signo,
TEST(sigwaitinfo(&sigs, (void *)1, NULL));
REPORT_SUCCESS(-1, EFAULT);
- SUCCEED_OR_DIE(sigprocmask, "sigprocmask failed", SIG_SETMASK, &oldmask,
- &oldmask);
+ TEST(sigprocmask(SIG_SETMASK, &oldmask, &oldmask));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigprocmask() failed");
SAFE_KILL(child, SIGTERM);
SAFE_WAIT(NULL);
@@ -398,7 +458,9 @@ void test_bad_address3(swi_func sigwaitinfo, int signo LTP_ATTRIBUTE_UNUSED,
enum tst_ts_type type LTP_ATTRIBUTE_UNUSED)
{
sigset_t sigs;
- SUCCEED_OR_DIE(sigemptyset, "sigemptyset failed", &sigs);
+ TEST(sigemptyset(&sigs));
+ if (TST_RET == -1)
+ tst_brk(TBROK | TTERRNO, "sigemptyset() failed");
TEST(sigwaitinfo(&sigs, NULL, (void *)1));
REPORT_SUCCESS(-1, EFAULT);
--
2.14.1
next prev parent reply other threads:[~2020-07-21 8:52 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-21 8:52 [LTP] [PATCH V2 0/3] syscalls: Rearrange sigwait syscall tests Viresh Kumar
2020-07-21 8:52 ` [LTP] [PATCH V2 1/3] syscalls: sigwaitinfo: Suppress all warnings around unused parameter Viresh Kumar
2020-07-21 8:52 ` Viresh Kumar [this message]
2020-07-22 10:09 ` [LTP] [PATCH V2 2/3] syscalls: sigwaitinfo: Remove SUCCEED_OR_DIE() macro Cyril Hrubis
2020-07-22 10:11 ` Cyril Hrubis
2020-07-21 8:52 ` [LTP] [PATCH V2 3/3] syscalls: sigwait: Separate out code to relevant folders Viresh Kumar
2020-07-22 14:05 ` Cyril Hrubis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8fbd46e33184029f89829af3cd471c05ee6ce496.1595321086.git.viresh.kumar@linaro.org \
--to=viresh.kumar@linaro.org \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.