From patchwork Wed May 11 19:54:56 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella Netto X-Patchwork-Id: 53823 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 7C62A3836427 for ; Wed, 11 May 2022 19:57:36 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 7C62A3836427 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1652299056; bh=KizOQ6DewSQwNdP2yEnOUAJF7CSc/MQwromnrTl8PDo=; h=To:Subject:Date:In-Reply-To:References:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=dZevbAVIMI3xZa2rozNGRNLzFAE6eSlvWuzwo2C+Py3xe2gzW3/VBxCbXGL1uCiFQ HEN0EkMiL0tbyy9KvPNlsQUr/KnRg+qZTP1cnbmsVXzWEOy7I40Na5A4SMswCvCCjC TEegP0irCHAQ2cEZfDV9ZqpB+3PRaCdvZ2FMKO6Y= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from mail-oi1-x22f.google.com (mail-oi1-x22f.google.com [IPv6:2607:f8b0:4864:20::22f]) by sourceware.org (Postfix) with ESMTPS id 3937A3836430 for ; Wed, 11 May 2022 19:55:10 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 3937A3836430 Received: by mail-oi1-x22f.google.com with SMTP id y63so3969351oia.7 for ; Wed, 11 May 2022 12:55:10 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=KizOQ6DewSQwNdP2yEnOUAJF7CSc/MQwromnrTl8PDo=; b=fBv/1R+ZbLZZJ8InTTiReaN1nLm5H2QRWBhz95Z7B6InIKJ700NGwa2nmwaIj2V/1J 8Amq5qDkWXjzlQhuhpCI5enqozOs219GWIU8YkedkOUUusg+nSMpWVhkeI6ELIV6YPRD uSj/2tBBCavQfJsOHxoozGNikiceLAbzulkeljFAPf19zMtzUIPQwQHEH+DPhBI2V35u GaS4ySUdOQaZC/ioGtZVBsdQ8OANdCQkoUsyiU8oD5w9uKFYE2rM7FLkqqAUWIhP2bDx Z9ubprIrhs/VQBSj4kUv5Pb6WErniJOX+yAc+yIqQNzxJEyMSMt7K8DK0ABweX9lATQF 912Q== X-Gm-Message-State: AOAM531NKQSIxHTKVBAdVi6qa7J5xAxP7JdlcptSxZNUkiDz9iD9ABY3 SS37Y02qwfHF8u6hkzGqWyPF/9gJUj+dbA== X-Google-Smtp-Source: ABdhPJxUgqlfMmppeVVV6qEHRKsPYwDzq4F+o5NSfZDU1r4F68q/oZMsxXzTZ8wadFAdbnzufTYlFg== X-Received: by 2002:a05:6808:1892:b0:327:fc9c:94de with SMTP id bi18-20020a056808189200b00327fc9c94demr3378471oib.143.1652298909061; Wed, 11 May 2022 12:55:09 -0700 (PDT) Received: from birita.. ([2804:431:c7ca:5fbd:9d46:f0f6:e4b6:b83b]) by smtp.gmail.com with ESMTPSA id k22-20020a05687015d600b000edde90fac9sm886322oad.53.2022.05.11.12.55.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 11 May 2022 12:55:08 -0700 (PDT) To: libc-alpha@sourceware.org, Fangrui Song Subject: [PATCH 3/4] stdio: Remove the usage of $(fno-unit-at-a-time) for siglist.c Date: Wed, 11 May 2022 16:54:56 -0300 Message-Id: <20220511195457.410681-4-adhemerval.zanella@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220511195457.410681-1-adhemerval.zanella@linaro.org> References: <20220511195457.410681-1-adhemerval.zanella@linaro.org> MIME-Version: 1.0 X-Spam-Status: No, score=-12.2 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) 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-Patchwork-Original-From: Adhemerval Zanella via Libc-alpha From: Adhemerval Zanella Netto Reply-To: Adhemerval Zanella Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libc-alpha" The siglist.c is build with -fno-toplevel-reorder to avoid compiler to reorder the compat assembly directives due a static linker issue [1] (fixed on 2.39). This patch removes the compiler flags by split the compat symbol generation in two phases. First the __sys_siglist and __sys_sigabbrev without any compat symbol directive is preprocessed to generate an assembly source code. This generate assembly is then used as input on a platform agnostic siglist.S which then creates the compat definitions. This prevents compiler to move any compat directive prior the _sys_errlist_internal definition itself. Checked on a make check run-built-tests=no on all affected ABIs. --- include/signal.h | 6 ++--- stdio-common/Makefile | 14 +++++++++- stdio-common/{siglist.c => siglist-gen.c} | 4 --- stdio-common/siglist.S | 7 +++++ ...{siglist-compat.h => siglist-compat-def.h} | 27 ++++++++++--------- sysdeps/generic/siglist-compat.c | 1 - .../{siglist-compat.c => siglist-compat.h} | 7 +++-- .../{siglist-compat.c => siglist-compat.h} | 7 +++-- 8 files changed, 47 insertions(+), 26 deletions(-) rename stdio-common/{siglist.c => siglist-gen.c} (92%) create mode 100644 stdio-common/siglist.S rename sysdeps/generic/{siglist-compat.h => siglist-compat-def.h} (69%) delete mode 100644 sysdeps/generic/siglist-compat.c rename sysdeps/mach/hurd/{siglist-compat.c => siglist-compat.h} (86%) rename sysdeps/unix/sysv/linux/{siglist-compat.c => siglist-compat.h} (87%) diff --git a/include/signal.h b/include/signal.h index 2a278697c1..73f18dddd7 100644 --- a/include/signal.h +++ b/include/signal.h @@ -14,10 +14,8 @@ libc_hidden_proto (__sigpause) libc_hidden_proto (raise) libc_hidden_proto (__libc_current_sigrtmin) libc_hidden_proto (__libc_current_sigrtmax) -extern const char * const __sys_siglist[_NSIG]; -libc_hidden_proto (__sys_siglist) -extern const char * const __sys_sigabbrev[_NSIG]; -libc_hidden_proto (__sys_sigabbrev) +extern const char * const __sys_siglist[_NSIG] attribute_hidden; +extern const char * const __sys_sigabbrev[_NSIG] attribute_hidden; /* Now define the internal interfaces. */ extern __sighandler_t __bsd_signal (int __sig, __sighandler_t __handler); diff --git a/stdio-common/Makefile b/stdio-common/Makefile index 087acb82b6..98872d9f88 100644 --- a/stdio-common/Makefile +++ b/stdio-common/Makefile @@ -215,6 +215,8 @@ tests := \ generated += \ errlist-data-aux.os \ errlist-data-aux.o \ + siglist-aux.os \ + siglist-aux.o \ # generated test-srcs = tst-unbputc tst-printf tst-printfsz-islongdouble @@ -262,6 +264,17 @@ $(objpfx)errlist-data-aux.o: errlist-data-gen.c $(objpfx)errlist-data.os: $(objpfx)errlist-data-aux.os $(objpfx)errlist-data.o: $(objpfx)errlist-data-aux.o +$(objpfx)siglist-aux.os: siglist-gen.c + $(make-target-directory) + $(compile-command.c) $(no-stack-protector) -S + +$(objpfx)siglist-aux.o: siglist-gen.c + $(make-target-directory) + $(compile-command.c) $(no-stack-protector) -S + +$(objpfx)siglist.os: $(objpfx)siglist-aux.os +$(objpfx)siglist.o: $(objpfx)siglist-aux.o + ifeq ($(run-built-tests),yes) LOCALES := \ de_DE.ISO-8859-1 \ @@ -350,7 +363,6 @@ CFLAGS-isoc99_vfscanf.c += -fexceptions CFLAGS-isoc99_vscanf.c += -fexceptions CFLAGS-isoc99_fscanf.c += -fexceptions CFLAGS-isoc99_scanf.c += -fexceptions -CFLAGS-siglist.c += $(fno-unit-at-a-time) # scanf14a.c and scanf16a.c test a deprecated extension which is no # longer visible under most conformance levels; see the source files diff --git a/stdio-common/siglist.c b/stdio-common/siglist-gen.c similarity index 92% rename from stdio-common/siglist.c rename to stdio-common/siglist-gen.c index bb88d5300f..95441b707d 100644 --- a/stdio-common/siglist.c +++ b/stdio-common/siglist-gen.c @@ -26,7 +26,6 @@ const char *const __sys_siglist[NSIG] = #include #undef init_sig }; -libc_hidden_def (__sys_siglist) const char *const __sys_sigabbrev[NSIG] = { @@ -34,6 +33,3 @@ const char *const __sys_sigabbrev[NSIG] = #include #undef init_sig }; -libc_hidden_def (__sys_sigabbrev) - -#include diff --git a/stdio-common/siglist.S b/stdio-common/siglist.S new file mode 100644 index 0000000000..327f105e6f --- /dev/null +++ b/stdio-common/siglist.S @@ -0,0 +1,7 @@ +#ifdef SHARED +# include "siglist-aux.os" +#else +# include "siglist-aux.o" +#endif + +#include diff --git a/sysdeps/generic/siglist-compat.h b/sysdeps/generic/siglist-compat-def.h similarity index 69% rename from sysdeps/generic/siglist-compat.h rename to sysdeps/generic/siglist-compat-def.h index 05a0d63855..f5a1840dc3 100644 --- a/sysdeps/generic/siglist-compat.h +++ b/sysdeps/generic/siglist-compat-def.h @@ -28,20 +28,23 @@ sys_sigabbrev alias to __sys_sigabbrev. Both target alias are define in siglist.c. */ #define DEFINE_COMPAT_SIGLIST(NUMBERSIG, VERSION) \ - declare_symbol_alias (__ ## VERSION ## _sys_siglist, \ - __sys_siglist, \ - object, NUMBERSIG * (ULONG_WIDTH / UCHAR_WIDTH)); \ - declare_symbol_alias (__ ## VERSION ## sys_siglist, \ - __sys_siglist, \ - object, NUMBERSIG * (ULONG_WIDTH / UCHAR_WIDTH)); \ - declare_symbol_alias (__ ## VERSION ## _sys_sigabbrev, \ - __sys_sigabbrev, \ - object, NUMBERSIG * (ULONG_WIDTH / UCHAR_WIDTH)); \ + declare_object_symbol_alias (__ ## VERSION ## _sys_siglist, \ + __sys_siglist, \ + NUMBERSIG * (ULONG_WIDTH / UCHAR_WIDTH)) \ + ASM_LINE_SEP \ + declare_object_symbol_alias (__ ## VERSION ## sys_siglist, \ + __sys_siglist, \ + NUMBERSIG * (ULONG_WIDTH / UCHAR_WIDTH)) \ + ASM_LINE_SEP \ + declare_object_symbol_alias (__ ## VERSION ## _sys_sigabbrev, \ + __sys_sigabbrev, \ + NUMBERSIG * (ULONG_WIDTH / UCHAR_WIDTH)) \ + ASM_LINE_SEP \ compat_symbol (libc, __## VERSION ## _sys_siglist, _sys_siglist, \ - VERSION); \ + VERSION) ASM_LINE_SEP \ compat_symbol (libc, __## VERSION ## sys_siglist, sys_siglist, \ - VERSION); \ + VERSION) ASM_LINE_SEP \ compat_symbol (libc, __## VERSION ## _sys_sigabbrev, sys_sigabbrev, \ - VERSION); \ + VERSION) #endif diff --git a/sysdeps/generic/siglist-compat.c b/sysdeps/generic/siglist-compat.c deleted file mode 100644 index 6e25b021ab..0000000000 --- a/sysdeps/generic/siglist-compat.c +++ /dev/null @@ -1 +0,0 @@ -/* Empty. */ diff --git a/sysdeps/mach/hurd/siglist-compat.c b/sysdeps/mach/hurd/siglist-compat.h similarity index 86% rename from sysdeps/mach/hurd/siglist-compat.c rename to sysdeps/mach/hurd/siglist-compat.h index 5334b0885b..17bb0401aa 100644 --- a/sysdeps/mach/hurd/siglist-compat.c +++ b/sysdeps/mach/hurd/siglist-compat.h @@ -16,12 +16,15 @@ License along with the GNU C Library; if not, see . */ -#include +#include +/* To get _NSIG definition. */ +#define _SIGNAL_H +#include #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1) DEFINE_COMPAT_SIGLIST (33, GLIBC_2_0) #endif #if SHLIB_COMPAT (libc, GLIBC_2_1, GLIBC_2_32) -DEFINE_COMPAT_SIGLIST (NSIG, GLIBC_2_1) +DEFINE_COMPAT_SIGLIST (_NSIG, GLIBC_2_1) #endif diff --git a/sysdeps/unix/sysv/linux/siglist-compat.c b/sysdeps/unix/sysv/linux/siglist-compat.h similarity index 87% rename from sysdeps/unix/sysv/linux/siglist-compat.c rename to sysdeps/unix/sysv/linux/siglist-compat.h index cbc77289c0..2a9801276d 100644 --- a/sysdeps/unix/sysv/linux/siglist-compat.c +++ b/sysdeps/unix/sysv/linux/siglist-compat.h @@ -16,7 +16,10 @@ License along with the GNU C Library; if not, see . */ -#include +#include +/* To get _NSIG definition. */ +#define _SIGNAL_H +#include #if SHLIB_COMPAT (libc, GLIBC_2_0, GLIBC_2_1) DEFINE_COMPAT_SIGLIST (32, GLIBC_2_0) @@ -27,5 +30,5 @@ DEFINE_COMPAT_SIGLIST (64, GLIBC_2_1) #endif #if SHLIB_COMPAT (libc, GLIBC_2_3_3, GLIBC_2_32) -DEFINE_COMPAT_SIGLIST (NSIG, GLIBC_2_3_3) +DEFINE_COMPAT_SIGLIST (_NSIG, GLIBC_2_3_3) #endif