This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 7/7] Consolidate wait3 implementations
On Thu, Nov 14, 2019 at 6:47 AM Adhemerval Zanella
<adhemerval.zanella@linaro.org> wrote:
>
> The generic one calls wait4.
>
> Checked on x86_64-linux-gnu.
> ---
Thanks for doing this! This whole series looks good to me.
Alistair
> posix/wait3.c | 14 +++---------
> sysdeps/posix/wait3.c | 39 ---------------------------------
> sysdeps/unix/bsd/wait3.c | 33 ----------------------------
> sysdeps/unix/sysv/linux/wait3.c | 1 -
> 4 files changed, 3 insertions(+), 84 deletions(-)
> delete mode 100644 sysdeps/posix/wait3.c
> delete mode 100644 sysdeps/unix/bsd/wait3.c
> delete mode 100644 sysdeps/unix/sysv/linux/wait3.c
>
> diff --git a/posix/wait3.c b/posix/wait3.c
> index 6b6e3a500d..f0abf24e31 100644
> --- a/posix/wait3.c
> +++ b/posix/wait3.c
> @@ -1,4 +1,5 @@
> -/* Copyright (C) 1991-2019 Free Software Foundation, Inc.
> +/* Wait for process to change state.
> + Copyright (C) 1991-2019 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
> @@ -15,7 +16,6 @@
> License along with the GNU C Library; if not, see
> <https://www.gnu.org/licenses/>. */
>
> -#include <errno.h>
> #include <sys/wait.h>
> #include <sys/types.h>
>
> @@ -27,15 +27,7 @@
> pid_t
> __wait3 (int *stat_loc, int options, struct rusage *usage)
> {
> - if ((options & ~(WNOHANG|WUNTRACED)) != 0)
> - {
> - __set_errno (EINVAL);
> - return (pid_t) -1;
> - }
> -
> - __set_errno (ENOSYS);
> - return (pid_t) -1;
> + return __wait4 (WAIT_ANY, stat_loc, options, usage);
> }
> -stub_warning (wait3)
>
> weak_alias (__wait3, wait3)
> diff --git a/sysdeps/posix/wait3.c b/sysdeps/posix/wait3.c
> deleted file mode 100644
> index daba36c6dc..0000000000
> --- a/sysdeps/posix/wait3.c
> +++ /dev/null
> @@ -1,39 +0,0 @@
> -/* Copyright (C) 1992-2019 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
> - <https://www.gnu.org/licenses/>. */
> -
> -#include <errno.h>
> -#include <sys/wait.h>
> -#include <sys/types.h>
> -#include <stddef.h>
> -
> -/* Wait for a child to exit. When one does, put its status in *STAT_LOC and
> - return its process ID. For errors return (pid_t) -1. If USAGE is not nil,
> - store information about the child's resource usage (as a `struct rusage')
> - there. If the WUNTRACED bit is set in OPTIONS, return status for stopped
> - children; otherwise don't. */
> -pid_t
> -__wait3 (int *stat_loc, int options, struct rusage *usage)
> -{
> - if (usage != NULL)
> - {
> - __set_errno (ENOSYS);
> - return (pid_t) -1;
> - }
> - return __waitpid (WAIT_ANY, stat_loc, options);
> -}
> -
> -weak_alias (__wait3, wait3)
> diff --git a/sysdeps/unix/bsd/wait3.c b/sysdeps/unix/bsd/wait3.c
> deleted file mode 100644
> index d7fa7e4496..0000000000
> --- a/sysdeps/unix/bsd/wait3.c
> +++ /dev/null
> @@ -1,33 +0,0 @@
> -/* Copyright (C) 1991-2019 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
> - <https://www.gnu.org/licenses/>. */
> -
> -#include <errno.h>
> -#include <sys/wait.h>
> -#include <sys/types.h>
> -
> -/* Wait for a child to exit. When one does, put its status in *STAT_LOC and
> - return its process ID. For errors return (pid_t) -1. If USAGE is not nil,
> - store information about the child's resource usage (as a `struct rusage')
> - there. If the WUNTRACED bit is set in OPTIONS, return status for stopped
> - children; otherwise don't. */
> -pid_t
> -__wait3 (int *stat_loc, int options, struct rusage *usage)
> -{
> - return __wait4 (WAIT_ANY, stat_loc, options, usage);
> -}
> -
> -weak_alias (__wait3, wait3)
> diff --git a/sysdeps/unix/sysv/linux/wait3.c b/sysdeps/unix/sysv/linux/wait3.c
> deleted file mode 100644
> index 2ff027f0e1..0000000000
> --- a/sysdeps/unix/sysv/linux/wait3.c
> +++ /dev/null
> @@ -1 +0,0 @@
> -#include <sysdeps/unix/bsd/wait3.c>
> --
> 2.17.1
>