From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from albireo.enyo.de (albireo.enyo.de [37.24.231.21]) by sourceware.org (Postfix) with ESMTPS id DA5C7388B03E for ; Mon, 27 Apr 2020 12:52:36 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org DA5C7388B03E Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=deneb.enyo.de Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=fw@deneb.enyo.de Received: from [172.17.203.2] (helo=deneb.enyo.de) by albireo.enyo.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) id 1jT3FT-0003P1-Dv; Mon, 27 Apr 2020 12:52:35 +0000 Received: from fw by deneb.enyo.de with local (Exim 4.92) (envelope-from ) id 1jT3FT-0003EX-AE; Mon, 27 Apr 2020 14:52:35 +0200 From: Florian Weimer To: Zack Weinberg Cc: GNU C Library Subject: Re: [PATCH] misc: Turn sstk into a compat symbol References: <87wo61xm8b.fsf@mid.deneb.enyo.de> Date: Mon, 27 Apr 2020 14:52:35 +0200 In-Reply-To: (Zack Weinberg's message of "Mon, 27 Apr 2020 08:38:26 -0400") Message-ID: <875zdlxf8s.fsf@mid.deneb.enyo.de> MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-21.8 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 27 Apr 2020 12:52:38 -0000 * Zack Weinberg: > On Mon, Apr 27, 2020 at 7:01 AM Florian Weimer wrote: >> >> It is not implemented anywhere. There is an osf_sstk system call on >> alpha, but it is not used to implement sstk, and the system call >> is not implemented on Linux, either. > > Maybe you should also remove the entry for sstk from > sysdeps/unix/syscalls.list? Indeed, and I also forgot to add attribute_compat_text_section. 8<------------------------------------------------------------------8< Subject: misc: Remove sstk from the autogenerated system call list This change should not have an effect because the system call was never defined. Also add the misssing attribute_compat_text_section attribute to the sstk function (a minor optimization). Fixes commit 9cc93ba0973ad04ee26c515a1552afb85e73c6ba ("misc: Turn sstk into a compat symbol"). diff --git a/misc/sstk.c b/misc/sstk.c index 0a2a967917..dda6f7b426 100644 --- a/misc/sstk.c +++ b/misc/sstk.c @@ -19,7 +19,7 @@ #include #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_32) -void * +void * attribute_compat_text_section sstk (int increment) { __set_errno (ENOSYS); diff --git a/sysdeps/unix/syscalls.list b/sysdeps/unix/syscalls.list index 01c4a0e6b1..e8f8718b04 100644 --- a/sysdeps/unix/syscalls.list +++ b/sysdeps/unix/syscalls.list @@ -78,7 +78,6 @@ sigaction - sigaction i:ipp __sigaction sigaction sigsuspend - sigsuspend Ci:p sigsuspend socket - socket i:iii __socket socket socketpair - socketpair i:iiif socketpair -sstk - sstk b:i sstk statfs - statfs i:sp __statfs statfs swapoff - swapoff i:s swapoff swapon - swapon i:s swapon