diff mbox

Fix test-errno issues

Message ID 1489414901-13605-1-git-send-email-adhemerval.zanella@linaro.org
State Superseded
Headers show

Commit Message

Adhemerval Zanella March 13, 2017, 2:21 p.m. UTC 
This patch fixes multiple issues of test-errno.c (9a56f8718341):

  - Rename Linux test-errno.c to test-errno-linux.c to avoid build
    the same source for both tests.

  - Add a mlock check for 32 bits build running on 64 bits kernels.
    Althuough man pages states that mlock fails with EINVAL if final
    address overflows, kernels does not return it for aforementioned
    condition (it returns ENOMEM instead).  Although it seems to be
    a kernel issue for compat syscall handling, I think it is worth
    to still check syscall return and document the behavior.

  - Initialize option lenght for setsockopt check.

Checked on x86_64-linux-gnu and i686-linux-gnu (running on 64 bits
kernel).

	* posix/test-errno.c (do_test): Initialize setsockopt optlen.
	* sysdeps/unix/sysv/linux/test-errno.c: Move to ...
	* sysdeps/unix/sysv/linux/test-errno-linux.c: ... here.
	(do_test): Handle mlock return on 64 bits kernels with 32 bits
	binaries.
---
 ChangeLog                                  |   8 ++
 posix/test-errno.c                         |   2 +-
 sysdeps/unix/sysv/linux/Makefile           |   2 +-
 sysdeps/unix/sysv/linux/test-errno-linux.c | 176 +++++++++++++++++++++++++++++
 sysdeps/unix/sysv/linux/test-errno.c       | 142 -----------------------
 5 files changed, 186 insertions(+), 144 deletions(-)
 create mode 100644 sysdeps/unix/sysv/linux/test-errno-linux.c
 delete mode 100644 sysdeps/unix/sysv/linux/test-errno.c

-- 
2.7.4

Comments

Stefan Liebler March 13, 2017, 4:13 p.m. UTC | #1 
On 03/13/2017 03:21 PM, Adhemerval Zanella wrote:
> This patch fixes multiple issues of test-errno.c (9a56f8718341):

>

>   - Rename Linux test-errno.c to test-errno-linux.c to avoid build

>     the same source for both tests.

>

>   - Add a mlock check for 32 bits build running on 64 bits kernels.

>     Althuough man pages states that mlock fails with EINVAL if final

>     address overflows, kernels does not return it for aforementioned

>     condition (it returns ENOMEM instead).  Although it seems to be

>     a kernel issue for compat syscall handling, I think it is worth

>     to still check syscall return and document the behavior.

>

>   - Initialize option lenght for setsockopt check.

>

> Checked on x86_64-linux-gnu and i686-linux-gnu (running on 64 bits

> kernel).

>

> 	* posix/test-errno.c (do_test): Initialize setsockopt optlen.

> 	* sysdeps/unix/sysv/linux/test-errno.c: Move to ...

> 	* sysdeps/unix/sysv/linux/test-errno-linux.c: ... here.

> 	(do_test): Handle mlock return on 64 bits kernels with 32 bits

> 	binaries.


Checked on s390x (64bit) and s390 (31bit - running on a 64bit kernel).
Thanks.
Wainer dos Santos Moschetta March 13, 2017, 9:12 p.m. UTC | #2 
On 03/13/2017 11:21 AM, Adhemerval Zanella wrote:
> This patch fixes multiple issues of test-errno.c (9a56f8718341):

>

>   - Rename Linux test-errno.c to test-errno-linux.c to avoid build

>     the same source for both tests.

>

>   - Add a mlock check for 32 bits build running on 64 bits kernels.

>     Althuough man pages states that mlock fails with EINVAL if final

>     address overflows, kernels does not return it for aforementioned

>     condition (it returns ENOMEM instead).  Although it seems to be

>     a kernel issue for compat syscall handling, I think it is worth

>     to still check syscall return and document the behavior.

>

>   - Initialize option lenght for setsockopt check.

>

> Checked on x86_64-linux-gnu and i686-linux-gnu (running on 64 bits

> kernel).

>

