From nobody Wed Mar 13 18:36:32 2024 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4Tvzk90lFrz5DRLG; Wed, 13 Mar 2024 18:36:33 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Tvzk907Vqz4Scw; Wed, 13 Mar 2024 18:36:33 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1710354993; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=jwD01VNexUp0weYXrk4T5hpFdKZEkQlv5cPpv1nzoVc=; b=QUaIfiWDkATULr8Hdj+8+wDP0tEsx+14dk5ZnMTNFTDgF3SfafaBAFsWJSgsCwLdO0kQdo MJDRWU4e7VGT+xkNwlWhceGtn7Wk7bRYRCZX0KbjS1s7bUQBPBFu9QnErKaL/xr4BeHypI DiRG1sdiqOgmb1q+IfL0L5pqD5tPu5H7qrl6go4PPp3z+QfrmPsJYd/nXSPinG1G4TOZZy wBP2QCoL8fOsSQavqPuGfAypr4fBEpniSEvC0cNZfxYj6cE6++Zz6641rlrnLk/GWjp+aB RIIFGuxAkB4zkgPc9RbSieWJG/hZ+EUI6D6BL+2T1M9QQQmNLu0rKGSeuD5bXg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1710354993; a=rsa-sha256; cv=none; b=RoHyfOW06Rps37JTQBrNBoNf2mIPCSF0wqc5y6sGV4VNS5gFq2y9pD6hDntf3WCqjunlFd YiiC07i+fnrXvzRobP72Lg5SabW08+RRkNWcH6lEIJB269SHwRwAEPqovWmU8ueWCkfYiy avQKKQ8FpXOLyN+488KQUoMc/B411aO4pVu/3oB403NmSiGcyAsXhuEby1A434OqYUEf14 HbEeC+qxmWhj+Loe/Nh+49QmSyrYu5B08MsBbb0EKqoW+9S3Ni7vxsLap7UOswTzflhF3n tzV8iyOc/e2FGIZg5egKJ/ndzeZLcPmsipc7IgDrJQNSgrUfBv4O61uixf8M0A== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1710354993; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=jwD01VNexUp0weYXrk4T5hpFdKZEkQlv5cPpv1nzoVc=; b=faQQZfPOB8VXo6aTVKboDiiiGsUc1nO5TYBXeMnwFgBgI0UbytzBkxM4y4cmY255jYZWPY IAa9LD5ZO6aclHkOVmpUZPFZCZW62SzTQedT5rc4TXaXIEXKw61csOBukiqkc7j4kxo8Nl ZzIEG08aoQFwgqzEJIFDPwy1aG+zGWbrqD5oB021vKOHASPZpiLqvYt6JxuEIZiFHqfeaj E2zTEh3Syw/evRsoP5dZOoTOUfuc3WC/q1QfyBHbpUtBRBwg9LFg3jBfJ8W7xjR4AWKQGL PYgA+zwTSNmG1wGX/my9cjapDmB7p49NKbJWZNheD0WpgsRhtQwufprjJ0w+FA== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4Tvzk86qb2ztlD; Wed, 13 Mar 2024 18:36:32 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.17.1/8.17.1) with ESMTP id 42DIaWv9002497; Wed, 13 Mar 2024 18:36:32 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.17.1/8.17.1/Submit) id 42DIaWxA002494; Wed, 13 Mar 2024 18:36:32 GMT (envelope-from git) Date: Wed, 13 Mar 2024 18:36:32 GMT Message-Id: <202403131836.42DIaWxA002494@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Brooks Davis Subject: git: f7dbbbd1762f - main - libsys: don't expose sigwait wrapper List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: brooks X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: f7dbbbd1762f8f74d5a585230c56eca92e4fdb8b Auto-Submitted: auto-generated The branch main has been updated by brooks: URL: https://cgit.FreeBSD.org/src/commit/?id=f7dbbbd1762f8f74d5a585230c56eca92e4fdb8b commit f7dbbbd1762f8f74d5a585230c56eca92e4fdb8b Author: Brooks Davis AuthorDate: 2024-03-13 17:04:07 +0000 Commit: Brooks Davis CommitDate: 2024-03-13 17:04:07 +0000 libsys: don't expose sigwait wrapper Long ago (e129c18a83ef) __sys_sigwait was wrapped to prevent sigwait() from returning with EINTR. Through a series of changes this wrapper become __libc_sigwait which was internal to libc and used solely in the interposing table. To support a move of sigwait back to libc, move this wrapper into libsys and rename it with an __libsys_ prefix. Reviewed by: kib Differential Revision: https://reviews.freebsd.org/D44238 --- lib/libc/include/libc_private.h | 3 +-- lib/libsys/Makefile.sys | 3 ++- lib/libsys/Symbol.sys.map | 1 - lib/libsys/interposing_table.c | 2 +- lib/libsys/libsys_sigwait.c | 45 +++++++++++++++++++++++++++++++++++++++++ lib/libsys/sigwait.c | 15 -------------- 6 files changed, 49 insertions(+), 20 deletions(-) diff --git a/lib/libc/include/libc_private.h b/lib/libc/include/libc_private.h index 7b5b987c22a3..6a6de5b5645d 100644 --- a/lib/libc/include/libc_private.h +++ b/lib/libc/include/libc_private.h @@ -414,8 +414,7 @@ int __libc_sigaction(int, const struct sigaction *, int __libc_sigprocmask(int, const __sigset_t *, __sigset_t *) __hidden; int __libc_sigsuspend(const __sigset_t *) __hidden; -int __libc_sigwait(const __sigset_t * __restrict, - int * restrict sig); +int __libsys_sigwait(const __sigset_t *, int *) __hidden; int __libc_system(const char *); int __libc_tcdrain(int); diff --git a/lib/libsys/Makefile.sys b/lib/libsys/Makefile.sys index b3d41c804290..e798a94e4ae0 100644 --- a/lib/libsys/Makefile.sys +++ b/lib/libsys/Makefile.sys @@ -37,7 +37,8 @@ SRCS+= \ __getosreldate.c \ getpagesize.c \ getpagesizes.c \ - interposing_table.c + interposing_table.c \ + libsys_sigwait.c SRCS+= getdents.c lstat.c mknod.c stat.c diff --git a/lib/libsys/Symbol.sys.map b/lib/libsys/Symbol.sys.map index ff368ebfac44..fa27dc95aea4 100644 --- a/lib/libsys/Symbol.sys.map +++ b/lib/libsys/Symbol.sys.map @@ -433,7 +433,6 @@ FBSD_1.8 { FBSDprivate_1.0 { /* Add entries in sort(1) order */ - __libc_sigwait; __libsys_interposing_slot; __set_error_selector; __sigwait; diff --git a/lib/libsys/interposing_table.c b/lib/libsys/interposing_table.c index 4ee36da4c4c4..e285def70fad 100644 --- a/lib/libsys/interposing_table.c +++ b/lib/libsys/interposing_table.c @@ -59,7 +59,7 @@ interpos_func_t __libsys_interposing[INTERPOS_MAX] = { SLOT(sigaction, __sys_sigaction), SLOT(sigprocmask, __sys_sigprocmask), SLOT(sigsuspend, __sys_sigsuspend), - SLOT(sigwait, __libc_sigwait), + SLOT(sigwait, __libsys_sigwait), SLOT(sigtimedwait, __sys_sigtimedwait), SLOT(sigwaitinfo, __sys_sigwaitinfo), SLOT(swapcontext, __sys_swapcontext), diff --git a/lib/libsys/libsys_sigwait.c b/lib/libsys/libsys_sigwait.c new file mode 100644 index 000000000000..ce1a416d76b3 --- /dev/null +++ b/lib/libsys/libsys_sigwait.c @@ -0,0 +1,45 @@ +/*- + * SPDX-License-Identifier: BSD-2-Clause + * + * Copyright (c) 2010 davidxu@freebsd.org + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * 1. Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * 2. Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE + * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF + * SUCH DAMAGE. + */ + +#include +#include +#include "libc_private.h" + +/* XXX: why does this symbol exist? */ +__weak_reference(__libsys_sigwait, __sigwait); + +int +__libsys_sigwait(const sigset_t *set, int *sig) +{ + int ret; + + /* POSIX does not allow EINTR to be returned */ + do { + ret = __sys_sigwait(set, sig); + } while (ret == EINTR); + return (ret); +} diff --git a/lib/libsys/sigwait.c b/lib/libsys/sigwait.c index 4b863eb04db0..412a9206b821 100644 --- a/lib/libsys/sigwait.c +++ b/lib/libsys/sigwait.c @@ -25,12 +25,9 @@ * SUCH DAMAGE. */ -#include #include #include "libc_private.h" -__weak_reference(__libc_sigwait, __sigwait); - #pragma weak sigwait int sigwait(const sigset_t *set, int *sig) @@ -38,15 +35,3 @@ sigwait(const sigset_t *set, int *sig) return (((int (*)(const sigset_t *, int *)) __libsys_interposing[INTERPOS_sigwait])(set, sig)); } - -int -__libc_sigwait(const sigset_t *set, int *sig) -{ - int ret; - - /* POSIX does not allow EINTR to be returned */ - do { - ret = __sys_sigwait(set, sig); - } while (ret == EINTR); - return (ret); -}