From patchwork Fri Jun 23 12:37:30 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella X-Patchwork-Id: 106262 Delivered-To: patch@linaro.org Received: by 10.140.91.2 with SMTP id y2csp184878qgd; Fri, 23 Jun 2017 05:37:59 -0700 (PDT) X-Received: by 10.99.0.65 with SMTP id 62mr1711150pga.115.1498221478960; Fri, 23 Jun 2017 05:37:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1498221478; cv=none; d=google.com; s=arc-20160816; b=qscYNNbjZ9Q0e+g5kSxk3S0axiHbxlKowyhNS11cV/9o7UWbKH1OwkYIgo6ICBSq+C glry7jmOFmqqpid7+Jgk85FOFocloX9d1rzS536Sn3YQVptHi18bQsWNLFLnqs/vZGg2 47l9R9PxkRsTRj2wVkSKKn3gKR4nGUF0kwnddBKZW2/aQLGy3pG9bQIB8HRNbo4CzwTN uiYX8fMNYdO2EsCDhwB20DQWnivyTcsyPEKBTNEa2ylh9QVOm9lRe0xdq1pTAOcCj/87 C4b0Gl7n80no5wYKV1tv0Tp7JiPYGQe97AuzIU0sWE3FxUA1q0Yr6DxQCVpNzNcIBCG0 SZog== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=message-id:date:subject:to:from:delivered-to:sender:list-help :list-post:list-archive:list-subscribe:list-unsubscribe:list-id :precedence:mailing-list:dkim-signature:domainkey-signature :arc-authentication-results; bh=bjBDKd3zOyl5NZqqbmhZZbluE6q3vJnThmGyJHif2xE=; b=I8m/Gmj3BAtObWSz41d9tUWRpqqNI2SgPfgcyLhLe6F8FfpJ6N62u7BWAEtglN6Y7d DIfJpF+TGj6IbKfBjfMOJzzox1sxR6eYQN/thzyt7MH6ZO0ZuWNdAc2i7nvS85zMbQGR f8xVOGD8cnzNOIiNPMeRmtFMEJB1alJGA4gpdnjrHi9TlzAMvDDP+WAUTouypnomPfeX o7B0xfi//CJFBDZUExYZvi6BdLYwlCGQ+Yc+Rwz0MJczm1nj6NQKV2TioyvtL9N+Z4Rq zu7unoRKR9oZv1oQ6lMIEPleJ/cD7UvWUM2kEa6GcSeZRdCX/S6GAmlEnaznjio/97aC 5ojg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sourceware.org header.b=UW5UT31m; spf=pass (google.com: domain of libc-alpha-return-81068-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom=libc-alpha-return-81068-patch=linaro.org@sourceware.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from sourceware.org (server1.sourceware.org. [209.132.180.131]) by mx.google.com with ESMTPS id 1si3513406plw.319.2017.06.23.05.37.58 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 23 Jun 2017 05:37:58 -0700 (PDT) Received-SPF: pass (google.com: domain of libc-alpha-return-81068-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) client-ip=209.132.180.131; Authentication-Results: mx.google.com; dkim=pass header.i=@sourceware.org header.b=UW5UT31m; spf=pass (google.com: domain of libc-alpha-return-81068-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom=libc-alpha-return-81068-patch=linaro.org@sourceware.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:subject:date:message-id; q=dns; s= default; b=qtIxDTbWWVmBDMBKc7YOBwy9pLJZl4sTiEhfBeGcG4zH++R7Iea1a EWeU5Jf27/qlMflvidUIbcCJN21a2466mAYph4oH6WfcwjR/TmjDwXhnHdqS22UU kzYxxxNTNKY4xE5AACJFOs5vRRITwboQIdO9MgcjhDLnF5Q9pxBZ+8= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:subject:date:message-id; s=default; bh=p2vVkOUUuroxL54fcwQTWp7yz2Q=; b=UW5UT31mf/fVl1Hy1vNz5wFTwYeb xBin+WsPJiiVp+PQl/i6A6ztRvNFBS2BxQ4x4ic5FZKsMrqrrFwSDV0UlH/Z0Ysy SiiBltnwdlia6f0Mvl+RD5I1bUtkRh8t7tb0hak6lF2nmt6Am8+8im1zHUboWTdR w/zLVrgdW6zgasI= Received: (qmail 70957 invoked by alias); 23 Jun 2017 12:37:43 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 70935 invoked by uid 89); 23 Jun 2017 12:37:41 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.2 spammy=1688 X-HELO: mail-qk0-f176.google.com X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=bjBDKd3zOyl5NZqqbmhZZbluE6q3vJnThmGyJHif2xE=; b=aBlYByrFFzqHVYKDV+F4viGmSQ4lMTweQ/Fr4KSrWgCYjo5cO2g6Dwv4bOcN4SMeFw ZnOekLh7rlJ+C3vvoARMOeVv12O1E4HTE/Z6jHnKzLzz6aYJg/gKhElHeARNQCmAWnMk XrR8eBfhjh0K0wHH8z0eJ0pNOnxQH3/To+b5Satf78Y8H9659+33DbtzczvSR/aQyKNn /khK60Jpi7HTnUvWYGdP7OHtqkIhmTUhdy++7huxLULBSSfusAMrKTCDQnA/CLSQBwDb b0sY3lFuBoRvK9f4ZT9xiW/TdaFS0DkIdhFzOVZ08cSlwwq/Isr8k7CETC5jui8PtwHK STLw== X-Gm-Message-State: AKS2vOw90zzh5HlEO8qPWGXIPSIwOCsmpfRb5oUE8cpshlqMqkL6jbUY XvXVkzqH3h42d1kYbQPO9Q== X-Received: by 10.55.31.162 with SMTP id n34mr8350417qkh.46.1498221455242; Fri, 23 Jun 2017 05:37:35 -0700 (PDT) From: Adhemerval Zanella To: libc-alpha@sourceware.org Subject: [PATCH] Call exit directly in clone (BZ #21512) Date: Fri, 23 Jun 2017 09:37:30 -0300 Message-Id: <1498221450-8381-1-git-send-email-adhemerval.zanella@linaro.org> On aarch64, alpha, arm, hppa, mips, nios2, powerpc, s390, sh, sparch, tile, and x86_64 the clone syscall jumps to _exit after the child execution and the function ends the process execution by calling exit_group. This behavior have a small issue where threads created with CLONE_THREAD using clone syscall directly will eventually exit the whole group altogether instead of just the thread created. Also, microblaze, ia64, i386, and m68k differs by calling exit syscall directly. This patch changes all architectures to call the exit syscall directly, as for microblaze, ia64, i386, and m68k. This do not have change glibc internal behavior in any sort, since the only usage of clone implementation in posix_spawn calls _exit directly in the created child (fork uses a direct call to clone). Checked on x86_64-linux-gnu, i686-linux-gnu, aarch64-linux-gnu, powerpc-linux-gnu, powerpc64le-linux-gnu, sparc64-linux-gnu, and sparcv9-linux-gnu. [BZ #21512] * sysdeps/unix/sysv/linux/aarch64/clone.S (__clone): Call exit syscall instead of jump to _exit. (CLONE_VM_BIT): Remove unused define. (CLONE_VM): Likewise. (CLONE_THREAD_BIT): Likewise. (CLONE_THREAD): Likewise. * sysdeps/unix/sysv/linux/alpha/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. * sysdeps/unix/sysv/linux/arm/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. (CLONE_THREAD): Likewise. * sysdeps/unix/sysv/linux/i386/clone.S (CLONE_VM): Likewise. * sysdeps/unix/sysv/linux/ia64/clone2.S (__clone2): Call exit syscall instead of jump to _exit. * sysdeps/unix/sysv/linux/hppa/clone.S (__clone): Likewise. * sysdeps/unix/sysv/linux/mips/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. (CLONE_THREAD): Likewise. * sysdeps/unix/sysv/linux/nios2/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. * sysdeps/unix/sysv/linux/powerpc/powerpc32/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. (CLONE_THREAD): Likewise. * sysdeps/unix/sysv/linux/powerpc/powerpc64/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. (CLONE_THREAD): Likewise. * sysdeps/unix/sysv/linux/s390/s390-32/clone.S (__clone): Likewise. * sysdeps/unix/sysv/linux/s390/s390-64/clone.S (__clone): Likewise. * sysdeps/unix/sysv/linux/sh/clone.S (__clone): Likewise. * sysdeps/unix/sysv/linux/sparc/sparc32/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. * sysdeps/unix/sysv/linux/sparc/sparc64/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. * sysdeps/unix/sysv/linux/tile/clone.S (__clone): Likewise. * sysdeps/unix/sysv/linux/x86_64/clone.S (__clone): Likewise. (CLONE_VM): Remove unused define. * sysdeps/unix/sysv/linux/Makefile (tests): Add tst-clone3. * sysdeps/unix/sysv/linux/tst-clone3.c: New file. --- sysdeps/unix/sysv/linux/Makefile | 4 +- sysdeps/unix/sysv/linux/aarch64/clone.S | 9 +-- sysdeps/unix/sysv/linux/alpha/clone.S | 11 +-- sysdeps/unix/sysv/linux/arm/clone.S | 6 +- sysdeps/unix/sysv/linux/hppa/clone.S | 6 +- sysdeps/unix/sysv/linux/i386/clone.S | 2 - sysdeps/unix/sysv/linux/ia64/clone2.S | 8 +- sysdeps/unix/sysv/linux/m68k/clone.S | 2 - sysdeps/unix/sysv/linux/mips/clone.S | 12 +-- sysdeps/unix/sysv/linux/nios2/clone.S | 17 +--- sysdeps/unix/sysv/linux/powerpc/powerpc32/clone.S | 7 +- sysdeps/unix/sysv/linux/powerpc/powerpc64/clone.S | 11 +-- sysdeps/unix/sysv/linux/s390/s390-32/clone.S | 2 +- sysdeps/unix/sysv/linux/s390/s390-64/clone.S | 2 +- sysdeps/unix/sysv/linux/sh/clone.S | 21 +---- sysdeps/unix/sysv/linux/sparc/sparc32/clone.S | 5 +- sysdeps/unix/sysv/linux/sparc/sparc64/clone.S | 5 +- sysdeps/unix/sysv/linux/tile/clone.S | 6 +- sysdeps/unix/sysv/linux/tst-clone3.c | 95 +++++++++++++++++++++++ sysdeps/unix/sysv/linux/x86_64/clone.S | 5 +- 20 files changed, 130 insertions(+), 106 deletions(-) create mode 100644 sysdeps/unix/sysv/linux/tst-clone3.c -- 2.7.4 diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile index 8b340d4..9d6a2de 100644 --- a/sysdeps/unix/sysv/linux/Makefile +++ b/sysdeps/unix/sysv/linux/Makefile @@ -49,8 +49,8 @@ sysdep_headers += sys/mount.h sys/acct.h sys/sysctl.h \ bits/mman-linux.h \ bits/siginfo-arch.h bits/siginfo-consts-arch.h -tests += tst-clone tst-clone2 tst-fanotify tst-personality tst-quota \ - tst-sync_file_range test-errno-linux +tests += tst-clone tst-clone2 tst-clone3 tst-fanotify tst-personality \ + tst-quota 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/aarch64/clone.S b/sysdeps/unix/sysv/linux/aarch64/clone.S index 259ec07..905915a 100644 --- a/sysdeps/unix/sysv/linux/aarch64/clone.S +++ b/sysdeps/unix/sysv/linux/aarch64/clone.S @@ -23,12 +23,6 @@ #define _ERRNO_H 1 #include -#define CLONE_VM_BIT 8 -#define CLONE_VM (1 << CLONE_VM_BIT) - -#define CLONE_THREAD_BIT 16 -#define CLONE_THREAD (1 << CLONE_THREAD_BIT) - /* int clone(int (*fn)(void *arg), x0 void *child_stack, x1 int flags, x2 @@ -84,7 +78,8 @@ thread_start: blr x10 /* We are done, pass the return value through x0. */ - b HIDDEN_JUMPTARGET(_exit) + mov x8, #SYS_ify(exit) + svc 0x0 cfi_endproc .size thread_start, .-thread_start diff --git a/sysdeps/unix/sysv/linux/alpha/clone.S b/sysdeps/unix/sysv/linux/alpha/clone.S index 20ae361..550461f 100644 --- a/sysdeps/unix/sysv/linux/alpha/clone.S +++ b/sysdeps/unix/sysv/linux/alpha/clone.S @@ -23,8 +23,6 @@ #define _ERRNO_H 1 #include -#define CLONE_VM 0x00000100 - /* int clone(int (*fn)(void *arg), void *child_stack, int flags, void *arg, pid_t *ptid, void *tls, pid_t *ctid); @@ -100,13 +98,8 @@ thread_start: jsr ra, (pv) ldgp gp, 0(ra) - /* Call _exit rather than doing it inline for breakpoint purposes. */ - mov v0, a0 -#ifdef PIC - bsr ra, HIDDEN_JUMPTARGET(_exit) !samegp -#else - jsr ra, HIDDEN_JUMPTARGET(_exit) -#endif + ldiq v0, __NR_exit + call_pal PAL_callsys /* Die horribly. */ .align 4 diff --git a/sysdeps/unix/sysv/linux/arm/clone.S b/sysdeps/unix/sysv/linux/arm/clone.S index a309add..f01968a 100644 --- a/sysdeps/unix/sysv/linux/arm/clone.S +++ b/sysdeps/unix/sysv/linux/arm/clone.S @@ -24,9 +24,6 @@ #define _ERRNO_H 1 #include -#define CLONE_VM 0x00000100 -#define CLONE_THREAD 0x00010000 - /* int clone(int (*fn)(void *arg), void *child_stack, int flags, void *arg, pid_t *ptid, struct user_desc *tls, pid_t *ctid); */ @@ -76,7 +73,8 @@ PSEUDO_END (__clone) BLX (ip) @ and we are done, passing the return value through r0 - b PLTJMP(HIDDEN_JUMPTARGET(_exit)) + ldr r7, =SYS_ify(clone) + swi 0x0 .fnend diff --git a/sysdeps/unix/sysv/linux/hppa/clone.S b/sysdeps/unix/sysv/linux/hppa/clone.S index d36b302..8c43944 100644 --- a/sysdeps/unix/sysv/linux/hppa/clone.S +++ b/sysdeps/unix/sysv/linux/hppa/clone.S @@ -148,10 +148,10 @@ ENTRY(__clone) copy %r4, %r19 #endif /* The call to _exit needs saved r19. */ - bl _exit, %rp - copy %ret0, %arg0 + ble 0x100(%sr2, %r0) + ldi __NR_exit, %r20 - /* We should not return from _exit. + /* We should not return from exit. We do not restore r4, or the stack state. */ iitlbp %r0, (%sr0, %r0) diff --git a/sysdeps/unix/sysv/linux/i386/clone.S b/sysdeps/unix/sysv/linux/i386/clone.S index a4ba3e2..49c82d9 100644 --- a/sysdeps/unix/sysv/linux/i386/clone.S +++ b/sysdeps/unix/sysv/linux/i386/clone.S @@ -39,8 +39,6 @@ #define __NR_clone 120 #define SYS_clone 120 -#define CLONE_VM 0x00000100 - .text ENTRY (__clone) /* Sanity check arguments. */ diff --git a/sysdeps/unix/sysv/linux/ia64/clone2.S b/sysdeps/unix/sysv/linux/ia64/clone2.S index 9b59473..3157ce9 100644 --- a/sysdeps/unix/sysv/linux/ia64/clone2.S +++ b/sysdeps/unix/sysv/linux/ia64/clone2.S @@ -74,11 +74,11 @@ ENTRY(__clone2) mov b6=out1 br.call.dptk.many rp=b6 /* Call fn(arg) in the child */ ;; - mov out0=r8 /* Argument to _exit */ + mov out0=r8 /* Argument to exit */ mov gp=loc0 - .globl HIDDEN_JUMPTARGET(_exit) - br.call.dpnt.many rp=HIDDEN_JUMPTARGET(_exit) - /* call _exit with result from fn. */ + mov r15=SYS_ify (exit) + .save rp, r0 + break __BREAK_SYSCALL ret /* Not reached. */ PSEUDO_END(__clone2) diff --git a/sysdeps/unix/sysv/linux/m68k/clone.S b/sysdeps/unix/sysv/linux/m68k/clone.S index a680191..0894b2a 100644 --- a/sysdeps/unix/sysv/linux/m68k/clone.S +++ b/sysdeps/unix/sysv/linux/m68k/clone.S @@ -24,8 +24,6 @@ #include #include -#define CLONE_VM 0x00000100 - /* int clone(int (*fn)(void *arg), void *child_stack, int flags, void *arg, void *parent_tidptr, void *tls, void *child_tidptr) */ diff --git a/sysdeps/unix/sysv/linux/mips/clone.S b/sysdeps/unix/sysv/linux/mips/clone.S index 8b79457..855f972 100644 --- a/sysdeps/unix/sysv/linux/mips/clone.S +++ b/sysdeps/unix/sysv/linux/mips/clone.S @@ -25,9 +25,6 @@ #include #include -#define CLONE_VM 0x00000100 -#define CLONE_THREAD 0x00010000 - /* int clone(int (*fn)(void *arg), void *child_stack, int flags, void *arg, void *parent_tidptr, void *tls, void *child_tidptr) */ @@ -137,14 +134,9 @@ L(thread_start): /* Call the user's function. */ jal t9 - /* Call _exit rather than doing it inline for breakpoint purposes. */ move a0,v0 -#ifdef __PIC__ - PTR_LA t9,_exit - jalr t9 -#else - jal _exit -#endif + li v0,__NR_clone + syscall END(__thread_start) diff --git a/sysdeps/unix/sysv/linux/nios2/clone.S b/sysdeps/unix/sysv/linux/nios2/clone.S index 7929dfa..2ba8258 100644 --- a/sysdeps/unix/sysv/linux/nios2/clone.S +++ b/sysdeps/unix/sysv/linux/nios2/clone.S @@ -25,8 +25,6 @@ #include #include -#define CLONE_VM 0x00000100 - /* int clone(int (*fn)(void *arg), void *child_stack, int flags, void *arg, void *parent_tidptr, void *tls, void *child_tidptr) */ @@ -75,18 +73,9 @@ thread_start: /* Call the user's function. */ callr r5 - /* _exit with the result. */ - mov r4, r2 -#ifdef PIC - nextpc r22 -1: movhi r8, %hiadj(_gp_got - 1b) - addi r8, r8, %lo(_gp_got - 1b) - add r22, r22, r8 - ldw r8, %call(HIDDEN_JUMPTARGET(_exit))(r22) - jmp r8 -#else - jmpi _exit -#endif + /* exit with the result. */ + movi r2, SYS_ify (exit) + trap cfi_endproc cfi_startproc diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc32/clone.S b/sysdeps/unix/sysv/linux/powerpc/powerpc32/clone.S index a07b7d3..e48cc5f 100644 --- a/sysdeps/unix/sysv/linux/powerpc/powerpc32/clone.S +++ b/sysdeps/unix/sysv/linux/powerpc/powerpc32/clone.S @@ -20,10 +20,6 @@ #define _ERRNO_H 1 #include -#define CLONE_VM 0x00000100 -#define CLONE_THREAD 0x00010000 - - /* This is the only really unusual system call in PPC linux, but not because of any weirdness in the system call itself; because of all the freaky stuff we have to do to make the call useful. */ @@ -80,8 +76,7 @@ ENTRY (__clone) mtctr r30 mr r3,r31 bctrl - /* Call _exit with result from procedure. */ - b HIDDEN_JUMPTARGET(_exit) + DO_CALL(SYS_ify(exit)) L(parent): /* Parent. Restore registers & return. */ diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc64/clone.S b/sysdeps/unix/sysv/linux/powerpc/powerpc64/clone.S index 9e5bfd2..78c353a 100644 --- a/sysdeps/unix/sysv/linux/powerpc/powerpc64/clone.S +++ b/sysdeps/unix/sysv/linux/powerpc/powerpc64/clone.S @@ -20,9 +20,6 @@ #define _ERRNO_H 1 #include -#define CLONE_VM 0x00000100 -#define CLONE_THREAD 0x00010000 - /* This is the only really unusual system call in PPC linux, but not because of any weirdness in the system call itself; because of all the freaky stuff we have to do to make the call useful. */ @@ -84,15 +81,11 @@ ENTRY (__clone) mr r3,r31 bctrl ld r2,FRAME_TOC_SAVE(r1) - /* Call _exit with result from procedure. */ -#ifdef SHARED - b JUMPTARGET(__GI__exit) -#else - bl JUMPTARGET(_exit) + + DO_CALL(SYS_ify(exit)) /* We won't ever get here but provide a nop so that the linker will insert a toc adjusting stub if necessary. */ nop -#endif L(badargs): cfi_startproc diff --git a/sysdeps/unix/sysv/linux/s390/s390-32/clone.S b/sysdeps/unix/sysv/linux/s390/s390-32/clone.S index a8b4dbc..1588e5f 100644 --- a/sysdeps/unix/sysv/linux/s390/s390-32/clone.S +++ b/sysdeps/unix/sysv/linux/s390/s390-32/clone.S @@ -59,7 +59,7 @@ thread_start: ahi %r15,-96 /* make room on the stack for the save area */ xc 0(4,%r15),0(%r15) basr %r14,%r1 /* jump to fn */ - DO_CALL (exit, 1) + svc SYS_ify(exit) libc_hidden_def (__clone) weak_alias (__clone, clone) diff --git a/sysdeps/unix/sysv/linux/s390/s390-64/clone.S b/sysdeps/unix/sysv/linux/s390/s390-64/clone.S index daf8a58..5843188 100644 --- a/sysdeps/unix/sysv/linux/s390/s390-64/clone.S +++ b/sysdeps/unix/sysv/linux/s390/s390-64/clone.S @@ -60,7 +60,7 @@ thread_start: aghi %r15,-160 /* make room on the stack for the save area */ xc 0(8,%r15),0(%r15) basr %r14,%r1 /* jump to fn */ - DO_CALL (exit, 1) + svc SYS_ify(exit) libc_hidden_def (__clone) weak_alias (__clone, clone) diff --git a/sysdeps/unix/sysv/linux/sh/clone.S b/sysdeps/unix/sysv/linux/sh/clone.S index 9063b21..b13a64b 100644 --- a/sysdeps/unix/sysv/linux/sh/clone.S +++ b/sysdeps/unix/sysv/linux/sh/clone.S @@ -73,25 +73,8 @@ ENTRY(__clone) mov.l @(4,r15), r4 /* we are done, passing the return value through r0 */ - mov.l .L3, r1 -#ifdef SHARED - mov.l r12, @-r15 - sts.l pr, @-r15 - mov r0, r4 - mova .LG, r0 - mov.l .LG, r12 - add r0, r12 - mova .L3, r0 - add r0, r1 - jsr @r1 - nop - lds.l @r15+, pr - rts - mov.l @r15+, r12 -#else - jmp @r1 - mov r0, r4 -#endif + mov #+SYS_ify(exit), r3 + trapa #0x15 .align 2 .LG: .long _GLOBAL_OFFSET_TABLE_ diff --git a/sysdeps/unix/sysv/linux/sparc/sparc32/clone.S b/sysdeps/unix/sysv/linux/sparc/sparc32/clone.S index 6d2f5bd..1afa26e 100644 --- a/sysdeps/unix/sysv/linux/sparc/sparc32/clone.S +++ b/sysdeps/unix/sysv/linux/sparc/sparc32/clone.S @@ -24,8 +24,6 @@ #include #include -#define CLONE_VM 0x00000100 - /* int clone(int (*fn)(void *arg), void *child_stack, int flags, void *arg, pid_t *ptid, void *tls, pid_t *ctid); */ @@ -81,7 +79,8 @@ __thread_start: mov %g0, %fp /* terminate backtrace */ call %g2 mov %g3,%o0 - call HIDDEN_JUMPTARGET(_exit),0 + set __NR_exit, %g1 + ta 0x10 nop .size __thread_start, .-__thread_start diff --git a/sysdeps/unix/sysv/linux/sparc/sparc64/clone.S b/sysdeps/unix/sysv/linux/sparc/sparc64/clone.S index fc28539..785ccd1 100644 --- a/sysdeps/unix/sysv/linux/sparc/sparc64/clone.S +++ b/sysdeps/unix/sysv/linux/sparc/sparc64/clone.S @@ -24,8 +24,6 @@ #include #include -#define CLONE_VM 0x00000100 - /* int clone(int (*fn)(void *arg), void *child_stack, int flags, void *arg, pid_t *ptid, void *tls, pid_t *ctid); */ @@ -78,7 +76,8 @@ __thread_start: mov %g0, %fp /* terminate backtrace */ call %g2 mov %g3,%o0 - call HIDDEN_JUMPTARGET(_exit),0 + set __NR_exit, %g1 + ta 0x6d nop .size __thread_start, .-__thread_start diff --git a/sysdeps/unix/sysv/linux/tile/clone.S b/sysdeps/unix/sysv/linux/tile/clone.S index d7d2a3b..9610acd 100644 --- a/sysdeps/unix/sysv/linux/tile/clone.S +++ b/sysdeps/unix/sysv/linux/tile/clone.S @@ -168,10 +168,8 @@ ENTRY (__clone) move r0, r31 jalr r32 } - { - j HIDDEN_JUMPTARGET(_exit) - info INFO_OP_CANNOT_BACKTRACE /* Notify backtracer to stop. */ - } + moveli TREG_SYSCALL_NR_NAME, __NR_exit + swint1 PSEUDO_END (__clone) libc_hidden_def (__clone) diff --git a/sysdeps/unix/sysv/linux/tst-clone3.c b/sysdeps/unix/sysv/linux/tst-clone3.c new file mode 100644 index 0000000..e893e4d --- /dev/null +++ b/sysdeps/unix/sysv/linux/tst-clone3.c @@ -0,0 +1,95 @@ +/* Check if clone (CLONE_THREAD) does not call exit_group (BZ #21512) + 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 + . */ + +#include +#include +#include +#include +#include +#include +#include +#include + +#include /* For _STACK_GROWS_{UP,DOWN}. */ +#include + +/* Test if clone call with CLONE_THREAD does not call exit_group. The 'f' + function returns '1', which will be used by clone thread to call the + 'exit' syscall directly. If _exit is used instead, exit_group will be + used and thus the thread group will finish with return value of '1' + (where '2' from main thread is expected. */ + +static int +f (void *a) +{ + return 1; +} + +/* Futex wait for TID argument, similar to pthread_join internal + implementation. */ +#define wait_tid(tid) \ + do { \ + __typeof (tid) __tid; \ + while ((__tid = (tid)) != 0) \ + futex_wait (&(tid), __tid); \ + } while (0) + +static inline int +futex_wait (int *futexp, int val) +{ + return syscall (__NR_futex, futexp, FUTEX_WAIT, val); +} + +static int +do_test (void) +{ + char st[1024] __attribute__ ((aligned)); + int clone_flags = CLONE_THREAD; + /* Minimum required flags to used along with CLONE_THREAD. */ + clone_flags |= CLONE_VM | CLONE_SIGHAND; + /* We will used ctid to call on futex to wait for thread exit. */ + clone_flags |= CLONE_CHILD_CLEARTID; + pid_t ctid, tid; + +#ifdef __ia64__ + extern int __clone2 (int (*__fn) (void *__arg), void *__child_stack_base, + size_t __child_stack_size, int __flags, + void *__arg, ...); + tid = __clone2 (f, st, sizeof (st), clone_flags, NULL, /* ptid */ NULL, + /* tls */ NULL, &ctid); +#else +#if _STACK_GROWS_DOWN + tid = clone (f, st + sizeof (st), clone_flags, NULL, /* ptid */ NULL, + /* tls */ NULL, &ctid); +#elif _STACK_GROWS_UP + tid = clone (f, st, clone_flags, NULL, /* ptid */ NULL, /* tls */ NULL, + &ctid); +#else +#error "Define either _STACK_GROWS_DOWN or _STACK_GROWS_UP" +#endif +#endif + if (tid == -1) + FAIL_EXIT1 ("clone failed: %m"); + + wait_tid (ctid); + + return 2; +} + +#define EXPECTED_STATUS 2 +#include diff --git a/sysdeps/unix/sysv/linux/x86_64/clone.S b/sysdeps/unix/sysv/linux/x86_64/clone.S index d5c2d07..b10fc29 100644 --- a/sysdeps/unix/sysv/linux/x86_64/clone.S +++ b/sysdeps/unix/sysv/linux/x86_64/clone.S @@ -23,8 +23,6 @@ #include #include -#define CLONE_VM 0x00000100 - /* The userland implementation is: int clone (int (*fn)(void *arg), void *child_stack, int flags, void *arg), the kernel entry is: @@ -97,7 +95,8 @@ L(thread_start): call *%rax /* Call exit with return value from function call. */ movq %rax, %rdi - call HIDDEN_JUMPTARGET (_exit) + movl $SYS_ify(exit), %eax + syscall cfi_endproc; cfi_startproc;