> 	* posix/test-errno.c (do_test): Initialize setsockopt optlen.

> 	* sysdeps/unix/sysv/linux/test-errno.c: Move to ...

> 	* sysdeps/unix/sysv/linux/test-errno-linux.c: ... here.

> 	(do_test): Handle mlock return on 64 bits kernels with 32 bits

> 	binaries.

With these changes test-errno passes in ppc64le and ppc32. It fails in ppc64 (big endian) with:
FAIL: open: errno is: 20 (Not a directory) expected: 22 (Invalid argument)

Nonetheless, it is OK if you commit these changes since they fix the test for other architectures. I'll investigate the failure in ppc64 and propose a further change later if applicable.
Adhemerval Zanella March 14, 2017, 5:10 p.m. UTC | #3 
On 13/03/2017 18:12, Wainer dos Santos Moschetta wrote:
> On 03/13/2017 11:21 AM, Adhemerval Zanella wrote:

>> This patch fixes multiple issues of test-errno.c (9a56f8718341):

>>

>>   - Rename Linux test-errno.c to test-errno-linux.c to avoid build

>>     the same source for both tests.

>>

>>   - Add a mlock check for 32 bits build running on 64 bits kernels.

>>     Althuough man pages states that mlock fails with EINVAL if final

>>     address overflows, kernels does not return it for aforementioned

>>     condition (it returns ENOMEM instead).  Although it seems to be

>>     a kernel issue for compat syscall handling, I think it is worth

>>     to still check syscall return and document the behavior.

>>

>>   - Initialize option lenght for setsockopt check.

>>

>> Checked on x86_64-linux-gnu and i686-linux-gnu (running on 64 bits

>> kernel).

>>

>> 	* posix/test-errno.c (do_test): Initialize setsockopt optlen.

>> 	* sysdeps/unix/sysv/linux/test-errno.c: Move to ...

>> 	* sysdeps/unix/sysv/linux/test-errno-linux.c: ... here.

>> 	(do_test): Handle mlock return on 64 bits kernels with 32 bits

>> 	binaries.

> With these changes test-errno passes in ppc64le and ppc32. It fails in ppc64 (big endian) with:

> FAIL: open: errno is: 20 (Not a directory) expected: 22 (Invalid argument)


The test is passing all possible flags on open and kernel had some change
on flags handling over the versions.  For instance, for 3.2 build_open_flags
does not return EINVAL.

So I think it might be the case where the flags combination plus specific
kernel version is not triggering EINVAL with all possible flag combination.
A more simple test should trigger an expected errno regardless, so I am
thinking about replacing the open testcase with:

fails |= test_wrp (EISDIR, open, "/bin", EISDIR, O_WRONLY);

> 

> Nonetheless, it is OK if you commit these changes since they fix the test for other architectures. I'll investigate the failure in ppc64 and propose a further change later if applicable.

>
Wainer dos Santos Moschetta March 14, 2017, 6:26 p.m. UTC | #4 
On 03/14/2017 02:10 PM, Adhemerval Zanella wrote:
>

> On 13/03/2017 18:12, Wainer dos Santos Moschetta wrote:

>> On 03/13/2017 11:21 AM, Adhemerval Zanella wrote:

>>> This patch fixes multiple issues of test-errno.c (9a56f8718341):

>>>

>>>   - Rename Linux test-errno.c to test-errno-linux.c to avoid build

>>>     the same source for both tests.

>>>

>>>   - Add a mlock check for 32 bits build running on 64 bits kernels.

>>>     Althuough man pages states that mlock fails with EINVAL if final

>>>     address overflows, kernels does not return it for aforementioned

>>>     condition (it returns ENOMEM instead).  Although it seems to be

>>>     a kernel issue for compat syscall handling, I think it is worth

>>>     to still check syscall return and document the behavior.

>>>

>>>   - Initialize option lenght for setsockopt check.

>>>

>>> Checked on x86_64-linux-gnu and i686-linux-gnu (running on 64 bits

>>> kernel).

>>>

>>> 	* posix/test-errno.c (do_test): Initialize setsockopt optlen.

