From patchwork Thu Jun 2 11:26:32 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella X-Patchwork-Id: 69153 Delivered-To: patch@linaro.org Received: by 10.140.106.246 with SMTP id e109csp63125qgf; Thu, 2 Jun 2016 04:27:17 -0700 (PDT) X-Received: by 10.98.55.129 with SMTP id e123mr3479953pfa.145.1464866837383; Thu, 02 Jun 2016 04:27:17 -0700 (PDT) Return-Path: Received: from sourceware.org (server1.sourceware.org. [209.132.180.131]) by mx.google.com with ESMTPS id r63si72657pfi.171.2016.06.02.04.27.17 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 02 Jun 2016 04:27:17 -0700 (PDT) Received-SPF: pass (google.com: domain of libc-alpha-return-70103-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; spf=pass (google.com: domain of libc-alpha-return-70103-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom=libc-alpha-return-70103-patch=linaro.org@sourceware.org; dmarc=fail (p=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:in-reply-to :references; q=dns; s=default; b=KNCgERhhOizte8ztOYhKmOHWfz4zNTz MlZ1YNHI5A0xDkqRjLR2VuyFbieQZhB6J02JRilQRXISOBRkNvkQKop12iIJEmON gAR040DxarZM1Zzix8SubaoITxEx9oDg3fVGHun2VqhHCI2oMJcEoyanuSm5IUbP T1IMbJwQOqUY= 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:in-reply-to :references; s=default; bh=PKVM23x6UHrwUBd3TGLGpMZH1ts=; b=eLc7P 7BNgyxwTC+UI78ykJpwJtIGw/DFOjfeQue8uDqCATqDJU0bWAzfpedeGJRL3wnOs uZzPrEwvAzkSxa2N+VRS67cCEzsFGrrci4vkcX6cT2zwfTblPNDo/ZUXGrBcL2KM inyFy8Y+q1ZVC+n/ek9NqmWn3XXo2RSMhSvUno= Received: (qmail 128844 invoked by alias); 2 Jun 2016 11:27:06 -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 128812 invoked by uid 89); 2 Jun 2016 11:27:05 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.2 spammy=Based, consolidation, 1567, Contributed X-HELO: mail-yw0-f169.google.com X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=+0u6GZ8tR2FCrvObN4vMBmd1eMke1WRc0hAGyLc+cyA=; b=P2TsYlQaiDh9BsrOArZreCC5GxWnNlpkcs4Hw6TXPCiwE/mNdYXc4PKV6yANkYCDFf lVKHbreKRKpPEVQmAkYkwGwKgeGs31+EOqEVKrXU4BaZlY+prPq1f2Y1JPgoaqSv2d7s lNmYSdmxxM52FPRHLAIsqA/oGCLxdvBdsq4S9OYxDtcBqrqz7kwrTWifwU22N3XfpbXZ DwDkjwXfG4fEBz6dq78oSvbHnEIH1xWWrPF/I/sWGMMuRyLI1KhGy8va7co07IkjncSu +lxSygMxMVZ0gjpf83DvITN1y5elBI+Raw6RSwGD7xhRv9uPciDTI9bg/sKeOJhb3pTS a5Tw== X-Gm-Message-State: ALyK8tIABxELucNEG3J9Uuq9l3/unSsp9R2qsf1RdzQcbxUijVBM4/YMgcD8JdAXmW+Is7tk X-Received: by 10.13.203.4 with SMTP id n4mr5757900ywd.217.1464866805220; Thu, 02 Jun 2016 04:26:45 -0700 (PDT) From: Adhemerval Zanella To: libc-alpha@sourceware.org Subject: [PATCH 2/2] Consolidate pwritev/pwritev64 implementations Date: Thu, 2 Jun 2016 08:26:32 -0300 Message-Id: <1464866792-11025-2-git-send-email-adhemerval.zanella@linaro.org> In-Reply-To: <1464866792-11025-1-git-send-email-adhemerval.zanella@linaro.org> References: <1464866792-11025-1-git-send-email-adhemerval.zanella@linaro.org> This patch consolidates all the pwritev{64} implementation for Linux in only one (sysdeps/unix/sysv/linux/pwritev{64}.c). It also removes the syscall from the auto-generation using assembly macros. It was based on previous pwrite/pwrite64 consolidation patch. The new macro SYSCALL_LL{64} is used to handle the offset argument and alias is created for __ASSUME_OFF_DIFF_OFF64 in case of pread64. Checked on x86_64, i386, aarch64, and powerpc64le. * misc/Makefile (CFLAGS-pwritev.c): New variable: add cancellation required flags. (CFLAGS-pwritev64.c): Likewise. * sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev.c: Remove file. * sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev64.c: Likewise. * sysdeps/unix/sysv/linux/mips/mips64/n64/pwritev64.c: Likewise. * sysdeps/unix/sysv/linux/wordsize-64/pwritev.c: Likewise. * sysdeps/unix/sysv/linux/wordsize-64/pwritev64.: Likwise. * sysdeps/unix/sysv/linux/x86_64/x32/syscalls.list (pwritev): Remove syscall from auto-generation. * sysdeps/unix/sysv/linux/pwritev.c: Rewrite implementation. [WORDSIZE == 64] (pwritev64): Remove macro. [!PWRITEV] (PWRITEV): Likewise. [!PWRITEV] (PWRITEV_REPLACEMENT): Likewise. [!PWRITEV] (PWRITE): Likewise. [!PWRITEV] (OFF_T): Likewise. [!__ASSUME_PWRITEV] (PWRITEV_REPLACEMENT): Likewise. (LO_HI_LONG): Remove macro. [__WORDSIZE != 64 || __ASSUME_OFF_DIFF_OFF64] (pwritev): Add function. * sysdeps/unix/sysv/linux/pwritev64.c: Rewrite implementation. (PWRITEV): Remove macro. (PWRITEV_REPLACEMENTE): Likewise. (PWRITE): Likewise. (OFF_T): Likewise. (pwritev64): New function. --- misc/Makefile | 2 + .../unix/sysv/linux/generic/wordsize-32/pwritev.c | 37 ------------- .../sysv/linux/generic/wordsize-32/pwritev64.c | 38 ------------- .../unix/sysv/linux/mips/mips64/n64/pwritev64.c | 1 - sysdeps/unix/sysv/linux/pwritev.c | 62 +++------------------ sysdeps/unix/sysv/linux/pwritev64.c | 58 ++++++++++++++++++-- sysdeps/unix/sysv/linux/wordsize-64/pwritev.c | 64 ---------------------- sysdeps/unix/sysv/linux/wordsize-64/pwritev64.c | 1 - sysdeps/unix/sysv/linux/x86_64/x32/syscalls.list | 1 - 10 files changed, 88 insertions(+), 202 deletions(-) delete mode 100644 sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev.c delete mode 100644 sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev64.c delete mode 100644 sysdeps/unix/sysv/linux/mips/mips64/n64/pwritev64.c delete mode 100644 sysdeps/unix/sysv/linux/wordsize-64/pwritev.c delete mode 100644 sysdeps/unix/sysv/linux/wordsize-64/pwritev64.c -- 2.7.4 diff --git a/misc/Makefile b/misc/Makefile index 658d4ba..aab1174 100644 --- a/misc/Makefile +++ b/misc/Makefile @@ -90,6 +90,8 @@ CFLAGS-readv.c = -fexceptions -fasynchronous-unwind-tables CFLAGS-writev.c = -fexceptions -fasynchronous-unwind-tables CFLAGS-preadv.c = -fexceptions -fasynchronous-unwind-tables CFLAGS-preadv64.c = -fexceptions -fasynchronous-unwind-tables +CFLAGS-pwritev.c = -fexceptions -fasynchronous-unwind-tables +CFLAGS-pwrite64.c = -fexceptions -fasynchronous-unwind-tables CFLAGS-usleep.c = -fexceptions CFLAGS-syslog.c = -fexceptions CFLAGS-error.c = -fexceptions diff --git a/sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev.c b/sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev.c deleted file mode 100644 index 9ef11b5..0000000 --- a/sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev.c +++ /dev/null @@ -1,37 +0,0 @@ -/* Copyright (C) 2011-2016 Free Software Foundation, Inc. - This file is part of the GNU C Library. - Contributed by Chris Metcalf , 2011. - Based on work contributed by Ulrich Drepper , 1997. - - 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 - -ssize_t -__libc_pwritev (int fd, const struct iovec *vector, int count, off_t offset) -{ - assert (sizeof (offset) == 4); - return SYSCALL_CANCEL (pwritev, fd, vector, count, __ALIGNMENT_ARG - __LONG_LONG_PAIR (offset >> 31, offset)); -} -strong_alias (__libc_pwritev, __pwritev) -weak_alias (__libc_pwritev, pwritev) diff --git a/sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev64.c b/sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev64.c deleted file mode 100644 index 49b723d..0000000 --- a/sysdeps/unix/sysv/linux/generic/wordsize-32/pwritev64.c +++ /dev/null @@ -1,38 +0,0 @@ -/* Copyright (C) 2011-2016 Free Software Foundation, Inc. - This file is part of the GNU C Library. - Contributed by Chris Metcalf , 2011. - Based on work contributed by Ulrich Drepper , 1997. - - 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 - -ssize_t -__libc_pwritev64 (int fd, const struct iovec *vector, int count, - off64_t offset) -{ - return SYSCALL_CANCEL (pwritev, fd, - vector, count, __ALIGNMENT_ARG - __LONG_LONG_PAIR ((off_t) (offset >> 32), - (off_t) (offset & 0xffffffff))); -} -strong_alias (__libc_pwritev64, pwritev64) -weak_alias (__libc_pwritev64, __pwritev64) diff --git a/sysdeps/unix/sysv/linux/mips/mips64/n64/pwritev64.c b/sysdeps/unix/sysv/linux/mips/mips64/n64/pwritev64.c deleted file mode 100644 index 8b72a29..0000000 --- a/sysdeps/unix/sysv/linux/mips/mips64/n64/pwritev64.c +++ /dev/null @@ -1 +0,0 @@ -/* Empty since the pwritev syscall is equivalent. */ diff --git a/sysdeps/unix/sysv/linux/pwritev.c b/sysdeps/unix/sysv/linux/pwritev.c index e66f08e..d3904d5 100644 --- a/sysdeps/unix/sysv/linux/pwritev.c +++ b/sysdeps/unix/sysv/linux/pwritev.c @@ -15,67 +15,19 @@ License along with the GNU C Library; if not, see . */ -#include -#include -#include -#include -#if __WORDSIZE == 64 && !defined PWRITEV -/* Hide the pwritev64 declaration. */ -# define pwritev64 __redirect_pwritev64 -#endif #include - #include -#include -#include - - -#ifndef PWRITEV -# define PWRITEV pwritev -# define PWRITEV_REPLACEMENT __atomic_pwritev_replacement -# define PWRITE __pwrite -# define OFF_T off_t -#endif -#define LO_HI_LONG(val) \ - (off_t) val, \ - (off_t) ((((uint64_t) (val)) >> (sizeof (long) * 4)) >> (sizeof (long) * 4)) - -#ifndef __ASSUME_PWRITEV -static ssize_t PWRITEV_REPLACEMENT (int, const struct iovec *, - int, OFF_T) internal_function; -#endif +#if __WORDSIZE != 64 || defined (__ASSUME_OFF_DIFF_OFF64) +# ifndef __NR_pwritev +# define __NR_pwritev __NR_pwritev64 +# endif ssize_t -PWRITEV (int fd, const struct iovec *vector, int count, OFF_T offset) +pwritev (int fd, const struct iovec *vector, int count, off_t offset) { -#ifdef __NR_pwritev - ssize_t result; - - result = SYSCALL_CANCEL (pwritev, fd, vector, count, LO_HI_LONG (offset)); - -# ifdef __ASSUME_PWRITEV - return result; -# endif -#endif - -#ifndef __ASSUME_PWRITEV -# ifdef __NR_pwritev - if (result >= 0 || errno != ENOSYS) - return result; -# endif - - return PWRITEV_REPLACEMENT (fd, vector, count, offset); -#endif + return SYSCALL_CANCEL (pwritev, fd, vector, count, + __ALIGNMENT_ARG SYSCALL_LL (offset)); } -#if __WORDSIZE == 64 && defined pwritev64 -# undef pwritev64 -strong_alias (pwritev, pwritev64) -#endif - -#ifndef __ASSUME_PWRITEV -# undef PWRITEV -# define PWRITEV static internal_function PWRITEV_REPLACEMENT -# include #endif diff --git a/sysdeps/unix/sysv/linux/pwritev64.c b/sysdeps/unix/sysv/linux/pwritev64.c index 1e8168f..5030d81 100644 --- a/sysdeps/unix/sysv/linux/pwritev64.c +++ b/sysdeps/unix/sysv/linux/pwritev64.c @@ -1,6 +1,54 @@ -#define PWRITEV pwritev64 -#define PWRITEV_REPLACEMENT __atomic_pwritev64_replacement -#define PWRITE __pwrite64 -#define OFF_T off64_t +/* Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. -#include "pwritev.c" + 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 + +#ifndef __NR_pwritev64 +# define __NR_pwritev64 __NR_pwritev +#endif + +#ifdef __ASSUME_PWRITEV +ssize_t +pwritev64 (int fd, const struct iovec *vector, int count, off64_t offset) +{ + return SYSCALL_CANCEL (pwritev64, fd, vector, count, + __ALIGNMENT_ARG SYSCALL_LL64 (offset)); +} +#else +static ssize_t __atomic_pwritev_replacement (int, const struct iovec *, + int, off_t) internal_function; +ssize_t +pwritev64 (int fd, const struct iovec *vector, int count, off64_t offset) +{ +#ifdef __NR_pwrite64v + ssize_t result = SYSCALL_CANCEL (pwritev64, fd, vector, count, + __ALIGNMENT_ARG SYSCALL_LL64 (offset)); + if (result >= 0 || errno != ENOSYS) + return result; +#endif + return __atomic_pwritev_replacement (fd, vector, count, offset); +} +# define PWRITEV static internal_function __atomic_pwritev_replacement +# define PWRITE __pwrite +# define OFF_T off_t +# include +#endif + +#if __WORDSIZE == 64 && !defined (__ASSUME_OFF_DIFF_OFF64) +strong_alias (pwritev64, pwritev) +#endif diff --git a/sysdeps/unix/sysv/linux/wordsize-64/pwritev.c b/sysdeps/unix/sysv/linux/wordsize-64/pwritev.c deleted file mode 100644 index 1fa6eb3..0000000 --- a/sysdeps/unix/sysv/linux/wordsize-64/pwritev.c +++ /dev/null @@ -1,64 +0,0 @@ -/* 64-bit pwritev. - Copyright (C) 2012-2016 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 -/* Hide the pwritev64 declaration. */ -#define pwritev64 __redirect_pwritev64 -#include - -#include -#include -#include - -#ifndef __ASSUME_PWRITEV -static ssize_t __atomic_pwritev_replacement (int, const struct iovec *, - int, off_t) internal_function; -#endif - -ssize_t -pwritev (int fd, const struct iovec *vector, int count, off_t offset) -{ -#ifdef __NR_pwritev - ssize_t result; - - result = SYSCALL_CANCEL (pwritev, fd, vector, count, offset); -# ifdef __ASSUME_PWRITEV - return result; -# endif -#endif - -#ifndef __ASSUME_PWRITEV -# ifdef __NR_pwritev - if (result >= 0 || errno != ENOSYS) - return result; -# endif - - return __atomic_pwritev_replacement (fd, vector, count, offset); -#endif -} -#undef pwritev64 -strong_alias (pwritev, pwritev64) - -#ifndef __ASSUME_PWRITEV -# define PWRITE __pwrite -# define PWRITEV static internal_function __atomic_pwritev_replacement -# define OFF_T off_t -# include -#endif diff --git a/sysdeps/unix/sysv/linux/wordsize-64/pwritev64.c b/sysdeps/unix/sysv/linux/wordsize-64/pwritev64.c deleted file mode 100644 index 8b72a29..0000000 --- a/sysdeps/unix/sysv/linux/wordsize-64/pwritev64.c +++ /dev/null @@ -1 +0,0 @@ -/* Empty since the pwritev syscall is equivalent. */ diff --git a/sysdeps/unix/sysv/linux/x86_64/x32/syscalls.list b/sysdeps/unix/sysv/linux/x86_64/x32/syscalls.list index 58cd7fc..dce4302 100644 --- a/sysdeps/unix/sysv/linux/x86_64/x32/syscalls.list +++ b/sysdeps/unix/sysv/linux/x86_64/x32/syscalls.list @@ -4,5 +4,4 @@ fallocate - fallocate Ci:iiii fallocate fallocate64 gettimeofday - gettimeofday:__vdso_gettimeofday@LINUX_2.6 i:pP __gettimeofday gettimeofday personality EXTRA personality Ei:i __personality personality posix_fadvise - fadvise64 Vi:iiii posix_fadvise posix_fadvise64 -pwritev - pwritev Ci:ipii pwritev pwritev64 time - time:__vdso_time@LINUX_2.6 Ei:P time