diff mbox series

Linux: Remove exit system call from _exit

Message ID 87o7x1b9y7.fsf@oldenburg.str.redhat.com
State New
Headers show
Series Linux: Remove exit system call from _exit | expand

Commit Message

Florian Weimer Aug. 3, 2022, 3:53 p.m. UTC 
exit only terminates the current thread, not the whole process, so it
is the wrong fallback system call in this context.  All supported
Linux versions implement the exit_group system call anyway.

---
 sysdeps/unix/sysv/linux/_exit.c | 1 -
 1 file changed, 1 deletion(-)

Comments

H.J. Lu Aug. 3, 2022, 4:58 p.m. UTC | #1 
On Wed, Aug 3, 2022 at 8:53 AM Florian Weimer via Libc-alpha
<libc-alpha@sourceware.org> wrote:
>
> exit only terminates the current thread, not the whole process, so it
> is the wrong fallback system call in this context.  All supported
> Linux versions implement the exit_group system call anyway.
>
> ---
>  sysdeps/unix/sysv/linux/_exit.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/sysdeps/unix/sysv/linux/_exit.c b/sysdeps/unix/sysv/linux/_exit.c
> index 7ad3e0aae4..f916d97ae2 100644
> --- a/sysdeps/unix/sysv/linux/_exit.c
> +++ b/sysdeps/unix/sysv/linux/_exit.c
> @@ -28,7 +28,6 @@ _exit (int status)
>    while (1)
>      {
>        INLINE_SYSCALL (exit_group, 1, status);
> -      INLINE_SYSCALL (exit, 1, status);
>
>  #ifdef ABORT_INSTRUCTION
>        ABORT_INSTRUCTION;
>

LGTM.

Thanks.
diff mbox series

Patch

diff --git a/sysdeps/unix/sysv/linux/_exit.c b/sysdeps/unix/sysv/linux/_exit.c
index 7ad3e0aae4..f916d97ae2 100644
--- a/sysdeps/unix/sysv/linux/_exit.c
+++ b/sysdeps/unix/sysv/linux/_exit.c
@@ -28,7 +28,6 @@  _exit (int status)
   while (1)
     {
       INLINE_SYSCALL (exit_group, 1, status);
-      INLINE_SYSCALL (exit, 1, status);
 
 #ifdef ABORT_INSTRUCTION
       ABORT_INSTRUCTION;