>>> 	* sysdeps/unix/sysv/linux/test-errno.c: Move to ...

>>> 	* sysdeps/unix/sysv/linux/test-errno-linux.c: ... here.

>>> 	(do_test): Handle mlock return on 64 bits kernels with 32 bits

>>> 	binaries.

>> With these changes test-errno passes in ppc64le and ppc32. It fails in ppc64 (big endian) with:

>> FAIL: open: errno is: 20 (Not a directory) expected: 22 (Invalid argument)

> The test is passing all possible flags on open and kernel had some change

> on flags handling over the versions.  For instance, for 3.2 build_open_flags

> does not return EINVAL.

>

> So I think it might be the case where the flags combination plus specific

> kernel version is not triggering EINVAL with all possible flag combination.

> A more simple test should trigger an expected errno regardless, so I am

> thinking about replacing the open testcase with:

>

> fails |= test_wrp (EISDIR, open, "/bin", EISDIR, O_WRONLY);


The open testcase with above change passes in ppc64le, ppc64 and ppc32.
Mike Frysinger March 14, 2017, 8:13 p.m. UTC | #5 
On 13 Mar 2017 11:21, Adhemerval Zanella wrote:
> This patch fixes multiple issues of test-errno.c (9a56f8718341):

> 

>   - Rename Linux test-errno.c to test-errno-linux.c to avoid build

>     the same source for both tests.

> 

>   - Add a mlock check for 32 bits build running on 64 bits kernels.

>     Althuough man pages states that mlock fails with EINVAL if final

>     address overflows, kernels does not return it for aforementioned

>     condition (it returns ENOMEM instead).  Although it seems to be

>     a kernel issue for compat syscall handling, I think it is worth

>     to still check syscall return and document the behavior.

> 

>   - Initialize option lenght for setsockopt check.


"length"

>  ChangeLog                                  |   8 ++

>  posix/test-errno.c                         |   2 +-

>  sysdeps/unix/sysv/linux/Makefile           |   2 +-

>  sysdeps/unix/sysv/linux/test-errno-linux.c | 176 +++++++++++++++++++++++++++++

>  sysdeps/unix/sysv/linux/test-errno.c       | 142 -----------------------

>  5 files changed, 186 insertions(+), 144 deletions(-)

>  create mode 100644 sysdeps/unix/sysv/linux/test-errno-linux.c

>  delete mode 100644 sysdeps/unix/sysv/linux/test-errno.c


if you use -C -M with send-email, then the diff below would show
a rename+changes instead of one large delete and one large create.
as-is, it's hard to review.

i use this in my ~/.gitconfig:
[alias]
	fp = format-patch -C -M
	se = send-email -C -M

this way `git se` will always include -C -M for me.
-mike
Adhemerval Zanella March 14, 2017, 8:22 p.m. UTC | #6 
On 14/03/2017 17:13, Mike Frysinger wrote:
> On 13 Mar 2017 11:21, Adhemerval Zanella wrote:

>> This patch fixes multiple issues of test-errno.c (9a56f8718341):

>>

>>   - Rename Linux test-errno.c to test-errno-linux.c to avoid build

>>     the same source for both tests.

>>

>>   - Add a mlock check for 32 bits build running on 64 bits kernels.

>>     Althuough man pages states that mlock fails with EINVAL if final

>>     address overflows, kernels does not return it for aforementioned

>>     condition (it returns ENOMEM instead).  Although it seems to be

>>     a kernel issue for compat syscall handling, I think it is worth

>>     to still check syscall return and document the behavior.

>>

>>   - Initialize option lenght for setsockopt check.

> 

> "length"

> 

>>  ChangeLog                                  |   8 ++

>>  posix/test-errno.c                         |   2 +-

>>  sysdeps/unix/sysv/linux/Makefile           |   2 +-

>>  sysdeps/unix/sysv/linux/test-errno-linux.c | 176 +++++++++++++++++++++++++++++

>>  sysdeps/unix/sysv/linux/test-errno.c       | 142 -----------------------

>>  5 files changed, 186 insertions(+), 144 deletions(-)

>>  create mode 100644 sysdeps/unix/sysv/linux/test-errno-linux.c

