From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 99131 invoked by alias); 2 Jun 2017 12:51:30 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 99005 invoked by uid 89); 2 Jun 2017 12:51:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-11.4 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=ham version=3.3.2 spammy=cancellation, consolidation X-HELO: mail-qk0-f178.google.com X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:from:to:references:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=LPAj84lvtdhksxOMr+SUrMqVXFna2GWU48grBtQInA0=; b=GuC6lg2JZMWTAZKccTWDKepHwQMuHYxopd0gz7VsLE7diFSKgI+b4SqC3bx/nF2AUK 6ilny3EU0ElMM62OK9xkvEGtk546SamZehErfRD+XT9ieHKmWb/YkNXb/taV6QHisG9H n1b3odp9NYaj/iwShMSw+QmiVG0HCzbVHtHAhVi5iDbHlSGvfpihljvaAg4XVIBrnyty H6JBl7v4jhuJiPvz36ARmOfGJk7NQ0Xg0CrqBCbi1GyAQbir4IKwFiDy6pRueQTKeZd4 uS+kBB7tNxeFFhVCp/2i1fxdTSIndNh8Ckc0l9vXo4X7akdB02Xr9+guG7Cd1hHjKd1L jHLA== X-Gm-Message-State: AODbwcC/1JRitFAkesJvWOh1no2imGg0O+rZKMkyYP2aqVokbv6rpSTz 5BhMiMDDloSztvwbBKeFag== X-Received: by 10.55.82.7 with SMTP id g7mr8108005qkb.253.1496407888404; Fri, 02 Jun 2017 05:51:28 -0700 (PDT) Subject: Re: [PATCH 0/5] Remove sysdep-cancel assembly macro From: Adhemerval Zanella To: libc-alpha@sourceware.org References: <1495563960-669-1-git-send-email-adhemerval.zanella@linaro.org> Message-ID: <8fa564d1-0892-e2cc-2124-6a3360e9ed92@linaro.org> Date: Fri, 02 Jun 2017 12:51:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <1495563960-669-1-git-send-email-adhemerval.zanella@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2017-06/txt/msg00088.txt.bz2 Ping. On 23/05/2017 15:25, Adhemerval Zanella wrote: > This patchset finishes my previous work of Linux syscall consolidation > and refactor to finally remove the requeriment of sysdeps-cancel.h > assembly macros for cancellable syscalls. All ports now uses the > SYSCALL_CANCEL macros with C implementation which in turn calls > INLINE_SYSCALL macros. > > Besides simplify the curent code requirement, it also simplify > possible future ports to require only the C bindings and the direct > assembly one for syscalls.list autogeneration (another possible > work for future would be to use C generate binding to simplify > even further the syscall requirements). > > I checked with a build for mostly supported architecture: > aarch64-linux-gnu, alpha-linux-gnu, arm-linux-gnueabhi, i686-linux-gnu, > ia64-linux-gnu, m68k-linux-gnu, microblaze-linux-gnu, > mips{64,64n32}-linux-gnu, nios2-linux-gnu, powerpc{64,64le}-linux-gnu, > s390{x}-linux-gnu, sh4-linux-gnu{-soft}, tile{pro,gx}-linux-gnu{-32}, > sparc{v9,64}-linux-gnu, and x86_64-linux-gnu. > > I also run a full make check on real hardware on aarch64-linux-gnu, > arm-linux-gnueabihf, i686-linux-gnu, powerpc{64le}-linux-gnu, > sparc{v9,64}-linux-gnu, and x86_64-linux-gnu{-x32}. > > Adhemerval Zanella (5): > Fix {INLINE,INTERNAL}_SYSCALL macros for x32 > Remove p{read,write}{v} and fallocate from x86 auto-generation list > linux: Consolidate sync_file_range implementation > Consolidate Linux fcntl implementation > Remove cancellation support for syscall generation > > ChangeLog | 250 ++++++++++++++++++++ > io/Makefile | 1 - > sysdeps/unix/make-syscalls.sh | 4 - > sysdeps/unix/syscall-template.S | 7 +- > sysdeps/unix/sysv/linux/Makefile | 1 + > sysdeps/unix/sysv/linux/aarch64/sysdep-cancel.h | 87 +------ > sysdeps/unix/sysv/linux/alpha/sysdep-cancel.h | 131 +---------- > sysdeps/unix/sysv/linux/arm/fcntl.c | 1 - > sysdeps/unix/sysv/linux/arm/sysdep-cancel.h | 197 +--------------- > sysdeps/unix/sysv/linux/fcntl.c | 52 ++--- > .../unix/sysv/linux/generic/wordsize-32/fcntl.c | 86 ------- > sysdeps/unix/sysv/linux/hppa/fcntl.c | 1 - > sysdeps/unix/sysv/linux/hppa/sysdep-cancel.h | 236 +------------------ > sysdeps/unix/sysv/linux/i386/fcntl.c | 61 ----- > sysdeps/unix/sysv/linux/i386/sysdep-cancel.h | 119 +--------- > sysdeps/unix/sysv/linux/ia64/sysdep-cancel.h | 194 +--------------- > sysdeps/unix/sysv/linux/m68k/fcntl.c | 1 - > sysdeps/unix/sysv/linux/m68k/sysdep-cancel.h | 108 +-------- > sysdeps/unix/sysv/linux/microblaze/fcntl.c | 1 - > sysdeps/unix/sysv/linux/microblaze/sysdep-cancel.h | 116 +--------- > sysdeps/unix/sysv/linux/mips/mips32/fcntl.c | 1 - > sysdeps/unix/sysv/linux/mips/mips64/n32/fcntl.c | 1 - > .../unix/sysv/linux/mips/mips64/sysdep-cancel.h | 249 -------------------- > sysdeps/unix/sysv/linux/mips/sysdep-cancel.h | 159 +------------ > sysdeps/unix/sysv/linux/nios2/sysdep-cancel.h | 110 +-------- > sysdeps/unix/sysv/linux/powerpc/powerpc32/fcntl.c | 1 - > .../sysv/linux/powerpc/powerpc32/sysdep-cancel.h | 118 ---------- > sysdeps/unix/sysv/linux/powerpc/powerpc64/fcntl.c | 49 +--- > .../sysv/linux/powerpc/powerpc64/sysdep-cancel.h | 147 ------------ > sysdeps/unix/sysv/linux/powerpc/sysdep-cancel.h | 38 ++++ > sysdeps/unix/sysv/linux/s390/s390-32/fcntl.c | 1 - > .../unix/sysv/linux/s390/s390-32/sysdep-cancel.h | 105 +-------- > .../unix/sysv/linux/s390/s390-64/sysdep-cancel.h | 109 +-------- > sysdeps/unix/sysv/linux/sh/fcntl.c | 1 - > sysdeps/unix/sysv/linux/sh/sysdep-cancel.h | 136 +---------- > sysdeps/unix/sysv/linux/sparc/sparc32/fcntl.c | 1 - > .../unix/sysv/linux/sparc/sparc32/sysdep-cancel.h | 111 --------- > .../unix/sysv/linux/sparc/sparc64/sysdep-cancel.h | 109 --------- > sysdeps/unix/sysv/linux/sparc/sysdep-cancel.h | 38 ++++ > sysdeps/unix/sysv/linux/tile/sysdep-cancel.h | 123 +--------- > sysdeps/unix/sysv/linux/wordsize-64/syscalls.list | 1 - > sysdeps/unix/sysv/linux/x86_64/syscalls.list | 4 - > sysdeps/unix/sysv/linux/x86_64/sysdep-cancel.h | 62 +---- > sysdeps/unix/sysv/linux/x86_64/sysdep.h | 251 ++++++++++++--------- > sysdeps/unix/sysv/linux/x86_64/x32/syscalls.list | 1 - > sysdeps/unix/sysv/linux/x86_64/x32/times.c | 24 +- > 46 files changed, 572 insertions(+), 3032 deletions(-) > delete mode 100644 sysdeps/unix/sysv/linux/arm/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/generic/wordsize-32/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/hppa/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/i386/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/m68k/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/microblaze/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/mips/mips32/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/mips/mips64/n32/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/mips/mips64/sysdep-cancel.h > delete mode 100644 sysdeps/unix/sysv/linux/powerpc/powerpc32/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/powerpc/powerpc32/sysdep-cancel.h > delete mode 100644 sysdeps/unix/sysv/linux/powerpc/powerpc64/sysdep-cancel.h > create mode 100644 sysdeps/unix/sysv/linux/powerpc/sysdep-cancel.h > delete mode 100644 sysdeps/unix/sysv/linux/s390/s390-32/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/sh/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/sparc/sparc32/fcntl.c > delete mode 100644 sysdeps/unix/sysv/linux/sparc/sparc32/sysdep-cancel.h > delete mode 100644 sysdeps/unix/sysv/linux/sparc/sparc64/sysdep-cancel.h > create mode 100644 sysdeps/unix/sysv/linux/sparc/sysdep-cancel.h >