From patchwork Thu Nov 14 18:50:58 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella Netto X-Patchwork-Id: 1195072 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=sourceware.org (client-ip=209.132.180.131; helo=sourceware.org; envelope-from=libc-alpha-return-107094-incoming=patchwork.ozlabs.org@sourceware.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.b="hztoEkwh"; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="YaPmHDHw"; dkim-atps=neutral Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 47DVtq2nyrz9s4Y for ; Fri, 15 Nov 2019 05:51:14 +1100 (AEDT) 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:cc:subject:date:message-id; q=dns; s= default; b=EyjaPrHnHOKF8LQwPcsBXZwCDVKp1KvG5XpeS5EQfVf0e4TxnhNie wHm9deM/SOpx+Fkx8beLHm5v5wFVWNppUsetVmTzrgVPYEumv+Qeg5uDysfMHExV 1ehCp8IrV4vt/fzObl08jK/SXIKOkvANEVHK7kC0+9jbmpb5I3iI0o= 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:cc:subject:date:message-id; s=default; bh=953D9xrMijm2u8jXCOG1Tttvqz4=; b=hztoEkwhDBfg/IaodFpnbkdGn5qA kc6xVZMWMwQm1EqTafLe6DULMiL4e7mDMPtwe0JWrAYSEhkALhM4fCURH5iDdMNm MLd6/6c2F2jVUBorCVzEod6gnWVo6xzQ/6VwoqJse+QCAhfSmuJJWGB90cWVk3e6 H0oPnGxuNRFQ//w= Received: (qmail 1107 invoked by alias); 14 Nov 2019 18:51:09 -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 1087 invoked by uid 89); 14 Nov 2019 18:51:08 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-21.7 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.1 spammy=smoke, HX-Spam-Relays-External:209.85.222.194, H*RU:209.85.222.194 X-HELO: mail-qk1-f194.google.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=2NnP16bp5braRTD+5KbBin1jezvM1AMwuVJjfNtWgXM=; b=YaPmHDHwR1Ec573m7CKdtzIVzAkhaEynkVQvyuWN/G3AOkLd0494JyPn2NgAO1tbEh /Zv3o+y/HnBCWVsxwxXSUpn51errto0En57wXSdkc+tHTtUVdKbHTnNKKbqpPSYjp2Ev PMc/55Tk22dTMpBJnIvFdmCsWrrVRusAUdQZ8QDwV7VR0ZW7dMcfn2JDMFpd0rzjQXZY JHaGKdkEJ97uXjyZyY70hCIB4fV3sDRAXblfbahwq9e0KoeWNyc9qE/IshicrH+f3uhl YWQyxpfH57TWFmhApWSFKUSjGHju6HezBv4Hir0FBYoMjc9agYsQaoozYAugPH7dg6mk jF3Q== From: Adhemerval Zanella To: libc-alpha@sourceware.org Cc: Lukasz Majewski Subject: [PATCH 1/2] misc: Set generic pselect as ENOSYS Date: Thu, 14 Nov 2019 15:50:58 -0300 Message-Id: <20191114185059.6403-1-adhemerval.zanella@linaro.org> The generic pselect implementation has the very specific race condition that motived the creation of the pselect syscall (no atomicity in signal mask set/reset). Using it as generic implementation is counterproductive Also currently only microblaze uses it as fallback when used on kernel prior 3.15. This patch moves the generic implementation to a microblaze specific one, sets the generic internal as a ENOSYS, and cleanups the Linux generic implementation. Also, the microblaze generic implementation first try to issue pselect instead of use the fallback (since it is expect that if the microblaze usage does rely on pselect, a sufficient updated kernel will be used). Microblaze defines __NR_pselect6 for Linux v3.2, although it was only wire-up on v3.15 (and the syscall number is the same as previous defined). Checked with a build against i686-linux-gnu (smoke test) and microblaze-linux-gnu. --- misc/pselect.c | 41 +----------- sysdeps/unix/sysv/linux/microblaze/pselect.c | 69 ++++++++++++++++++++ sysdeps/unix/sysv/linux/pselect.c | 31 +-------- 3 files changed, 75 insertions(+), 66 deletions(-) create mode 100644 sysdeps/unix/sysv/linux/microblaze/pselect.c diff --git a/misc/pselect.c b/misc/pselect.c index 76ded850a5..7296b6ef61 100644 --- a/misc/pselect.c +++ b/misc/pselect.c @@ -34,43 +34,8 @@ int __pselect (int nfds, fd_set *readfds, fd_set *writefds, fd_set *exceptfds, const struct timespec *timeout, const sigset_t *sigmask) { - struct timeval tval; - int retval; - sigset_t savemask; - - /* Change nanosecond number to microseconds. This might mean losing - precision and therefore the `pselect` should be available. But - for now it is hardly found. */ - if (timeout != NULL) - { - /* Catch bugs which would be hidden by the TIMESPEC_TO_TIMEVAL - computations. The division by 1000 truncates values. */ - if (__glibc_unlikely (timeout->tv_nsec < 0)) - { - __set_errno (EINVAL); - return -1; - } - - TIMESPEC_TO_TIMEVAL (&tval, timeout); - } - - /* The setting and restoring of the signal mask and the select call - should be an atomic operation. This can't be done without kernel - help. */ - if (sigmask != NULL) - __sigprocmask (SIG_SETMASK, sigmask, &savemask); - - /* Note the pselect() is a cancellation point. But since we call - select() which itself is a cancellation point we do not have - to do anything here. */ - retval = __select (nfds, readfds, writefds, exceptfds, - timeout != NULL ? &tval : NULL); - - if (sigmask != NULL) - __sigprocmask (SIG_SETMASK, &savemask, NULL); - - return retval; + __set_errno (ENOSYS); + return -1; } -#ifndef __pselect weak_alias (__pselect, pselect) -#endif +stub_warning (pselect) diff --git a/sysdeps/unix/sysv/linux/microblaze/pselect.c b/sysdeps/unix/sysv/linux/microblaze/pselect.c new file mode 100644 index 0000000000..62430ca69a --- /dev/null +++ b/sysdeps/unix/sysv/linux/microblaze/pselect.c @@ -0,0 +1,69 @@ +/* Synchronous I/O multiplexing. Linux/microblaze version. + Copyright (C) 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 + . */ + +#include +#include +#include +#include +#include + +/* Microblaze defines __NR_pselect6 for Linux v3.2, although it was wire-up + on v3.15. */ +#define __pselect __pselect_syscall +#include +#undef __pselect + +int +__pselect (int nfds, fd_set *readfds, fd_set *writefds, fd_set *exceptfds, + const struct timespec *timeout, const sigset_t *sigmask) +{ + int ret = __pselect_syscall (nfds, readfds, writefds, exceptfds, timeout, + sigmask); + if (ret < 0 && errno != ENOSYS) + return ret; + + /* The fallback uses 'selects' which shows the race condition regarding + signal mask set/restore, requires two additional syscalls, and has + a worse timeout precision (microseconds instead of nanoseconds). */ + + struct timeval tval, *ptval = NULL; + if (timeout != NULL) + { + if (! valid_nanoseconds (timeout->tv_nsec)) + { + __set_errno (EINVAL); + return -1; + } + + TIMESPEC_TO_TIMEVAL (&tval, timeout); + ptval = &tval; + } + + sigset_t savemask; + if (sigmask != NULL) + __sigprocmask (SIG_SETMASK, sigmask, &savemask); + + /* select itself is a cancellation entrypoint. */ + ret = __select (nfds, readfds, writefds, exceptfds, ptval); + + if (sigmask != NULL) + __sigprocmask (SIG_SETMASK, &savemask, NULL); + + return ret; +} +weak_alias (__pselect, pselect) diff --git a/sysdeps/unix/sysv/linux/pselect.c b/sysdeps/unix/sysv/linux/pselect.c index acda3e0cdd..77a5179925 100644 --- a/sysdeps/unix/sysv/linux/pselect.c +++ b/sysdeps/unix/sysv/linux/pselect.c @@ -23,16 +23,6 @@ #include #include - -#ifdef __NR_pselect6 -# ifndef __ASSUME_PSELECT -static int __generic_pselect (int nfds, fd_set *readfds, fd_set *writefds, - fd_set *exceptfds, - const struct timespec *timeout, - const sigset_t *sigmask); -# endif - - int __pselect (int nfds, fd_set *readfds, fd_set *writefds, fd_set *exceptfds, const struct timespec *timeout, const sigset_t *sigmask) @@ -59,24 +49,9 @@ __pselect (int nfds, fd_set *readfds, fd_set *writefds, fd_set *exceptfds, data.ss = (__syscall_ulong_t) (uintptr_t) sigmask; data.ss_len = _NSIG / 8; - int result = SYSCALL_CANCEL (pselect6, nfds, readfds, writefds, exceptfds, - timeout, &data); - -# ifndef __ASSUME_PSELECT - if (result == -1 && errno == ENOSYS) - result = __generic_pselect (nfds, readfds, writefds, exceptfds, timeout, - sigmask); -# endif - - return result; + return SYSCALL_CANCEL (pselect6, nfds, readfds, writefds, exceptfds, + timeout, &data); } +#ifndef __pselect weak_alias (__pselect, pselect) - -# ifndef __ASSUME_PSELECT -# define __pselect static __generic_pselect -# endif -#endif - -#ifndef __ASSUME_PSELECT -# include #endif From patchwork Thu Nov 14 18:50:59 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella Netto X-Patchwork-Id: 1195073 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=sourceware.org (client-ip=209.132.180.131; helo=sourceware.org; envelope-from=libc-alpha-return-107095-incoming=patchwork.ozlabs.org@sourceware.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.b="BJIfn0VD"; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="AmbU6DGI"; dkim-atps=neutral Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 47DVtz6hp6z9sPc for ; Fri, 15 Nov 2019 05:51:23 +1100 (AEDT) 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:cc:subject:date:message-id:in-reply-to :references; q=dns; s=default; b=B7E9xCGEMaFZlC8YJqPbqG2ZTLawZA2 2GpAReKiq8eMdtxv06s+W3GmAB8flI84GvIBRL1WMXK7+yI7RFt194do5rkBra/w 1p8QPgXb5xAFh/NIdLKwbpyN2cHlfi/TILvM6jzJB9ncageP3MpbGcseYB6dapYx /dv1iRXdrPuU= 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:cc:subject:date:message-id:in-reply-to :references; s=default; bh=wPwqfvWhQKb16VNWJMwzoKgF2js=; b=BJIfn 0VDd+ID2KpsnwoSNrPbyZfHBkKq6o58KKyHDzZ2IVs+PPH8nenhvcZuCXMfqjw2p M3bAACaq+kAXpCxrIyK0ltZ6kPjQ6FR2syuAN80t+3MxwgPCViArBFdE9lYrlml4 mVCQyEvpOeV2clEW43nRy8L2+qxb/Wn75fSD4I= Received: (qmail 1212 invoked by alias); 14 Nov 2019 18:51:09 -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 1146 invoked by uid 89); 14 Nov 2019 18:51:09 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-21.7 required=5.0 tests=AWL, 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.1 spammy=2574 X-HELO: mail-qt1-f195.google.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=tMSYaBJh4gV1UiyeCgAR/oU+2TkW69zjHrOQaHpzs7k=; b=AmbU6DGI7FHwrE3naiGf66mtRrtfgQMnLzuuzvJicm8F3So/4fWpd+8bU+mBNwdYEJ BCELeiaCmjuYh0F/xjfyrCHUWvQoIHlAAvs5r6ID/084pg8gDEpNIWvpewfHCRDdLPe6 G34z2j+1RKP1EQoK02a/O7qi+OP+pVnhDESqBvBbrgtyuXpQvbKDvX+8pDXt1uYl9nFk nTBaAXhSc4ULTCHBHf0o2t8W9qyr8x/fB0WnmUeioaA8PAvNQgBrmskLHtf46PakWjWb psbwUkJqYsG+jmCC0UdYiuqwpQ1hnmtLuNLaKWh59gv7iRRoPrSjVFY/AygRzQ3ISXqd ZQtQ== From: Adhemerval Zanella To: libc-alpha@sourceware.org Cc: Lukasz Majewski Subject: [PATCH 2/2] linux: Remove __ASSUME_PSELECT Date: Thu, 14 Nov 2019 15:50:59 -0300 Message-Id: <20191114185059.6403-2-adhemerval.zanella@linaro.org> In-Reply-To: <20191114185059.6403-1-adhemerval.zanella@linaro.org> References: <20191114185059.6403-1-adhemerval.zanella@linaro.org> The specific microblaze pselect implementation does not use __ASSUME_PSELECT anymore. Checked with a build against microblaze-linux-gnu. --- sysdeps/unix/sysv/linux/kernel-features.h | 4 ---- sysdeps/unix/sysv/linux/microblaze/kernel-features.h | 1 - 2 files changed, 5 deletions(-) diff --git a/sysdeps/unix/sysv/linux/kernel-features.h b/sysdeps/unix/sysv/linux/kernel-features.h index e6be76ff46..9012aae4c0 100644 --- a/sysdeps/unix/sysv/linux/kernel-features.h +++ b/sysdeps/unix/sysv/linux/kernel-features.h @@ -45,10 +45,6 @@ /* The statfs64 syscalls are available in 2.5.74 (but not for alpha). */ #define __ASSUME_STATFS64 1 -/* pselect/ppoll were introduced just after 2.6.16-rc1. On x86_64 and - SH this appeared first in 2.6.19-rc1, on ia64 in 2.6.22-rc1. */ -#define __ASSUME_PSELECT 1 - /* The *at syscalls were introduced just after 2.6.16-rc1. On PPC they were introduced in 2.6.17-rc1, on SH in 2.6.19-rc1. */ #define __ASSUME_ATFCTS 1 diff --git a/sysdeps/unix/sysv/linux/microblaze/kernel-features.h b/sysdeps/unix/sysv/linux/microblaze/kernel-features.h index 2dd9810f93..1c60488ea9 100644 --- a/sysdeps/unix/sysv/linux/microblaze/kernel-features.h +++ b/sysdeps/unix/sysv/linux/microblaze/kernel-features.h @@ -36,7 +36,6 @@ /* Support for the pselect6, preadv and pwritev syscalls was added in 3.15. */ #if __LINUX_KERNEL_VERSION < 0x030f00 -# undef __ASSUME_PSELECT # undef __ASSUME_PREADV # undef __ASSUME_PWRITEV #endif