>>  delete mode 100644 sysdeps/unix/sysv/linux/test-errno.c

> 

> if you use -C -M with send-email, then the diff below would show

> a rename+changes instead of one large delete and one large create.

> as-is, it's hard to review.

> 

> i use this in my ~/.gitconfig:

> [alias]

> 	fp = format-patch -C -M

> 	se = send-email -C -M

> 

> this way `git se` will always include -C -M for me.

> -mike

> 


Thanks, I will send a v2 for this patch and I will make sure to use
these options.
diff mbox

Patch

diff --git a/posix/test-errno.c b/posix/test-errno.c
index 98df344..9c6c533 100644
--- a/posix/test-errno.c
+++ b/posix/test-errno.c
@@ -142,7 +142,7 @@  do_test (void)
   fails |= test_wrp (EBADF, send, -1, buf, 1, 0);
   fails |= test_wrp (EBADF, sendmsg, -1, &msg, 0);
   fails |= test_wrp (EBADF, sendto, -1, buf, 1, 0, &sa, sl);
-  fails |= test_wrp (EBADF, setsockopt, -1, 0, 0, buf, sl);
+  fails |= test_wrp (EBADF, setsockopt, -1, 0, 0, buf, sizeof (*buf));
   fails |= test_wrp (EBADF, shutdown, -1, SHUT_RD);
   fails |= test_wrp (EBADF, write, -1, "Hello", sizeof ("Hello") );
   fails |= test_wrp (EBADF, writev, -1, iov, 1 );
diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile
index 6b7aa3f..1872cdb 100644
--- a/sysdeps/unix/sysv/linux/Makefile
+++ b/sysdeps/unix/sysv/linux/Makefile
@@ -43,7 +43,7 @@  sysdep_headers += sys/mount.h sys/acct.h sys/sysctl.h \
 		  bits/mman-linux.h
 
 tests += tst-clone tst-clone2 tst-fanotify tst-personality tst-quota \
-	 tst-sync_file_range test-errno
+	 tst-sync_file_range test-errno-linux
 
 # Generate the list of SYS_* macros for the system calls (__NR_* macros).
 
