[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] rt/tst-mqueue*: Return UNSUPPORTED when mq_open fails with E
From: |
Adhemerval Zanella |
Subject: |
Re: [PATCH] rt/tst-mqueue*: Return UNSUPPORTED when mq_open fails with ENOSYS |
Date: |
Mon, 17 Jan 2022 14:46:37 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 |
On 15/01/2022 11:53, Samuel Thibault wrote:
> Rather than returning 0 or a failure.
LGTM, thanks.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
> ---
> rt/tst-mqueue1.c | 10 +++++++---
> rt/tst-mqueue10.c | 10 +++++++++-
> rt/tst-mqueue2.c | 10 +++++++---
> rt/tst-mqueue3.c | 9 ++++-----
> rt/tst-mqueue4.c | 10 +++++++---
> rt/tst-mqueue5.c | 10 +++++++---
> rt/tst-mqueue6.c | 10 +++++++---
> rt/tst-mqueue7.c | 1 +
> rt/tst-mqueue8.c | 6 +++++-
> rt/tst-mqueue9.c | 10 +++++++---
> 10 files changed, 61 insertions(+), 25 deletions(-)
>
> diff --git a/rt/tst-mqueue1.c b/rt/tst-mqueue1.c
> index 1fa807f19a..3aa0369a2b 100644
> --- a/rt/tst-mqueue1.c
> +++ b/rt/tst-mqueue1.c
> @@ -26,6 +26,7 @@
> #include <time.h>
> #include <unistd.h>
> #include <stdint.h>
> +#include <support/check.h>
> #include "tst-mqueue.h"
>
> static int
> @@ -322,11 +323,14 @@ do_test (void)
>
> if (q == (mqd_t) -1)
> {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> printf ("mq_open failed with: %m\n");
> - return result;
> + return 1;
> }
> - else
> - add_temp_mq (name);
> +
> + add_temp_mq (name);
>
> result |= do_one_test (q, name, 0);
>
> diff --git a/rt/tst-mqueue10.c b/rt/tst-mqueue10.c
> index 9da9838258..059c064488 100644
> --- a/rt/tst-mqueue10.c
> +++ b/rt/tst-mqueue10.c
> @@ -43,7 +43,15 @@ do_test (void)
>
> struct mq_attr attr = { .mq_maxmsg = 1, .mq_msgsize = sizeof (msg) };
> mqd_t q = mq_open (name, O_CREAT | O_EXCL | O_RDWR, 0600, &attr);
> - TEST_VERIFY_EXIT (q != (mqd_t) -1);
> +
> + if (q == (mqd_t) -1)
> + {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> + printf ("mq_open failed with: %m\n");
> + return 1;
> + }
>
> struct timespec ts = { TYPE_MAXIMUM (time_t), 0 };
>
> diff --git a/rt/tst-mqueue2.c b/rt/tst-mqueue2.c
> index 18fadfe09c..b355d22474 100644
> --- a/rt/tst-mqueue2.c
> +++ b/rt/tst-mqueue2.c
> @@ -27,6 +27,7 @@
> #include <sys/wait.h>
> #include <time.h>
> #include <unistd.h>
> +#include <support/check.h>
> #include "tst-mqueue.h"
>
> static void
> @@ -48,11 +49,14 @@ do_test (void)
>
> if (q == (mqd_t) -1)
> {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> printf ("mq_open failed with: %m\n");
> - return result;
> + return 1;
> }
> - else
> - add_temp_mq (name);
> +
> + add_temp_mq (name);
>
> mqd_t q2 = mq_open (name, O_CREAT | O_EXCL | O_RDWR, 0600, &attr);
> if (q2 != (mqd_t) -1)
> diff --git a/rt/tst-mqueue3.c b/rt/tst-mqueue3.c
> index 812602e2c7..634db921e6 100644
> --- a/rt/tst-mqueue3.c
> +++ b/rt/tst-mqueue3.c
> @@ -27,6 +27,7 @@
> #include <sys/mman.h>
> #include <sys/wait.h>
> #include <unistd.h>
> +#include <support/check.h>
>
> #if _POSIX_THREADS
> # include <pthread.h>
> @@ -161,15 +162,13 @@ do_test (void)
> /* Create the message queue. */
> struct mq_attr attr = { .mq_maxmsg = MAXMSG, .mq_msgsize = MSGSIZE };
> m = mq_open (mqname, O_CREAT | O_EXCL | O_RDWR, 0600, &attr);
> +
> if (m == -1)
> {
> if (errno == ENOSYS)
> - {
> - puts ("not implemented");
> - return 0;
> - }
> + FAIL_UNSUPPORTED ("mq_open not supported");
>
> - puts ("mq_open failed");
> + printf ("mq_open failed with: %m\n");
> return 1;
> }
>
> diff --git a/rt/tst-mqueue4.c b/rt/tst-mqueue4.c
> index ea19c27ab4..62d427a3ca 100644
> --- a/rt/tst-mqueue4.c
> +++ b/rt/tst-mqueue4.c
> @@ -29,6 +29,7 @@
> #include <sys/wait.h>
> #include <time.h>
> #include <unistd.h>
> +#include <support/check.h>
> #include "tst-mqueue.h"
>
> #define TEST_FUNCTION do_test ()
> @@ -45,11 +46,14 @@ do_test (void)
>
> if (q == (mqd_t) -1)
> {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> printf ("mq_open failed with: %m\n");
> - return result;
> + return 1;
> }
> - else
> - add_temp_mq (name);
> +
> + add_temp_mq (name);
>
> *p = '.';
> memset (p + 1, 'x', NAME_MAX + 1 - (p - name));
> diff --git a/rt/tst-mqueue5.c b/rt/tst-mqueue5.c
> index 24539dbb14..70d97a36c2 100644
> --- a/rt/tst-mqueue5.c
> +++ b/rt/tst-mqueue5.c
> @@ -30,6 +30,7 @@
> #include <sys/wait.h>
> #include <time.h>
> #include <unistd.h>
> +#include <support/check.h>
> #include "tst-mqueue.h"
>
> #if _POSIX_THREADS && defined SIGRTMIN && defined SA_SIGINFO
> @@ -630,11 +631,14 @@ do_test (void)
>
> if (q == (mqd_t) -1)
> {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> printf ("mq_open failed with: %m\n");
> - return result;
> + return 1;
> }
> - else
> - add_temp_mq (name);
> +
> + add_temp_mq (name);
>
> struct sigevent ev;
> memset (&ev, 0xaa, sizeof (ev));
> diff --git a/rt/tst-mqueue6.c b/rt/tst-mqueue6.c
> index 4831bf2dc1..bc875f101e 100644
> --- a/rt/tst-mqueue6.c
> +++ b/rt/tst-mqueue6.c
> @@ -30,6 +30,7 @@
> #include <sys/wait.h>
> #include <time.h>
> #include <unistd.h>
> +#include <support/check.h>
> #include "tst-mqueue.h"
>
> #if _POSIX_THREADS
> @@ -116,11 +117,14 @@ do_test (void)
>
> if (q == (mqd_t) -1)
> {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> printf ("mq_open failed with: %m\n");
> - return result;
> + return 1;
> }
> - else
> - add_temp_mq (name);
> +
> + add_temp_mq (name);
>
> pthread_attr_t nattr;
> if (pthread_attr_init (&nattr)
> diff --git a/rt/tst-mqueue7.c b/rt/tst-mqueue7.c
> index 302e9e454c..948c0d1326 100644
> --- a/rt/tst-mqueue7.c
> +++ b/rt/tst-mqueue7.c
> @@ -25,6 +25,7 @@
> #include <sys/wait.h>
> #include <time.h>
> #include <unistd.h>
> +#include <support/check.h>
>
> #define OPT_AFTEREXEC 20000
>
> diff --git a/rt/tst-mqueue8.c b/rt/tst-mqueue8.c
> index da1e7644e6..b14cde1188 100644
> --- a/rt/tst-mqueue8.c
> +++ b/rt/tst-mqueue8.c
> @@ -23,6 +23,7 @@
> #include <unistd.h>
> #if _POSIX_THREADS
> # include <pthread.h>
> +# include <support/check.h>
>
> static pthread_barrier_t b;
>
> @@ -95,8 +96,11 @@ do_test (void)
>
> if (q == (mqd_t) -1)
> {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> printf ("mq_open failed with: %m\n");
> - return 0;
> + return 1;
> }
>
> if (mq_unlink (name) != 0)
> diff --git a/rt/tst-mqueue9.c b/rt/tst-mqueue9.c
> index ef2233623b..fe991ec32f 100644
> --- a/rt/tst-mqueue9.c
> +++ b/rt/tst-mqueue9.c
> @@ -21,6 +21,7 @@
> #include <stdlib.h>
> #include <string.h>
> #include <unistd.h>
> +#include <support/check.h>
> #include "tst-mqueue.h"
>
> #define TEST_FUNCTION do_test ()
> @@ -41,11 +42,14 @@ do_test (void)
>
> if (q == (mqd_t) -1)
> {
> + if (errno == ENOSYS)
> + FAIL_UNSUPPORTED ("mq_open not supported");
> +
> printf ("mq_open failed with: %m\n");
> - return 0;
> + return 1;
> }
> - else
> - add_temp_mq (name);
> +
> + add_temp_mq (name);
>
> if (seteuid (1) != 0)
> {