From patchwork Thu May 18 16:52:38 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Carlos O'Donell X-Patchwork-Id: 1783416 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=sourceware.org (client-ip=2620:52:3:1:0:246e:9693:128c; helo=sourceware.org; envelope-from=libc-alpha-bounces+incoming=patchwork.ozlabs.org@sourceware.org; receiver=) Authentication-Results: legolas.ozlabs.org; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=ogfcVokI; dkim-atps=neutral Received: from sourceware.org (server2.sourceware.org [IPv6:2620:52:3:1:0:246e:9693:128c]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4QMbd955t8z20dn for ; Fri, 19 May 2023 02:53:01 +1000 (AEST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id A51B53854179 for ; Thu, 18 May 2023 16:52:59 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org A51B53854179 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1684428779; bh=vVlL+sZm56qbwJalBPktbEOVUrwPpk6mmnZ8bEDYxHk=; h=To:Cc:Subject:Date:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From:Reply-To:From; b=ogfcVokIqsSc6kX7TfuiIL6+QMdGbWfC1RskfmqBT2lIpiBMKKWIDKafvV3Vdq/GK dT3gCaqyVsx2XZIB6ibDXDo5SJLzAouR/k1NoxJe2BArvdokwwyBw57kbwyQadpt41 mShlasA5SlxIzFQ47/sIXYjQq5gevdiHRFzntVKM= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id A4E023858CDB for ; Thu, 18 May 2023 16:52:43 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A4E023858CDB Received: from mail-yw1-f197.google.com (mail-yw1-f197.google.com [209.85.128.197]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-127-CBUiBlnIO2K0UjuxPRq5QQ-1; Thu, 18 May 2023 12:52:42 -0400 X-MC-Unique: CBUiBlnIO2K0UjuxPRq5QQ-1 Received: by mail-yw1-f197.google.com with SMTP id 00721157ae682-560ee0df572so35988527b3.0 for ; Thu, 18 May 2023 09:52:41 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684428761; x=1687020761; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=vVlL+sZm56qbwJalBPktbEOVUrwPpk6mmnZ8bEDYxHk=; b=Pyr6DGCpv/2/UwyB+X6g2vueXbe7yrHU8XtTiflRkiP2VY/7WQUp7X2iDjWuAQkpXV LqAb7m9bgS0OaIomvcHes7mW4gBdoIazEZ/3CHeWs8Q6cQnJi7XHEanwVTCpUaWuBN91 gx/lZM9QbZJXFw92icDQTQIq34jVpCEqt1uPkeyIHIAy/kcKORyuS90bKy7wMxNtl2KM Hk5vE8loVt73G5cFmxcRGrrz/TIEzg5QsGCzTfF/e+j2MAYCvJ41Vyk2INCbjBBuH9EW C3jsW0Z2e0bhLFEnY03VjF6Lw2S/NVErqeSttP6ohPNJe+AjQ1qp76Vv99oqQfhxAqxA 1hKg== X-Gm-Message-State: AC+VfDwg5GKSD4HXA7DJOFtwy/yK93vsgZOkpP+qZ7SlLxm2QZLxOt/9 SLRZuCjaaAzUpU25YkNphPAGH/M+evizU4xku408AMQylxiVsteVIn2+RMtu6HnKtAhEVvmM6dE JxiIHvxvQRz+7oOOQ0OQ/fb6I5Ot+A4i0xC+1FXOeENV2aYlKK/1NaJNByD1zxjxmLiGwV5MkP/ 7OrA== X-Received: by 2002:a81:5fd5:0:b0:55d:7d07:7fbb with SMTP id t204-20020a815fd5000000b0055d7d077fbbmr1807991ywb.27.1684428760921; Thu, 18 May 2023 09:52:40 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ67girJJx9Ewh38kVFiKQYSJHf6Ac1xADKCf/IEFRBhLEQGE3sl8wtXB715dyZpKJhg7uPyxw== X-Received: by 2002:a81:5fd5:0:b0:55d:7d07:7fbb with SMTP id t204-20020a815fd5000000b0055d7d077fbbmr1807976ywb.27.1684428760509; Thu, 18 May 2023 09:52:40 -0700 (PDT) Received: from DESKTOP-1DH6NSL.localdomain ([198.48.244.52]) by smtp.gmail.com with ESMTPSA id y16-20020a0dd610000000b00561608bc0dfsm535880ywd.56.2023.05.18.09.52.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 18 May 2023 09:52:40 -0700 (PDT) To: libc-alpha@sourceware.org Cc: Carlos O'Donell Subject: [COMMITTED] nptl: Reformat Makefile. Date: Thu, 18 May 2023 12:52:38 -0400 Message-Id: <20230518165238.68453-1-carlos@redhat.com> X-Mailer: git-send-email 2.40.0 MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-12.6 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) 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: Carlos O'Donell via Libc-alpha From: Carlos O'Donell Reply-To: Carlos O'Donell Errors-To: libc-alpha-bounces+incoming=patchwork.ozlabs.org@sourceware.org Sender: "Libc-alpha" Reflow all long lines adding comment terminators. Rename files that cause inconsistent ordering. Sort all reflowed text using scripts/sort-makefile-lines.py. No code generation changes observed in binary artifacts. No regressions on x86_64 and i686. --- nptl/Makefile | 275 +++++++++++++++------- nptl/{tst-mutexpi5a.c => tst-mutexpi11.c} | 0 nptl/{tst-mutexpi7a.c => tst-mutexpi12.c} | 0 nptl/{tst-rwlock2a.c => tst-rwlock21.c} | 0 nptl/{tst-rwlock2b.c => tst-rwlock22.c} | 0 5 files changed, 196 insertions(+), 79 deletions(-) rename nptl/{tst-mutexpi5a.c => tst-mutexpi11.c} (100%) rename nptl/{tst-mutexpi7a.c => tst-mutexpi12.c} (100%) rename nptl/{tst-rwlock2a.c => tst-rwlock21.c} (100%) rename nptl/{tst-rwlock2b.c => tst-rwlock22.c} (100%) diff --git a/nptl/Makefile b/nptl/Makefile index 8cec6faee3..f8365467d9 100644 --- a/nptl/Makefile +++ b/nptl/Makefile @@ -207,6 +207,7 @@ routines = \ tpp \ unwind \ vars \ + # routines static-only-routines = pthread_atfork libpthread-routines = libpthread-compat @@ -270,68 +271,131 @@ LDLIBS-tst-thread-exit-clobber = -lstdc++ CFLAGS-tst-minstack-throw.o = -std=gnu++11 LDLIBS-tst-minstack-throw = -lstdc++ -tests = tst-attr2 tst-attr3 tst-default-attr \ - tst-mutex5a tst-mutex7a \ - tst-mutexpi1 tst-mutexpi2 tst-mutexpi3 tst-mutexpi4 \ - tst-mutexpi5 tst-mutexpi5a tst-mutexpi6 tst-mutexpi7 tst-mutexpi7a \ - tst-mutexpi9 tst-mutexpi10 \ - tst-cond26 \ - tst-robustpi1 tst-robustpi2 tst-robustpi3 tst-robustpi4 tst-robustpi5 \ - tst-robustpi6 tst-robustpi7 tst-robustpi9 \ - tst-rwlock2 tst-rwlock2a tst-rwlock2b tst-rwlock3 \ - tst-rwlock6 tst-rwlock7 tst-rwlock8 \ - tst-rwlock9 tst-rwlock10 tst-rwlock11 \ - tst-rwlock15 tst-rwlock17 tst-rwlock18 \ - tst-once5 \ - tst-sem17 \ - tst-tsd3 tst-tsd4 \ - tst-cancel4_1 tst-cancel4_2 \ - tst-cancel7 tst-cancel17 tst-cancel24 \ - tst-signal3 \ - tst-exec4 tst-exec5 \ - tst-stack2 tst-stack3 tst-stack4 \ - tst-pthread-attr-affinity \ - tst-pthread-attr-affinity-fail \ - tst-dlsym1 \ - tst-context1 \ - tst-sched1 \ - tst-initializers1 $(addprefix tst-initializers1-,\ - c89 gnu89 c99 gnu99 c11 gnu11) \ - tst-thread_local1 \ - tst-robust-fork \ - tst-thread-exit-clobber tst-minstack-cancel tst-minstack-exit \ - tst-minstack-throw \ - tst-rwlock-pwn \ - tst-thread-affinity-pthread \ - tst-thread-affinity-pthread2 \ - tst-thread-affinity-sched \ - tst-pthread-defaultattr-free \ - tst-pthread-attr-sigmask \ - tst-pthread-timedlock-lockloop \ - tst-pthread-gdb-attach tst-pthread-gdb-attach-static \ - tst-pthread_exit-nothreads \ - tst-pthread_exit-nothreads-static \ - tst-thread-setspecific +tests = \ + tst-attr2 \ + tst-attr3 \ + tst-cancel4_1 \ + tst-cancel4_2 \ + tst-cancel7 \ + tst-cancel17 \ + tst-cancel24 \ + tst-cond26 \ + tst-context1 \ + tst-default-attr \ + tst-dlsym1 \ + tst-exec4 \ + tst-exec5 \ + tst-initializers1 \ + tst-initializers1-c11 \ + tst-initializers1-c89 \ + tst-initializers1-c99 \ + tst-initializers1-gnu11 \ + tst-initializers1-gnu89 \ + tst-initializers1-gnu99 \ + tst-minstack-cancel \ + tst-minstack-exit \ + tst-minstack-throw \ + tst-mutex5a \ + tst-mutex7a \ + tst-mutexpi1 \ + tst-mutexpi2 \ + tst-mutexpi3 \ + tst-mutexpi4 \ + tst-mutexpi5 \ + tst-mutexpi6 \ + tst-mutexpi7 \ + tst-mutexpi9 \ + tst-mutexpi10 \ + tst-mutexpi11 \ + tst-mutexpi12 \ + tst-once5 \ + tst-pthread-attr-affinity \ + tst-pthread-attr-affinity-fail \ + tst-pthread-attr-sigmask \ + tst-pthread-defaultattr-free \ + tst-pthread-gdb-attach \ + tst-pthread-gdb-attach-static \ + tst-pthread-timedlock-lockloop \ + tst-pthread_exit-nothreads \ + tst-pthread_exit-nothreads-static \ + tst-robust-fork \ + tst-robustpi1 \ + tst-robustpi2 \ + tst-robustpi3 \ + tst-robustpi4 \ + tst-robustpi5 \ + tst-robustpi6 \ + tst-robustpi7 \ + tst-robustpi9 \ + tst-rwlock-pwn \ + tst-rwlock2 \ + tst-rwlock3 \ + tst-rwlock6 \ + tst-rwlock7 \ + tst-rwlock8 \ + tst-rwlock9 \ + tst-rwlock10 \ + tst-rwlock11 \ + tst-rwlock15 \ + tst-rwlock17 \ + tst-rwlock18 \ + tst-rwlock21 \ + tst-rwlock22 \ + tst-sched1 \ + tst-sem17 \ + tst-signal3 \ + tst-stack2 \ + tst-stack3 \ + tst-stack4 \ + tst-thread-affinity-pthread \ + tst-thread-affinity-pthread2 \ + tst-thread-affinity-sched \ + tst-thread-exit-clobber \ + tst-thread-setspecific \ + tst-thread_local1 \ + tst-tsd3 \ + tst-tsd4 \ + # tests tests-nolibpthread = \ tst-pthread_exit-nothreads \ tst-pthread_exit-nothreads-static \ + # tests-nolibpthread tests-container = tst-pthread-getattr -tests-internal := tst-robustpi8 tst-rwlock19 tst-rwlock20 \ - tst-sem11 tst-sem12 tst-sem13 \ - tst-barrier5 tst-signal7 tst-mutex8 tst-mutex8-static \ - tst-mutexpi8 tst-mutexpi8-static \ - tst-setgetname \ - tst-cond22 \ - -xtests = tst-setuid1 tst-setuid1-static tst-setuid2 \ - tst-mutexpp1 tst-mutexpp6 tst-mutexpp10 tst-setgroups \ - tst-mutexpp5 tst-mutexpp9 +tests-internal := \ + tst-barrier5 \ + tst-cond22 \ + tst-mutex8 \ + tst-mutex8-static \ + tst-mutexpi8 \ + tst-mutexpi8-static \ + tst-robustpi8 \ + tst-rwlock19 \ + tst-rwlock20 \ + tst-sem11 \ + tst-sem12 \ + tst-sem13 \ + tst-setgetname \ + tst-signal7 \ + # tests-internal + +xtests = \ + tst-mutexpp1 \ + tst-mutexpp5 \ + tst-mutexpp6 \ + tst-mutexpp9 \ + tst-mutexpp10 \ + tst-setgroups \ + tst-setuid1 \ + tst-setuid1-static \ + tst-setuid2 \ + # xtests tests-time64 := \ tst-cancel4_2-time64 + # tests-time64 # This test can run into task limits because of a linux kernel bug # and then cause the make process to fail too, see bug 24537. @@ -343,9 +407,14 @@ gen-as-const-headers = unwindbuf.sym gen-py-const-headers := nptl_lock_constants.pysym pretty-printers := nptl-printers.py -tests-printers := test-mutexattr-printers test-mutex-printers \ - test-condattr-printers test-cond-printers \ - test-rwlockattr-printers test-rwlock-printers +tests-printers := \ + test-cond-printers \ + test-condattr-printers \ + test-mutex-printers \ + test-mutexattr-printers \ + test-rwlock-printers \ + test-rwlockattr-printers \ + # tests-printers # We must specify both CFLAGS and CPPFLAGS to override any # compiler options the user might have provided that conflict @@ -380,25 +449,51 @@ CPPFLAGS-tst-pthread-gdb-attach-static.c := \ # were launched with an explicit ld.so invocation. tst-pthread-gdb-attach-no-pie = yes -tests += tst-cancelx7 tst-cancelx17 +tests += \ + tst-cancelx7 \ + tst-cancelx17 \ + # tests ifeq ($(build-shared),yes) -tests += tst-compat-forwarder tst-audit-threads -tests-internal += tst-tls3 tst-tls3-malloc tst-tls5 tst-stackguard1 +tests += \ + tst-audit-threads \ + tst-compat-forwarder \ + # tests +tests-internal += \ + tst-stackguard1 \ + tst-tls3 \ + tst-tls3-malloc \ + tst-tls5 \ + # tests-internal ifeq ($(have-z-execstack),yes) tests += tst-execstack endif endif -modules-names = tst-tls3mod \ - tst-tls5mod tst-tls5moda tst-tls5modb tst-tls5modc \ - tst-tls5modd tst-tls5mode tst-tls5modf tst-stack4mod \ - tst-execstack-mod \ - tst-compat-forwarder-mod tst-audit-threads-mod1 \ - tst-audit-threads-mod2 -extra-test-objs += $(addsuffix .os,$(strip $(modules-names))) \ - tst-cleanup4aux.o tst-cleanupx4aux.o -test-extras += tst-cleanup4aux tst-cleanupx4aux +modules-names = \ + tst-audit-threads-mod1 \ + tst-audit-threads-mod2 \ + tst-compat-forwarder-mod \ + tst-execstack-mod \ + tst-stack4mod \ + tst-tls3mod \ + tst-tls5mod \ + tst-tls5moda \ + tst-tls5modb \ + tst-tls5modc \ + tst-tls5modd \ + tst-tls5mode \ + tst-tls5modf \ + # modules-names +extra-test-objs += \ + $(addsuffix .os,$(strip $(modules-names))) \ + tst-cleanup4aux.o \ + tst-cleanupx4aux.o \ + # extra-test-objs +test-extras += \ + tst-cleanup4aux \ + tst-cleanupx4aux \ + # test-extras # This test exercises compat symbols removed in glibc 2.34. ifdef have-GLIBC_2.33 @@ -436,20 +531,31 @@ CFLAGS-funlockfile.c += $(libio-mtsafe) link-libc-static := $(common-objpfx)libc.a $(static-gnulib) \ $(common-objpfx)libc.a -tests-static += tst-stackguard1-static \ - tst-cancel24-static \ - tst-mutex8-static tst-mutexpi8-static tst-sem11-static \ - tst-sem12-static tst-cond11-static \ - tst-pthread-gdb-attach-static \ - tst-pthread_exit-nothreads-static +tests-static += \ + tst-cancel24-static \ + tst-mutex8-static \ + tst-mutexpi8-static \ + tst-pthread-gdb-attach-static \ + tst-pthread_exit-nothreads-static \ + tst-sem11-static \ + tst-sem12-static tst-cond11-static \ + tst-stackguard1-static \ + # tests-static tests += tst-cancel24-static -tests-internal += tst-sem11-static tst-sem12-static tst-stackguard1-static +tests-internal += \ + tst-sem11-static \ + tst-sem12-static \ + tst-stackguard1-static \ + # tests-internal xtests-static += tst-setuid1-static ifeq ($(run-built-tests),yes) -tests-special += $(objpfx)tst-stack3-mem.out $(objpfx)tst-oddstacklimit.out +tests-special += \ + $(objpfx)tst-oddstacklimit.out \ + $(objpfx)tst-stack3-mem.out \ + # tests-special ifeq ($(build-shared),yes) tests-special += $(objpfx)tst-tls6.out endif @@ -457,8 +563,13 @@ endif ifeq (,$(CXX)) # These tests require a C++ compiler and runtime. -tests-unsupported += tst-cancel24 tst-cancel24-static tst-once5 \ - tst-thread-exit-clobber tst-minstack-throw +tests-unsupported += \ + tst-cancel24 \ + tst-cancel24-static \ + tst-minstack-throw \ + tst-once5 \ + tst-thread-exit-clobber \ + # tests-unsupported endif # These tests require a C++ compiler and runtime with thread_local support. ifneq ($(have-cxx-thread_local),yes) @@ -504,7 +615,10 @@ tst-stack3-ENV = MALLOC_TRACE=$(objpfx)tst-stack3.mtrace \ $(objpfx)tst-stack3-mem.out: $(objpfx)tst-stack3.out $(common-objpfx)malloc/mtrace $(objpfx)tst-stack3.mtrace > $@; \ $(evaluate-test) -generated += tst-stack3-mem.out tst-stack3.mtrace +generated += \ + tst-stack3-mem.out \ + tst-stack3.mtrace \ + # generated tst-stack4mod.sos=$(shell for i in 0 1 2 3 4 5 6 7 8 9 10 \ 11 12 13 14 15 16 17 18 19; do \ @@ -546,7 +660,10 @@ LDLIBS-tst-cancel24-static = $(LDLIBS-tst-cancel24) ifeq ($(build-shared),yes) -generated += multidir.mk tst-tls6.out +generated += \ + multidir.mk \ + tst-tls6.out \ + # generated endif tst-exec4-ARGS = $(host-test-program-cmd) diff --git a/nptl/tst-mutexpi5a.c b/nptl/tst-mutexpi11.c similarity index 100% rename from nptl/tst-mutexpi5a.c rename to nptl/tst-mutexpi11.c diff --git a/nptl/tst-mutexpi7a.c b/nptl/tst-mutexpi12.c similarity index 100% rename from nptl/tst-mutexpi7a.c rename to nptl/tst-mutexpi12.c diff --git a/nptl/tst-rwlock2a.c b/nptl/tst-rwlock21.c similarity index 100% rename from nptl/tst-rwlock2a.c rename to nptl/tst-rwlock21.c diff --git a/nptl/tst-rwlock2b.c b/nptl/tst-rwlock22.c similarity index 100% rename from nptl/tst-rwlock2b.c rename to nptl/tst-rwlock22.c