diff --git a/sysdeps/unix/sysv/linux/test-errno-linux.c b/sysdeps/unix/sysv/linux/test-errno-linux.c
new file mode 100644
index 0000000..b00d14e
--- /dev/null
+++ b/sysdeps/unix/sysv/linux/test-errno-linux.c
@@ -0,0 +1,176 @@ 
+/* Test that failing system calls do set errno to the correct value.
+   Linux sycalls version.
+
+   Copyright (C) 2017 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <http://www.gnu.org/licenses/>.  */
+
+#include <errno.h>
+#include <fcntl.h>
+#include <mqueue.h>
+#include <sched.h>
+#include <signal.h>
+#include <stdio.h>
+#include <string.h>
+#include <time.h>
+#include <unistd.h>
+#include <sys/epoll.h>
+#include <sys/eventfd.h>
+#include <sys/file.h>
+#include <sys/fsuid.h>
+#include <sys/inotify.h>
+#include <sys/mman.h>
+#include <sys/poll.h>
+#include <sys/quota.h>
+#include <sys/resource.h>
+#include <sys/select.h>
+#include <sys/sendfile.h>
+#include <sys/swap.h>
+#include <sys/time.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+
+/* This is not an exhaustive test: only system calls that can be
+   persuaded to fail with a consistent error code and no side effects
+   are included.  Usually these are failures due to invalid arguments,
+   with errno code EBADF or EINVAL.  The order of argument checks is
+   unspecified, so we must take care to provide arguments that only
+   allow _one_ failure mode.
+
+   Note that all system calls that can fail with EFAULT are permitted
+   to deliver a SIGSEGV signal instead, so we avoid supplying invalid
+   pointers in general, and we do not attempt to test system calls
+   that can only fail with EFAULT (e.g. gettimeofday, gethostname).
+
+   Also note that root-only system calls (e.g. acct, reboot) may, when
+   the test is run as an unprivileged user, fail due to insufficient
+   privileges before bothering to do argument checks, so those are not
+   tested either.
+
+   Also, system calls that take enum or a set of flags as argument is
+   not tested if POSIX doesn't specify exact binary values for all
+   flags, and so any value passed to flags may become valid.
+
+   Some tests assume "/bin/sh" names a file that exists and is not a
+   directory.  */
+
+#define test_wrp_rv(rtype, prtype, experr, syscall, ...)	\
+  (__extension__ ({						\
+    errno = 0xdead;						\
+    rtype ret = syscall (__VA_ARGS__);				\
+    int err = errno;						\
+    int fail;							\
+    if (ret == (rtype) -1 && err == experr)			\
+      fail = 0;							\
+    else							\
+      {								\
+        fail = 1;						\
+        if (ret != (rtype) -1)					\
+          printf ("FAIL: " #syscall ": didn't fail as expected"	\
+               " (return "prtype")\n", ret);			\
+        else if (err == 0xdead)					\
+          puts("FAIL: " #syscall ": didn't update errno\n");	\
+        else if (err != experr)					\
+          printf ("FAIL: " #syscall				\
+               ": errno is: %d (%s) expected: %d (%s)\n",	\
+               err, strerror (err), experr, strerror (experr));	\
+      }								\
+    fail;							\
+  }))
+
+#define test_wrp_rv2(rtype, prtype, experr1, experr2, syscall, ...) 	\
+  (__extension__ ({							\
+    errno = 0xdead;							\
+    rtype ret = syscall (__VA_ARGS__);					\
+    int err = errno;							\
+    int fail;								\
+    if (ret == (rtype) -1 && (err == experr1 || err == experr2))	\
+      fail = 0;								\
+    else								\
+      {									\
+        fail = 1;							\
+        if (ret != (rtype) -1)						\
+          printf ("FAIL: " #syscall ": didn't fail as expected"		\
+               " (return "prtype")\n", ret);				\
+        else if (err == 0xdead)						\
+          puts("FAIL: " #syscall ": didn't update errno\n");		\
+        else if (err != experr1 && err != experr2)			\
+          printf ("FAIL: " #syscall					\
+               ": errno is: %d (%s) expected: %d (%s) or %d (%s)\n",	\
+               err, strerror (err), experr1, strerror (experr1),	\
+               experr2, strerror (experr2));				\
+      }									\
+    fail;								\
+  }))
+
+#define test_wrp(experr, syscall, ...)				\
+  test_wrp_rv(int, "%d", experr, syscall, __VA_ARGS__)
+
+#define test_wrp2(experr1, experr2, syscall, ...)		\
+  test_wrp_rv2(int, "%d", experr1, experr2, syscall, __VA_ARGS__)
+
+static int
+do_test (void)
+{
+  fd_set rs, ws, es;
+  int status;
+  off_t off;
+  stack_t ss;
+  struct dqblk dqblk;
+  struct epoll_event epoll_event;
+  struct pollfd pollfd;
+  struct sched_param sch_param;
+  struct timespec ts;
+  struct timeval tv;
+  unsigned char vec[16];
+  ss.ss_flags = ~SS_DISABLE;
+  ts.tv_sec = -1;
+
+  int fails = 0;
+  fails |= test_wrp (EINVAL, epoll_create, -1);
+  fails |= test_wrp (EINVAL, epoll_create1, EPOLL_CLOEXEC + 1);
+  fails |= test_wrp (EBADF, epoll_ctl, -1, EPOLL_CTL_ADD, 0, &epoll_event);
+  fails |= test_wrp (EBADF, epoll_wait, -1, &epoll_event, 1, 1);
+  fails |= test_wrp (EBADF, fdatasync, -1);
+  fails |= test_wrp (EBADF, flock, -1, LOCK_SH);
+  fails |= test_wrp (ESRCH, getpgid, -1);
+  fails |= test_wrp (EINVAL, inotify_add_watch, -1, "/", 0);
+  fails |= test_wrp (EINVAL, mincore, (void *) -1, 0, vec);
+  /* mlock fails if the result of the addition addr+len was less than addr
+     (which indicates final address overflow), however on 32 bits binaries
+     running on 64 bits kernels, internal syscall address check won't result
+     in an invalid address and thus syscalls fails later in vma
+     allocation.  */
+  fails |= test_wrp2 (EINVAL, ENOMEM, mlock, (void *) -1, 1);
+  fails |= test_wrp (EINVAL, nanosleep, &ts, &ts);
+  fails |= test_wrp (EINVAL, poll, &pollfd, -1, 0);
+  fails |= test_wrp (ENODEV, quotactl, Q_GETINFO, NULL, -1, (caddr_t) &dqblk);
+  fails |= test_wrp (EINVAL, sched_getparam, -1, &sch_param);
+  fails |= test_wrp (EINVAL, sched_getscheduler, -1);
+  fails |= test_wrp (EINVAL, sched_get_priority_max, -1);
+  fails |= test_wrp (EINVAL, sched_get_priority_min, -1);
+  fails |= test_wrp (EINVAL, sched_rr_get_interval, -1, &ts);
+  fails |= test_wrp (EINVAL, sched_setparam, -1, &sch_param);
+  fails |= test_wrp (EINVAL, sched_setscheduler, -1, 0, &sch_param);
+  fails |= test_wrp (EINVAL, select, -1, &rs, &ws, &es, &tv);
+  fails |= test_wrp (EBADF, sendfile, -1, -1, &off, 0);
+  fails |= test_wrp (EINVAL, sigaltstack, &ss, NULL);
+  fails |= test_wrp (ECHILD, wait4, -1, &status, 0, NULL);
+
+  return fails;
+}
+
+#include "support/test-driver.c"
diff --git a/sysdeps/unix/sysv/linux/test-errno.c b/sysdeps/unix/sysv/linux/test-errno.c
deleted file mode 100644
index ab3735f..0000000
--- a/sysdeps/unix/sysv/linux/test-errno.c
+++ /dev/null
@@ -1,142 +0,0 @@ 
-/* Test that failing system calls do set errno to the correct value.
-
-   Copyright (C) 2017 Free Software Foundation, Inc.
-   This file is part of the GNU C Library.
-
-   The GNU C Library is free software; you can redistribute it and/or
-   modify it under the terms of the GNU Lesser General Public
-   License as published by the Free Software Foundation; either
-   version 2.1 of the License, or (at your option) any later version.
-
-   The GNU C Library is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-   Lesser General Public License for more details.
-
-   You should have received a copy of the GNU Lesser General Public
-   License along with the GNU C Library; if not, see
-   <http://www.gnu.org/licenses/>.  */
-
-#include <errno.h>
-#include <fcntl.h>
-#include <mqueue.h>
-#include <sched.h>
-#include <signal.h>
-#include <stdio.h>
-#include <string.h>
-#include <time.h>
-#include <unistd.h>
-#include <sys/epoll.h>
-#include <sys/eventfd.h>
-#include <sys/file.h>
-#include <sys/fsuid.h>
-#include <sys/inotify.h>
-#include <sys/mman.h>
-#include <sys/poll.h>
-#include <sys/quota.h>
-#include <sys/resource.h>
-#include <sys/select.h>
-#include <sys/sendfile.h>
-#include <sys/swap.h>
-#include <sys/time.h>
-#include <sys/types.h>
-#include <sys/wait.h>
-
-/* This is not an exhaustive test: only system calls that can be
-   persuaded to fail with a consistent error code and no side effects
-   are included.  Usually these are failures due to invalid arguments,
-   with errno code EBADF or EINVAL.  The order of argument checks is
-   unspecified, so we must take care to provide arguments that only
-   allow _one_ failure mode.
-
-   Note that all system calls that can fail with EFAULT are permitted
-   to deliver a SIGSEGV signal instead, so we avoid supplying invalid
-   pointers in general, and we do not attempt to test system calls
-   that can only fail with EFAULT (e.g. gettimeofday, gethostname).
-
-   Also note that root-only system calls (e.g. acct, reboot) may, when
-   the test is run as an unprivileged user, fail due to insufficient
-   privileges before bothering to do argument checks, so those are not
-   tested either.
-
-   Also, system calls that take enum or a set of flags as argument is
-   not tested if POSIX doesn't specify exact binary values for all
-   flags, and so any value passed to flags may become valid.
-
-   Some tests assume "/bin/sh" names a file that exists and is not a
-   directory.  */
-
-#define test_wrp_rv(rtype, prtype, experr, syscall, ...)	\
-  (__extension__ ({						\
-    errno = 0xdead;						\
-    rtype ret = syscall (__VA_ARGS__);				\
-    int err = errno;						\
-    int fail;							\
-    if (ret == (rtype) -1 && err == experr)			\
-      fail = 0;							\
-    else							\
-      {								\
-        fail = 1;						\
-        if (ret != (rtype) -1)					\
-          printf ("FAIL: " #syscall ": didn't fail as expected"	\
-               " (return "prtype")\n", ret);			\
-        else if (err == 0xdead)					\
-          puts("FAIL: " #syscall ": didn't update errno\n");	\
-        else if (err != experr)					\
-          printf ("FAIL: " #syscall				\
-               ": errno is: %d (%s) expected: %d (%s)\n",	\
-               err, strerror (err), experr, strerror (experr));	\
-      }								\
-    fail;							\
-  }))
-
-#define test_wrp(experr, syscall, ...)				\
-  test_wrp_rv(int, "%d", experr, syscall, __VA_ARGS__)
-
-static int
-do_test (void)
-{
-  fd_set rs, ws, es;
-  int status;
-  off_t off;
-  stack_t ss;
-  struct dqblk dqblk;
-  struct epoll_event epoll_event;
-  struct pollfd pollfd;
-  struct sched_param sch_param;
-  struct timespec ts;
-  struct timeval tv;
-  unsigned char vec[16];
-  ss.ss_flags = ~SS_DISABLE;
-  ts.tv_sec = -1;
-
-  int fails = 0;
-  fails |= test_wrp (EINVAL, epoll_create, -1);
-  fails |= test_wrp (EINVAL, epoll_create1, EPOLL_CLOEXEC + 1);
-  fails |= test_wrp (EBADF, epoll_ctl, -1, EPOLL_CTL_ADD, 0, &epoll_event);
-  fails |= test_wrp (EBADF, epoll_wait, -1, &epoll_event, 1, 1);
-  fails |= test_wrp (EBADF, fdatasync, -1);
-  fails |= test_wrp (EBADF, flock, -1, LOCK_SH);
-  fails |= test_wrp (ESRCH, getpgid, -1);
-  fails |= test_wrp (EINVAL, inotify_add_watch, -1, "/", 0);
-  fails |= test_wrp (EINVAL, mincore, (void *) -1, 0, vec);
-  fails |= test_wrp (EINVAL, mlock, (void *) -1, 1); // different errors
-  fails |= test_wrp (EINVAL, nanosleep, &ts, &ts);
-  fails |= test_wrp (EINVAL, poll, &pollfd, -1, 0);
-  fails |= test_wrp (ENODEV, quotactl, Q_GETINFO, NULL, -1, (caddr_t) &dqblk);
-  fails |= test_wrp (EINVAL, sched_getparam, -1, &sch_param);
-  fails |= test_wrp (EINVAL, sched_getscheduler, -1);
-  fails |= test_wrp (EINVAL, sched_get_priority_max, -1);
-  fails |= test_wrp (EINVAL, sched_get_priority_min, -1);
-  fails |= test_wrp (EINVAL, sched_rr_get_interval, -1, &ts);
-  fails |= test_wrp (EINVAL, sched_setparam, -1, &sch_param);
-  fails |= test_wrp (EINVAL, sched_setscheduler, -1, 0, &sch_param);
-  fails |= test_wrp (EINVAL, select, -1, &rs, &ws, &es, &tv);
-  fails |= test_wrp (EBADF, sendfile, -1, -1, &off, 0);
-  fails |= test_wrp (EINVAL, sigaltstack, &ss, NULL);
-  fails |= test_wrp (ECHILD, wait4, -1, &status, 0, NULL);
-
-  return fails;
-}
-
-#include "support/test-driver.c"