From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dormouse.elm.relay.mailchannels.net (dormouse.elm.relay.mailchannels.net [23.83.212.50]) by sourceware.org (Postfix) with ESMTPS id 15CEE3858D3C for ; Wed, 22 Mar 2023 16:25:39 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 15CEE3858D3C Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=gotplt.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gotplt.org X-Sender-Id: dreamhost|x-authsender|siddhesh@gotplt.org Received: from relay.mailchannels.net (localhost [127.0.0.1]) by relay.mailchannels.net (Postfix) with ESMTP id 5AE92501696; Wed, 22 Mar 2023 16:25:36 +0000 (UTC) Received: from pdx1-sub0-mail-a306.dreamhost.com (unknown [127.0.0.6]) (Authenticated sender: dreamhost) by relay.mailchannels.net (Postfix) with ESMTPA id EB0685015C0; Wed, 22 Mar 2023 16:25:35 +0000 (UTC) ARC-Seal: i=1; s=arc-2022; d=mailchannels.net; t=1679502336; a=rsa-sha256; cv=none; b=CMaoFXQOQCaBHcO+fuTNKfi3ht81fWnN0+V8sTHvMpGNvMNGX0OImP5PcbwxcS1geh+nZ6 I08bPSEg64TxMZB9+hNIpSkBWhqQ+4UqTaDnylX9WKLIoi6VI1exk4mekLpXyacQYo7PyA GCpa3FXuG5TmvtyuCGMCJbkeHDcJVoaCDGd31jJOUb9lE1LtAVjlfNd1mxHDQZby26qfYz XpW4xcwuZzJ3m6LBxELpnEHK5tQcOqZqVIzh3zn40dmXn60AaXMK0iX26kv0BCQcAmT2dG DQxstwZeQm2vPZzzIWbT74zdASGTNpy1aHp37oNO82DmGPYlyAyNl5kcyXeuCg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=mailchannels.net; s=arc-2022; t=1679502336; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=pT1Jgi0CEJqvz9WR2N5/iJWWhzvBxTSIotVwIgC83Gg=; b=awdEUCdzNVUUU0uqwyPC6xxRM78vz+G0lYVVT/0uajER+Fjitr4/k6cv0BaxIYicRM4hzk dPDHG+l+asY7MS/uaBFA1Vs2d/ggLU1dVdHgVvC9jQRxm+NbCbVudtm96sbMuOd70TySVn TFMwKRRB+WYNohrHmUP/HXhLYzqhQoRjSPrptILmOUG1n/kvk5J/9dXk8bJBQQMuLJjsYG AR03PJf8Pk6YrnpSN/9lJurccF1x/vOd+Nbruo9wLo2MwPK35Yxq3coGF/au1oM2pmHT2I xnDassC4sDPiMRqXIdZXNOBL6xQIFMTs7ykOQydIRxDy11EcDj9wiSIJsFLcmQ== ARC-Authentication-Results: i=1; rspamd-c98fcbcd4-f5z5z; auth=pass smtp.auth=dreamhost smtp.mailfrom=siddhesh@gotplt.org X-Sender-Id: dreamhost|x-authsender|siddhesh@gotplt.org X-MC-Relay: Neutral X-MailChannels-SenderId: dreamhost|x-authsender|siddhesh@gotplt.org X-MailChannels-Auth-Id: dreamhost X-Eyes-Exultant: 0e51ef0c295b8774_1679502336223_472702786 X-MC-Loop-Signature: 1679502336223:28960100 X-MC-Ingress-Time: 1679502336223 Received: from pdx1-sub0-mail-a306.dreamhost.com (pop.dreamhost.com [64.90.62.162]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384) by 100.102.66.37 (trex/6.7.2); Wed, 22 Mar 2023 16:25:36 +0000 Received: from [192.168.0.182] (bras-vprn-toroon4834w-lp130-09-174-91-45-153.dsl.bell.ca [174.91.45.153]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: siddhesh@gotplt.org) by pdx1-sub0-mail-a306.dreamhost.com (Postfix) with ESMTPSA id 4PhYjq1yq0z3D; Wed, 22 Mar 2023 09:25:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gotplt.org; s=dreamhost; t=1679502335; bh=pT1Jgi0CEJqvz9WR2N5/iJWWhzvBxTSIotVwIgC83Gg=; h=Date:Subject:To:From:Content-Type:Content-Transfer-Encoding; b=EebcqYJlWlFyggRqgn6GDJoxMa+nTkNhDi7Nsi01++b9/XeRHsNAOPFJnxLAI0dOV 0kMURxGMsV1TPrn4ki0OiTmd/KxxP8tvRNX+6a+OI9K/v0IiMr5q7GngGhhysIp03n Q8WE0GPRVKJBiShhV+gkERnZ0XKmg/f9qlEU0EEmfAQIioVfAXDwnsWeyeMQ9bp6wm MwNLLg/C7/E64FtyiUVAMWGo13DUcJKq8z9mRkQMSY3SDsrXOb3lBVv1IBhtAG4lJM Vsiv5OgNkff2WRuCTPgzvCf050umw1IowCSQnKCbHKthO+N7rraF2m/VU3IgZGI2zK its66uZCzKTGw== Message-ID: <0607ab4d-868b-bba9-d58a-d055a09b73bf@gotplt.org> Date: Wed, 22 Mar 2023 12:25:34 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.8.0 Subject: Re: [PATCH 2/2] Remove --enable-tunables configure option Content-Language: en-US To: Adhemerval Zanella , libc-alpha@sourceware.org, Florian Weimer References: <20230313190627.2000578-1-adhemerval.zanella@linaro.org> <20230313190627.2000578-3-adhemerval.zanella@linaro.org> From: Siddhesh Poyarekar In-Reply-To: <20230313190627.2000578-3-adhemerval.zanella@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-3034.5 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,KAM_SHORT,NICE_REPLY_A,RCVD_IN_BL_SPAMCOP_NET,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 2023-03-13 15:06, Adhemerval Zanella wrote: > And make always supported. The configure option was added on glibc 2.25 > and some option require it (such as hwcap mask, huge pages support, and > lock elisition tuning). It also simplifies the build permutations. > --- > INSTALL | 19 --- > Makeconfig | 4 - > NEWS | 3 + > config.h.in | 3 - > config.make.in | 1 - > configure | 17 --- > configure.ac | 10 -- > elf/Makefile | 8 +- > elf/dl-cache.c | 2 +- > elf/dl-hwcaps.h | 11 -- > elf/dl-support.c | 15 --- > elf/dl-tls.c | 6 - > elf/dl-tunables.h | 35 ++---- > elf/dl-usage.c | 8 +- > elf/rtld.c | 23 +--- > elf/tst-env-setuid-tunables.c | 11 -- > gmon/gmon.c | 12 +- > malloc/Makefile | 2 - > malloc/arena.c | 118 +----------------- > malloc/malloc-check.c | 8 -- > malloc/malloc.c | 14 +-- > manual/install.texi | 19 --- > nptl/pthread_mutex_conf.c | 4 +- > nptl/tst-mutex8.c | 2 - > scripts/build-many-glibcs.py | 1 - > sysdeps/generic/ldsodefs.h | 17 --- > sysdeps/generic/unsecvars.h | 8 +- > sysdeps/nptl/dl-tls_init_tp.c | 2 - > sysdeps/nptl/pthreadP.h | 4 - > sysdeps/nptl/pthread_early_init.h | 2 - > sysdeps/nptl/pthread_mutex_conf.h | 8 -- > sysdeps/powerpc/cpu-features.c | 9 +- > sysdeps/s390/cpu-features.c | 14 +-- > sysdeps/sparc/sparc32/dl-machine.h | 5 +- > sysdeps/unix/sysv/linux/Makefile | 4 +- > .../unix/sysv/linux/aarch64/cpu-features.c | 6 - > .../unix/sysv/linux/powerpc/elision-conf.c | 8 +- > sysdeps/unix/sysv/linux/s390/elision-conf.c | 8 +- > sysdeps/unix/sysv/linux/x86/elision-conf.c | 8 +- > sysdeps/unix/sysv/linux/x86_64/64/Makefile | 2 - > sysdeps/x86/Makefile | 10 -- > sysdeps/x86/cacheinfo.h | 8 +- > sysdeps/x86/cpu-features.c | 24 +--- > sysdeps/x86/cpu-tunables.c | 45 ++++--- > sysdeps/x86/dl-cacheinfo.h | 10 -- > sysdeps/x86_64/Makefile | 2 - > 46 files changed, 72 insertions(+), 488 deletions(-) > > diff --git a/INSTALL b/INSTALL > index 970d6627e2..021943d83f 100644 > --- a/INSTALL > +++ b/INSTALL > @@ -234,25 +234,6 @@ if 'CFLAGS' is specified it must enable optimization. For example: > By default for x86_64, the GNU C Library is built with the vector > math library. Use this option to disable the vector math library. > > -'--enable-tunables' > - Tunables support allows additional library parameters to be > - customized at runtime. This feature is enabled by default. This > - option can take the following values: > - > - 'yes' > - This is the default if no option is passed to configure. This > - enables tunables and selects the default frontend (currently > - 'valstring'). > - > - 'no' > - This option disables tunables. > - > - 'valstring' > - This enables tunables and selects the 'valstring' frontend for > - tunables. This frontend allows users to specify tunables as a > - colon-separated list in a single environment variable > - 'GLIBC_TUNABLES'. > - > '--disable-crypt' > Do not install the passphrase-hashing library 'libcrypt' or the > header file 'crypt.h'. 'unistd.h' will still declare the function > diff --git a/Makeconfig b/Makeconfig > index 2fda4af5f7..3dd69d83b6 100644 > --- a/Makeconfig > +++ b/Makeconfig > @@ -1019,9 +1019,7 @@ CPPFLAGS = $(config-extra-cppflags) $(CPPUNDEFS) $(CPPFLAGS-config) \ > $(libof-$( $(CPPFLAGS-$( > -ifneq (no,$(have-tunables)) > CPPFLAGS += -DTOP_NAMESPACE=glibc > -endif Maybe just merge this into the CPPFLAGS initialization above? > > override CFLAGS = -std=gnu11 -fgnu89-inline $(config-extra-cflags) \ > $(filter-out %frame-pointer,$(+cflags)) $(+gccwarn-c) \ > @@ -1203,7 +1201,6 @@ endif > > # Build the tunables list header early since it could be used by any module in > # glibc. > -ifneq (no,$(have-tunables)) > before-compile += $(common-objpfx)dl-tunable-list.h > common-generated += dl-tunable-list.h dl-tunable-list.stmp > > @@ -1216,7 +1213,6 @@ $(common-objpfx)dl-tunable-list.stmp: \ > $(AWK) -f $^ > ${@:stmp=T} > $(move-if-change) ${@:stmp=T} ${@:stmp=h} > touch $@ > -endif > > # Dump the GCC macros used by the default compiler flags to a header > # file, so that they can be inspected when using different compiler > diff --git a/NEWS b/NEWS > index 0f76e7a416..3958e133a1 100644 > --- a/NEWS > +++ b/NEWS > @@ -28,6 +28,9 @@ Deprecated and removed features, and other changes affecting compatibility: > alternatively stable kernels from versions 6.1.6, 5.15.87, 5.10.163, > 5.4.228, 4.19.270 or 4.14.303. > > +* The configure option --enable-tunables has been removed. The tunable > + features is now always supported. > + Suggest: The tunables feature is now always enabled. > Changes to build and runtime requirements: > > [Add changes to build and runtime requirements here] > diff --git a/config.h.in b/config.h.in > index 09730d9d52..c87008b6a9 100644 > --- a/config.h.in > +++ b/config.h.in > @@ -271,9 +271,6 @@ > /* PowerPC32 uses fctidz for floating point to long long conversions. */ > #define HAVE_PPC_FCTIDZ 0 > > -/* Build glibc with tunables support. */ > -#define HAVE_TUNABLES 0 > - > /* Define if PIE is unsupported. */ > #undef PIE_UNSUPPORTED > > diff --git a/config.make.in b/config.make.in > index d7c416cbea..9186a8aac9 100644 > --- a/config.make.in > +++ b/config.make.in > @@ -96,7 +96,6 @@ build-nscd = @build_nscd@ > use-nscd = @use_nscd@ > build-hardcoded-path-in-tests= @hardcoded_path_in_tests@ > build-pt-chown = @build_pt_chown@ > -have-tunables = @have_tunables@ > pthread-in-libc = @pthread_in_libc@ > > # Build tools. > diff --git a/configure b/configure > index efb891456a..f357aa0b9c 100755 > --- a/configure > +++ b/configure > @@ -664,7 +664,6 @@ INSTALL_DATA > INSTALL_SCRIPT > INSTALL_PROGRAM > base_machine > -have_tunables > build_pt_chown > build_nscd > libc_cv_static_nss_crypt > @@ -784,7 +783,6 @@ enable_systemtap > enable_build_nscd > enable_nscd > enable_pt_chown > -enable_tunables > enable_mathvec > enable_cet > enable_scv > @@ -1454,8 +1452,6 @@ Optional Features: > --disable-build-nscd disable building and installing the nscd daemon > --disable-nscd library functions will not contact the nscd daemon > --enable-pt_chown Enable building and installing pt_chown > - --enable-tunables Enable tunables support. Known values are 'yes', > - 'no' and 'valstring' > --enable-mathvec Enable building and installing mathvec [default > depends on architecture] > --enable-cet enable Intel Control-flow Enforcement Technology > @@ -3684,19 +3680,6 @@ if test "$build_pt_chown" = yes; then > > fi > > -# Check whether --enable-tunables was given. > -if test "${enable_tunables+set}" = set; then : > - enableval=$enable_tunables; have_tunables=$enableval > -else > - have_tunables=yes > -fi > - > - > -if test "$have_tunables" = yes; then > - $as_echo "#define HAVE_TUNABLES 1" >>confdefs.h > - > -fi > - > # The abi-tags file uses a fairly simplistic model for name recognition that > # can't distinguish i486-pc-linux-gnu fully from i486-pc-gnu. So we mutate a > # $host_os of `gnu*' here to be `gnu-gnu*' just so that it can tell. > diff --git a/configure.ac b/configure.ac > index 011844a3d4..6b316bfd44 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -441,16 +441,6 @@ if test "$build_pt_chown" = yes; then > AC_DEFINE(HAVE_PT_CHOWN) > fi > > -AC_ARG_ENABLE([tunables], > - [AS_HELP_STRING([--enable-tunables], > - [Enable tunables support. Known values are 'yes', 'no' and 'valstring'])], > - [have_tunables=$enableval], > - [have_tunables=yes]) > -AC_SUBST(have_tunables) > -if test "$have_tunables" = yes; then > - AC_DEFINE(HAVE_TUNABLES) > -fi > - > # The abi-tags file uses a fairly simplistic model for name recognition that > # can't distinguish i486-pc-linux-gnu fully from i486-pc-gnu. So we mutate a > # $host_os of `gnu*' here to be `gnu-gnu*' just so that it can tell. > diff --git a/elf/Makefile b/elf/Makefile > index 4d0e04b2a2..75bfd82c34 100644 > --- a/elf/Makefile > +++ b/elf/Makefile > @@ -90,10 +90,9 @@ ifeq (yes,$(use-ldconfig)) > dl-routines += dl-cache > endif > > -ifneq (no,$(have-tunables)) > dl-routines += dl-tunables > -tunables-type = $(addprefix TUNABLES_FRONTEND_,$(have-tunables)) > -CPPFLAGS-dl-tunables.c += -DTUNABLES_FRONTEND=$(tunables-type) > +tunables-type = $(addprefix TUNABLES_FRONTEND_,yes) > +CPPFLAGS-dl-tunables.c += -DTUNABLES_FRONTEND=1 Couldn't we get rid of TUNABLES_FRONTEND too? > > ifeq (yesyes,$(build-shared)$(run-built-tests)) > tests-special += $(objpfx)list-tunables.out > @@ -104,7 +103,6 @@ endif > ifeq (yes,$(have-loop-to-function)) > CFLAGS-dl-tunables.c += -fno-tree-loop-distribute-patterns > endif > -endif > > all-dl-routines = $(dl-routines) $(sysdep-dl-routines) > # But they are absent from the shared libc, because that code is in ld.so. > @@ -1187,7 +1185,6 @@ postclean-generated += $(objpfx)/dso-sort-tests-2.generated-makefile \ > $(objpfx)/dso-sort-tests-2.generated-makefile > > # Generate from each testcase description file > -ifeq (yes,$(have-tunables)) > $(eval $(call include_dsosort_tests,dso-sort-tests-1.def)) > $(eval $(call include_dsosort_tests,dso-sort-tests-2.def)) > > @@ -1195,7 +1192,6 @@ $(eval $(call include_dsosort_tests,dso-sort-tests-2.def)) > LDFLAGS-tst-bz15311-b.so += -Wl,-z,lazy > LDFLAGS-tst-bz15311-c.so += -Wl,-z,lazy > LDFLAGS-tst-bz15311-d.so += -Wl,-z,lazy > -endif > > check-abi: $(objpfx)check-abi-ld.out \ > $(objpfx)check-abi-version-libc.out > diff --git a/elf/dl-cache.c b/elf/dl-cache.c > index 07c054b11a..631286832d 100644 > --- a/elf/dl-cache.c > +++ b/elf/dl-cache.c > @@ -205,7 +205,7 @@ search_cache (const char *string_table, uint32_t string_table_size, > uint64_t platform = _dl_string_platform (GLRO (dl_platform)); > if (platform != (uint64_t) -1) > platform = 1ULL << platform; > - uint64_t hwcap_mask = GET_HWCAP_MASK (); > + uint64_t hwcap_mask = TUNABLE_GET (glibc, cpu, hwcap_mask, uint64_t, NULL); > #define _DL_HWCAP_TLS_MASK (1LL << 63) > uint64_t hwcap_exclude = ~((GLRO (dl_hwcap) & hwcap_mask) > | _DL_HWCAP_PLATFORM | _DL_HWCAP_TLS_MASK); > diff --git a/elf/dl-hwcaps.h b/elf/dl-hwcaps.h > index abe6aaf7c5..3012ebf2b4 100644 > --- a/elf/dl-hwcaps.h > +++ b/elf/dl-hwcaps.h > @@ -24,17 +24,6 @@ > > #include > > -#if HAVE_TUNABLES > -# define GET_HWCAP_MASK() TUNABLE_GET (glibc, cpu, hwcap_mask, uint64_t, NULL) > -#else > -# ifdef SHARED > -# define GET_HWCAP_MASK() GLRO(dl_hwcap_mask) > -# else > -/* HWCAP_MASK is ignored in static binaries when built without tunables. */ > -# define GET_HWCAP_MASK() (0) > -# endif > -#endif > - > #define GLIBC_HWCAPS_SUBDIRECTORY "glibc-hwcaps" > #define GLIBC_HWCAPS_PREFIX GLIBC_HWCAPS_SUBDIRECTORY "/" > > diff --git a/elf/dl-support.c b/elf/dl-support.c > index d2519ce1a9..322599916f 100644 > --- a/elf/dl-support.c > +++ b/elf/dl-support.c > @@ -168,16 +168,6 @@ enum dso_sort_algorithm _dl_dso_sort_algo; > /* The value of the FPU control word the kernel will preset in hardware. */ > fpu_control_t _dl_fpu_control = _FPU_DEFAULT; > > -#if !HAVE_TUNABLES > -/* This is not initialized to HWCAP_IMPORTANT, matching the definition > - of _dl_important_hwcaps, below, where no hwcap strings are ever > - used. This mask is still used to mediate the lookups in the cache > - file. Since there is no way to set this nonzero (we don't grok the > - LD_HWCAP_MASK environment variable here), there is no real point in > - setting _dl_hwcap nonzero below, but we do anyway. */ > -uint64_t _dl_hwcap_mask; > -#endif > - > /* Prevailing state of the stack. Generally this includes PF_X, indicating it's > * executable but this isn't true for all platforms. */ > ElfW(Word) _dl_stack_flags = DEFAULT_STACK_PERMS; > @@ -328,11 +318,6 @@ _dl_non_dynamic_init (void) > __unsetenv (cp); > cp = strchr (cp, '\0') + 1; > } > - > -#if !HAVE_TUNABLES > - if (__access ("/etc/suid-debug", F_OK) != 0) > - __unsetenv ("MALLOC_CHECK_"); > -#endif > } > > #ifdef DL_PLATFORM_INIT > diff --git a/elf/dl-tls.c b/elf/dl-tls.c > index 8943a3b4ae..4ef7bc3f1e 100644 > --- a/elf/dl-tls.c > +++ b/elf/dl-tls.c > @@ -97,14 +97,8 @@ _dl_tls_static_surplus_init (size_t naudit) > { > size_t nns, opt_tls; > > -#if HAVE_TUNABLES > nns = TUNABLE_GET (nns, size_t, NULL); > opt_tls = TUNABLE_GET (optional_static_tls, size_t, NULL); > -#else > - /* Default values of the tunables. */ > - nns = DEFAULT_NNS; > - opt_tls = OPTIONAL_TLS; > -#endif > if (nns > DL_NNS) > nns = DL_NNS; > if (DL_NNS - nns < naudit) > diff --git a/elf/dl-tunables.h b/elf/dl-tunables.h > index ae6e014b95..7c7de704e9 100644 > --- a/elf/dl-tunables.h > +++ b/elf/dl-tunables.h > @@ -21,17 +21,9 @@ > #ifndef _TUNABLES_H_ > #define _TUNABLES_H_ > > -#if !HAVE_TUNABLES > -static inline void > -__always_inline > -__tunables_init (char **unused __attribute__ ((unused))) > -{ > - /* This is optimized out if tunables are not enabled. */ > -} > -#else > -# include > -# include > -# include > +#include > +#include > +#include > > typedef intmax_t tunable_num_t; > > @@ -44,12 +36,12 @@ typedef union > typedef void (*tunable_callback_t) (tunable_val_t *); > > /* Full name for a tunable is top_ns.tunable_ns.id. */ > -# define TUNABLE_NAME_S(top,ns,id) #top "." #ns "." #id > +#define TUNABLE_NAME_S(top,ns,id) #top "." #ns "." #id > > -# define TUNABLE_ENUM_NAME(__top,__ns,__id) TUNABLE_ENUM_NAME1 (__top,__ns,__id) > -# define TUNABLE_ENUM_NAME1(__top,__ns,__id) __top ## _ ## __ns ## _ ## __id > +#define TUNABLE_ENUM_NAME(__top,__ns,__id) TUNABLE_ENUM_NAME1 (__top,__ns,__id) > +#define TUNABLE_ENUM_NAME1(__top,__ns,__id) __top ## _ ## __ns ## _ ## __id > > -# include "dl-tunable-list.h" > +#include "dl-tunable-list.h" > > extern void __tunables_init (char **); > extern void __tunables_print (void); > @@ -83,7 +75,7 @@ rtld_hidden_proto (__tunable_set_val) > > /* Get and return a tunable value. If the tunable was set externally and __CB > is defined then call __CB before returning the value. */ > -# define TUNABLE_GET_FULL(__top, __ns, __id, __type, __cb) \ > +#define TUNABLE_GET_FULL(__top, __ns, __id, __type, __cb) \ > ({ \ > tunable_id_t id = TUNABLE_ENUM_NAME (__top, __ns, __id); \ > __type ret; \ > @@ -92,14 +84,14 @@ rtld_hidden_proto (__tunable_set_val) > }) > > /* Set a tunable value. */ > -# define TUNABLE_SET_FULL(__top, __ns, __id, __val) \ > +#define TUNABLE_SET_FULL(__top, __ns, __id, __val) \ > ({ \ > __tunable_set_val (TUNABLE_ENUM_NAME (__top, __ns, __id), \ > & (tunable_val_t) {.numval = __val}, NULL, NULL); \ > }) > > /* Set a tunable value together with min/max values. */ > -# define TUNABLE_SET_WITH_BOUNDS_FULL(__top, __ns, __id,__val, __min, __max) \ > +#define TUNABLE_SET_WITH_BOUNDS_FULL(__top, __ns, __id,__val, __min, __max) \ > ({ \ > __tunable_set_val (TUNABLE_ENUM_NAME (__top, __ns, __id), \ > & (tunable_val_t) {.numval = __val}, \ > @@ -109,11 +101,11 @@ rtld_hidden_proto (__tunable_set_val) > > /* Namespace sanity for callback functions. Use this macro to keep the > namespace of the modules clean. */ > -# define TUNABLE_CALLBACK(__name) _dl_tunable_ ## __name > +#define TUNABLE_CALLBACK(__name) _dl_tunable_ ## __name > > -# define TUNABLES_FRONTEND_valstring 1 > +#define TUNABLES_FRONTEND_valstring 1 > /* The default value for TUNABLES_FRONTEND. */ > -# define TUNABLES_FRONTEND_yes TUNABLES_FRONTEND_valstring > +#define TUNABLES_FRONTEND_yes TUNABLES_FRONTEND_valstring > > static __always_inline bool > tunable_val_lt (tunable_num_t lhs, tunable_num_t rhs, bool unsigned_cmp) > @@ -149,4 +141,3 @@ tunable_is_name (const char *orig, const char *envname) > } > > #endif > -#endif > diff --git a/elf/dl-usage.c b/elf/dl-usage.c > index 53535c1583..98f0b0d027 100644 > --- a/elf/dl-usage.c > +++ b/elf/dl-usage.c > @@ -195,12 +195,8 @@ setting environment variables (which would be inherited by subprocesses).\n\ > in LIST\n\ > --audit LIST use objects named in LIST as auditors\n\ > --preload LIST preload objects named in LIST\n\ > - --argv0 STRING set argv[0] to STRING before running\n" > -#if HAVE_TUNABLES > -"\ > - --list-tunables list all tunables with minimum and maximum values\n" > -#endif > -"\ > + --argv0 STRING set argv[0] to STRING before running\n\ > + --list-tunables list all tunables with minimum and maximum values\n\ The indentation seems a bit off, but maybe that's just my email client. > --list-diagnostics list diagnostics information\n\ > --help display this help and exit\n\ > --version output version information and exit\n\ > diff --git a/elf/rtld.c b/elf/rtld.c > index f82fbeb132..c1e383b055 100644 > --- a/elf/rtld.c > +++ b/elf/rtld.c > @@ -357,9 +357,6 @@ struct rtld_global_ro _rtld_global_ro attribute_relro = > ._dl_sysinfo = DL_SYSINFO_DEFAULT, > #endif > ._dl_debug_fd = STDERR_FILENO, > -#if !HAVE_TUNABLES > - ._dl_hwcap_mask = HWCAP_IMPORTANT, > -#endif > ._dl_lazy = 1, > ._dl_fpu_control = _FPU_DEFAULT, > ._dl_pagesize = EXEC_PAGESIZE, > @@ -1483,7 +1480,6 @@ dl_main (const ElfW(Phdr) *phdr, > _dl_argc -= 2; > _dl_argv += 2; > } > -#if HAVE_TUNABLES > else if (! strcmp (_dl_argv[1], "--list-tunables")) > { > state.mode = rtld_mode_list_tunables; > @@ -1491,7 +1487,6 @@ dl_main (const ElfW(Phdr) *phdr, > --_dl_argc; > ++_dl_argv; > } > -#endif > else if (! strcmp (_dl_argv[1], "--list-diagnostics")) > { > state.mode = rtld_mode_list_diagnostics; > @@ -1519,13 +1514,11 @@ dl_main (const ElfW(Phdr) *phdr, > else > break; > > -#if HAVE_TUNABLES > if (__glibc_unlikely (state.mode == rtld_mode_list_tunables)) > { > __tunables_print (); > _exit (0); > } > -#endif > > if (state.mode == rtld_mode_list_diagnostics) > _dl_print_diagnostics (_environ); > @@ -2624,15 +2617,6 @@ process_envvars (struct dl_main_state *state) > _dl_show_auxv (); > break; > > -#if !HAVE_TUNABLES > - case 10: > - /* Mask for the important hardware capabilities. */ > - if (!__libc_enable_secure > - && memcmp (envline, "HWCAP_MASK", 10) == 0) > - GLRO(dl_hwcap_mask) = _dl_strtoul (&envline[11], NULL); > - break; > -#endif > - > case 11: > /* Path where the binary is found. */ > if (!__libc_enable_secure > @@ -2695,12 +2679,7 @@ process_envvars (struct dl_main_state *state) > while (*nextp != '\0'); > > if (__access ("/etc/suid-debug", F_OK) != 0) > - { > -#if !HAVE_TUNABLES > - unsetenv ("MALLOC_CHECK_"); > -#endif > - GLRO(dl_debug_mask) = 0; > - } > + GLRO(dl_debug_mask) = 0; > > if (state->mode != rtld_mode_normal) > _exit (5); > diff --git a/elf/tst-env-setuid-tunables.c b/elf/tst-env-setuid-tunables.c > index 807b426012..7dfb0e073a 100644 > --- a/elf/tst-env-setuid-tunables.c > +++ b/elf/tst-env-setuid-tunables.c > @@ -19,8 +19,6 @@ > glibc.malloc.check and glibc.malloc.mmap_threshold but also retain > glibc.malloc.mmap_threshold in an unprivileged child. */ > > -/* This is compiled as part of the testsuite but needs to see > - HAVE_TUNABLES. */ > #define _LIBC 1 > #include "config.h" > #undef _LIBC > @@ -83,7 +81,6 @@ test_child (int off) > { > const char *val = getenv ("GLIBC_TUNABLES"); > > -#if HAVE_TUNABLES > if (val != NULL && strcmp (val, resultstrings[off]) == 0) > return 0; > > @@ -91,14 +88,6 @@ test_child (int off) > printf ("[%d] Unexpected GLIBC_TUNABLES VALUE %s\n", off, val); > > return 1; > -#else > - if (val != NULL) > - { > - printf ("[%d] GLIBC_TUNABLES not cleared\n", off); > - return 1; > - } > - return 0; > -#endif > } > > static int > diff --git a/gmon/gmon.c b/gmon/gmon.c > index 5e99a7351d..bc0e29438a 100644 > --- a/gmon/gmon.c > +++ b/gmon/gmon.c > @@ -46,10 +46,8 @@ > #include > #include > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE gmon > -# include > -#endif > +#define TUNABLE_NAMESPACE gmon > +#include > > #ifdef PIC > # include > @@ -130,7 +128,6 @@ __monstartup (u_long lowpc, u_long highpc) > struct gmonparam *p = &_gmonparam; > long int minarcs, maxarcs; > > -#if HAVE_TUNABLES > /* Read minarcs/maxarcs tunables. */ > minarcs = TUNABLE_GET (minarcs, int32_t, NULL); > maxarcs = TUNABLE_GET (maxarcs, int32_t, NULL); > @@ -139,11 +136,6 @@ __monstartup (u_long lowpc, u_long highpc) > ERR("monstartup: maxarcs < minarcs, setting maxarcs = minarcs\n"); > maxarcs = minarcs; > } > -#else > - /* No tunables, we use hardcoded defaults */ > - minarcs = MINARCS; > - maxarcs = MAXARCS; > -#endif > > /* > * If we are incorrectly called twice in a row (without an > diff --git a/malloc/Makefile b/malloc/Makefile > index dfb51d344c..2b7bd10d49 100644 > --- a/malloc/Makefile > +++ b/malloc/Makefile > @@ -61,9 +61,7 @@ tests-internal += \ > tst-dynarray-fail \ > tst-dynarray-at-fail \ > > -ifneq (no,$(have-tunables)) > tests += tst-malloc-usable-tunables tst-mxfast > -endif > > tests += $(tests-static) > test-srcs = tst-mtrace > diff --git a/malloc/arena.c b/malloc/arena.c > index ccd193953b..e98b779dbb 100644 > --- a/malloc/arena.c > +++ b/malloc/arena.c > @@ -18,9 +18,7 @@ > > #include > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE malloc > -#endif > +#define TUNABLE_NAMESPACE malloc > #include > > /* Compile-time constants. */ > @@ -47,21 +45,13 @@ > static inline size_t > heap_min_size (void) > { > -#if HAVE_TUNABLES > return mp_.hp_pagesize == 0 ? HEAP_MIN_SIZE : mp_.hp_pagesize; > -#else > - return HEAP_MIN_SIZE; > -#endif > } > > static inline size_t > heap_max_size (void) > { > -#if HAVE_TUNABLES > return mp_.hp_pagesize == 0 ? HEAP_MAX_SIZE : mp_.hp_pagesize * 4; > -#else > - return HEAP_MAX_SIZE; > -#endif > } > > /***************************************************************************/ > @@ -239,8 +229,7 @@ __malloc_fork_unlock_child (void) > __libc_lock_init (list_lock); > } > > -#if HAVE_TUNABLES > -# define TUNABLE_CALLBACK_FNDECL(__name, __type) \ > +#define TUNABLE_CALLBACK_FNDECL(__name, __type) \ > static inline int do_ ## __name (__type value); \ > static void \ > TUNABLE_CALLBACK (__name) (tunable_val_t *valp) \ > @@ -263,42 +252,6 @@ TUNABLE_CALLBACK_FNDECL (set_tcache_unsorted_limit, size_t) > #endif > TUNABLE_CALLBACK_FNDECL (set_mxfast, size_t) > TUNABLE_CALLBACK_FNDECL (set_hugetlb, size_t) > -#else > -/* Initialization routine. */ > -#include > -extern char **_environ; > - > -static char * > -next_env_entry (char ***position) > -{ > - char **current = *position; > - char *result = NULL; > - > - while (*current != NULL) > - { > - if (__builtin_expect ((*current)[0] == 'M', 0) > - && (*current)[1] == 'A' > - && (*current)[2] == 'L' > - && (*current)[3] == 'L' > - && (*current)[4] == 'O' > - && (*current)[5] == 'C' > - && (*current)[6] == '_') > - { > - result = &(*current)[7]; > - > - /* Save current position for next visit. */ > - *position = ++current; > - > - break; > - } > - > - ++current; > - } > - > - return result; > -} > -#endif > - > > #if USE_TCACHE > static void tcache_key_initialize (void); > @@ -343,7 +296,6 @@ ptmalloc_init (void) > > malloc_init_state (&main_arena); > > -#if HAVE_TUNABLES > TUNABLE_GET (top_pad, size_t, TUNABLE_CALLBACK (set_top_pad)); > TUNABLE_GET (perturb, int32_t, TUNABLE_CALLBACK (set_perturb_byte)); > TUNABLE_GET (mmap_threshold, size_t, TUNABLE_CALLBACK (set_mmap_threshold)); > @@ -363,70 +315,6 @@ ptmalloc_init (void) > /* Force mmap for main arena instead of sbrk, so hugepages are explicitly > used. */ > __always_fail_morecore = true; > -#else > - if (__glibc_likely (_environ != NULL)) > - { > - char **runp = _environ; > - char *envline; > - > - while (__builtin_expect ((envline = next_env_entry (&runp)) != NULL, > - 0)) > - { > - size_t len = strcspn (envline, "="); > - > - if (envline[len] != '=') > - /* This is a "MALLOC_" variable at the end of the string > - without a '=' character. Ignore it since otherwise we > - will access invalid memory below. */ > - continue; > - > - switch (len) > - { > - case 8: > - if (!__builtin_expect (__libc_enable_secure, 0)) > - { > - if (memcmp (envline, "TOP_PAD_", 8) == 0) > - __libc_mallopt (M_TOP_PAD, strtol (&envline[9], NULL, 10)); > - else if (memcmp (envline, "PERTURB_", 8) == 0) > - __libc_mallopt (M_PERTURB, strtol (&envline[9], NULL, 10)); > - } > - break; > - case 9: > - if (!__builtin_expect (__libc_enable_secure, 0)) > - { > - if (memcmp (envline, "MMAP_MAX_", 9) == 0) > - __libc_mallopt (M_MMAP_MAX, strtol (&envline[10], > - NULL, 10)); > - else if (memcmp (envline, "ARENA_MAX", 9) == 0) > - __libc_mallopt (M_ARENA_MAX, strtol (&envline[10], > - NULL, 10)); > - } > - break; > - case 10: > - if (!__builtin_expect (__libc_enable_secure, 0)) > - { > - if (memcmp (envline, "ARENA_TEST", 10) == 0) > - __libc_mallopt (M_ARENA_TEST, strtol (&envline[11], > - NULL, 10)); > - } > - break; > - case 15: > - if (!__builtin_expect (__libc_enable_secure, 0)) > - { > - if (memcmp (envline, "TRIM_THRESHOLD_", 15) == 0) > - __libc_mallopt (M_TRIM_THRESHOLD, strtol (&envline[16], > - NULL, 10)); > - else if (memcmp (envline, "MMAP_THRESHOLD_", 15) == 0) > - __libc_mallopt (M_MMAP_THRESHOLD, strtol (&envline[16], > - NULL, 10)); > - } > - break; > - default: > - break; > - } > - } > - } > -#endif > } > > /* Managing heaps and arenas (for concurrent threads) */ > @@ -561,7 +449,6 @@ alloc_new_heap (size_t size, size_t top_pad, size_t pagesize, > static heap_info * > new_heap (size_t size, size_t top_pad) > { > -#if HAVE_TUNABLES > if (__glibc_unlikely (mp_.hp_pagesize != 0)) > { > heap_info *h = alloc_new_heap (size, top_pad, mp_.hp_pagesize, > @@ -569,7 +456,6 @@ new_heap (size_t size, size_t top_pad) > if (h != NULL) > return h; > } > -#endif > return alloc_new_heap (size, top_pad, GLRO (dl_pagesize), 0); > } > > diff --git a/malloc/malloc-check.c b/malloc/malloc-check.c > index af27fd2c52..5732d81bf1 100644 > --- a/malloc/malloc-check.c > +++ b/malloc/malloc-check.c > @@ -376,7 +376,6 @@ memalign_check (size_t alignment, size_t bytes) > return mem2mem_check (tag_new_usable (mem), bytes); > } > > -#if HAVE_TUNABLES > static void > TUNABLE_CALLBACK (set_mallopt_check) (tunable_val_t *valp) > { > @@ -384,7 +383,6 @@ TUNABLE_CALLBACK (set_mallopt_check) (tunable_val_t *valp) > if (value != 0) > __malloc_debug_enable (MALLOC_CHECK_HOOK); > } > -#endif > > static bool > initialize_malloc_check (void) > @@ -392,12 +390,6 @@ initialize_malloc_check (void) > /* This is the copy of the malloc initializer that we pulled in along with > malloc-check. This does not affect any of the libc malloc structures. */ > ptmalloc_init (); > -#if HAVE_TUNABLES > TUNABLE_GET (check, int32_t, TUNABLE_CALLBACK (set_mallopt_check)); > -#else > - const char *s = secure_getenv ("MALLOC_CHECK_"); > - if (s && s[0] != '\0' && s[0] != '0') > - __malloc_debug_enable (MALLOC_CHECK_HOOK); > -#endif > return __is_malloc_debug_enabled (MALLOC_CHECK_HOOK); > } > diff --git a/malloc/malloc.c b/malloc/malloc.c > index 76c50e3f58..05e65a2d54 100644 > --- a/malloc/malloc.c > +++ b/malloc/malloc.c > @@ -1856,14 +1856,12 @@ struct malloc_par > INTERNAL_SIZE_T arena_test; > INTERNAL_SIZE_T arena_max; > > -#if HAVE_TUNABLES > /* Transparent Large Page support. */ > INTERNAL_SIZE_T thp_pagesize; > /* A value different than 0 means to align mmap allocation to hp_pagesize > add hp_flags on flags. */ > INTERNAL_SIZE_T hp_pagesize; > int hp_flags; > -#endif > > /* Memory map support */ > int n_mmaps; > @@ -1998,7 +1996,7 @@ free_perturb (char *p, size_t n) > static inline void > madvise_thp (void *p, INTERNAL_SIZE_T size) > { > -#if HAVE_TUNABLES && defined (MADV_HUGEPAGE) > +#ifdef MADV_HUGEPAGE > /* Do not consider areas smaller than a huge page or if the tunable is > not active. */ > if (mp_.thp_pagesize == 0 || size < mp_.thp_pagesize) > @@ -2557,7 +2555,6 @@ sysmalloc (INTERNAL_SIZE_T nb, mstate av) > && (mp_.n_mmaps < mp_.n_mmaps_max))) > { > char *mm; > -#if HAVE_TUNABLES > if (mp_.hp_pagesize > 0 && nb >= mp_.hp_pagesize) > { > /* There is no need to isse the THP madvise call if Huge Pages are > @@ -2566,7 +2563,6 @@ sysmalloc (INTERNAL_SIZE_T nb, mstate av) > if (mm != MAP_FAILED) > return mm; > } > -#endif > mm = sysmalloc_mmap (nb, pagesize, 0, av); > if (mm != MAP_FAILED) > return mm; > @@ -2679,7 +2675,7 @@ sysmalloc (INTERNAL_SIZE_T nb, mstate av) > previous calls. Otherwise, we correct to page-align below. > */ > > -#if HAVE_TUNABLES && defined (MADV_HUGEPAGE) > +#ifdef MADV_HUGEPAGE > /* Defined in brk.c. */ > extern void *__curbrk; > if (__glibc_unlikely (mp_.thp_pagesize != 0)) > @@ -2718,12 +2714,10 @@ sysmalloc (INTERNAL_SIZE_T nb, mstate av) > */ > > char *mbrk = MAP_FAILED; > -#if HAVE_TUNABLES > if (mp_.hp_pagesize > 0) > mbrk = sysmalloc_mmap_fallback (&size, nb, old_size, > mp_.hp_pagesize, mp_.hp_pagesize, > mp_.hp_flags, av); > -#endif > if (mbrk == MAP_FAILED) > mbrk = sysmalloc_mmap_fallback (&size, nb, old_size, MMAP_AS_MORECORE_SIZE, > pagesize, 0, av); > @@ -2966,7 +2960,7 @@ systrim (size_t pad, mstate av) > return 0; > > /* Release in pagesize units and round down to the nearest page. */ > -#if HAVE_TUNABLES && defined (MADV_HUGEPAGE) > +#ifdef MADV_HUGEPAGE > if (__glibc_unlikely (mp_.thp_pagesize != 0)) > extra = ALIGN_DOWN (top_area - pad, mp_.thp_pagesize); > else > @@ -5410,7 +5404,6 @@ do_set_mxfast (size_t value) > return 0; > } > > -#if HAVE_TUNABLES > static __always_inline int > do_set_hugetlb (size_t value) > { > @@ -5429,7 +5422,6 @@ do_set_hugetlb (size_t value) > &mp_.hp_flags); > return 0; > } > -#endif > > int > __libc_mallopt (int param_number, int value) > diff --git a/manual/install.texi b/manual/install.texi > index 260f8a5c82..37edf26195 100644 > --- a/manual/install.texi > +++ b/manual/install.texi > @@ -262,25 +262,6 @@ configure with @option{--disable-werror}. > By default for x86_64, @theglibc{} is built with the vector math library. > Use this option to disable the vector math library. > > -@item --enable-tunables > -Tunables support allows additional library parameters to be customized at > -runtime. This feature is enabled by default. This option can take the > -following values: > - > -@table @code > -@item yes > -This is the default if no option is passed to configure. This enables tunables > -and selects the default frontend (currently @samp{valstring}). > - > -@item no > -This option disables tunables. > - > -@item valstring > -This enables tunables and selects the @samp{valstring} frontend for tunables. > -This frontend allows users to specify tunables as a colon-separated list in a > -single environment variable @env{GLIBC_TUNABLES}. > -@end table > - > @item --disable-crypt > Do not install the passphrase-hashing library @file{libcrypt} or the > header file @file{crypt.h}. @file{unistd.h} will still declare the > diff --git a/nptl/pthread_mutex_conf.c b/nptl/pthread_mutex_conf.c > index 329c4cbb8f..9133384d47 100644 > --- a/nptl/pthread_mutex_conf.c > +++ b/nptl/pthread_mutex_conf.c > @@ -16,8 +16,7 @@ > License along with the GNU C Library; if not, see > . */ > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE pthread > +#define TUNABLE_NAMESPACE pthread > #include > #include > #include > @@ -53,4 +52,3 @@ __pthread_tunables_init (void) > TUNABLE_GET (stack_cache_size, size_t, > TUNABLE_CALLBACK (set_stack_cache_size)); > } > -#endif > diff --git a/nptl/tst-mutex8.c b/nptl/tst-mutex8.c > index 4ac5f5059f..e16284d29e 100644 > --- a/nptl/tst-mutex8.c > +++ b/nptl/tst-mutex8.c > @@ -98,7 +98,6 @@ check_type (const char *mas, pthread_mutexattr_t *ma) > /* Check if a mutex will be elided. Lock elision can only be activated via > the tunables framework. By default, lock elision is disabled. */ > bool assume_elided_mutex = false; > -#if HAVE_TUNABLES > int ma_type = PTHREAD_MUTEX_TIMED_NP; > if (ma != NULL) > { > @@ -117,7 +116,6 @@ check_type (const char *mas, pthread_mutexattr_t *ma) > if (TUNABLE_GET_FULL (glibc, elision, enable, int32_t, NULL) == 1) > assume_elided_mutex = true; > } > -#endif > > e = pthread_mutex_init (m, ma); > if (e != 0) > diff --git a/scripts/build-many-glibcs.py b/scripts/build-many-glibcs.py > index 29c10c2916..7d7a10bffa 100755 > --- a/scripts/build-many-glibcs.py > +++ b/scripts/build-many-glibcs.py > @@ -445,7 +445,6 @@ class Context(object): > '--disable-profile', > '--disable-timezone-tools', > '--disable-mathvec', > - '--disable-tunables', > '--disable-crypt', > '--disable-experimental-malloc', > '--disable-build-nscd', > diff --git a/sysdeps/generic/ldsodefs.h b/sysdeps/generic/ldsodefs.h > index c99dad77cc..bf16b66c9c 100644 > --- a/sysdeps/generic/ldsodefs.h > +++ b/sysdeps/generic/ldsodefs.h > @@ -586,11 +586,6 @@ struct rtld_global_ro > /* Mask for hardware capabilities that are available. */ > EXTERN uint64_t _dl_hwcap; > > -#if !HAVE_TUNABLES > - /* Mask for important hardware capabilities we honour. */ > - EXTERN uint64_t _dl_hwcap_mask; > -#endif > - > #ifdef HAVE_AUX_VECTOR > /* Pointer to the auxv list supplied to the program at startup. */ > EXTERN ElfW(auxv_t) *_dl_auxv; > @@ -1192,18 +1187,6 @@ extern struct link_map * _dl_get_dl_main_map (void) > brk. */ > void *_dl_early_allocate (size_t size) attribute_hidden; > > -/* Initialize the DSO sort algorithm to use. */ > -#if !HAVE_TUNABLES > -static inline void > -__always_inline > -_dl_sort_maps_init (void) > -{ > - /* This is optimized out if tunables are not enabled. */ > -} > -#else > -extern void _dl_sort_maps_init (void) attribute_hidden; > -#endif > - > /* Initialization of libpthread for statically linked applications. > If libpthread is not linked in, this is an empty function. */ > void __pthread_initialize_minimal (void) weak_function; > diff --git a/sysdeps/generic/unsecvars.h b/sysdeps/generic/unsecvars.h > index b50331b50f..81397fb90b 100644 > --- a/sysdeps/generic/unsecvars.h > +++ b/sysdeps/generic/unsecvars.h > @@ -1,16 +1,10 @@ > -#if !HAVE_TUNABLES > -# define GLIBC_TUNABLES_ENVVAR "GLIBC_TUNABLES\0" > -#else > -# define GLIBC_TUNABLES_ENVVAR > -#endif > - > /* Environment variable to be removed for SUID programs. The names are > all stuffed in a single string which means they have to be terminated > with a '\0' explicitly. */ > #define UNSECURE_ENVVARS \ > "GCONV_PATH\0" \ > "GETCONF_DIR\0" \ > - GLIBC_TUNABLES_ENVVAR \ > + "GLIBC_TUNABLES\0" \ > "HOSTALIASES\0" \ > "LD_AUDIT\0" \ > "LD_DEBUG\0" \ > diff --git a/sysdeps/nptl/dl-tls_init_tp.c b/sysdeps/nptl/dl-tls_init_tp.c > index 0db0349c4b..50dc778852 100644 > --- a/sysdeps/nptl/dl-tls_init_tp.c > +++ b/sysdeps/nptl/dl-tls_init_tp.c > @@ -102,9 +102,7 @@ __tls_init_tp (void) > > { > bool do_rseq = true; > -#if HAVE_TUNABLES > do_rseq = TUNABLE_GET (rseq, int, NULL); > -#endif > if (rseq_register_current_thread (pd, do_rseq)) > { > /* We need a writable view of the variables. They are in > diff --git a/sysdeps/nptl/pthreadP.h b/sysdeps/nptl/pthreadP.h > index 536b75b582..54f9198681 100644 > --- a/sysdeps/nptl/pthreadP.h > +++ b/sysdeps/nptl/pthreadP.h > @@ -45,11 +45,7 @@ > > static inline short max_adaptive_count (void) > { > -#if HAVE_TUNABLES > return __mutex_aconf.spin_count; > -#else > - return DEFAULT_ADAPTIVE_COUNT; > -#endif > } > > > diff --git a/sysdeps/nptl/pthread_early_init.h b/sysdeps/nptl/pthread_early_init.h > index 39255d928c..f1a3b65d1a 100644 > --- a/sysdeps/nptl/pthread_early_init.h > +++ b/sysdeps/nptl/pthread_early_init.h > @@ -53,9 +53,7 @@ __pthread_early_init (void) > __default_pthread_attr.internal.stacksize = limit.rlim_cur; > __default_pthread_attr.internal.guardsize = GLRO (dl_pagesize); > > -#if HAVE_TUNABLES > __pthread_tunables_init (); > -#endif > } > > #endif /* _PTHREAD_EARLY_INIT_H */ > diff --git a/sysdeps/nptl/pthread_mutex_conf.h b/sysdeps/nptl/pthread_mutex_conf.h > index 6d4b6ad880..8151bafce7 100644 > --- a/sysdeps/nptl/pthread_mutex_conf.h > +++ b/sysdeps/nptl/pthread_mutex_conf.h > @@ -20,7 +20,6 @@ > > #include > > -#if HAVE_TUNABLES > struct mutex_config > { > int spin_count; > @@ -30,12 +29,5 @@ extern struct mutex_config __mutex_aconf; > libc_hidden_proto (__mutex_aconf) > > extern void __pthread_tunables_init (void) attribute_hidden; > -#else > -static inline void > -__pthread_tunables_init (void) > -{ > - /* No tunables to initialize. */ > -} > -#endif > > #endif > diff --git a/sysdeps/powerpc/cpu-features.c b/sysdeps/powerpc/cpu-features.c > index 9ed0e5e8c1..0ef3cf89d2 100644 > --- a/sysdeps/powerpc/cpu-features.c > +++ b/sysdeps/powerpc/cpu-features.c > @@ -18,10 +18,7 @@ > > #include > #include > - > -#if HAVE_TUNABLES > -# include > -#endif > +#include > > static inline void > init_cpu_features (struct cpu_features *cpu_features) > @@ -29,11 +26,7 @@ init_cpu_features (struct cpu_features *cpu_features) > /* Default is to use aligned memory access on optimized function unless > tunables is enable, since for this case user can explicit disable > unaligned optimizations. */ > -#if HAVE_TUNABLES > int32_t cached_memfunc = TUNABLE_GET (glibc, cpu, cached_memopt, int32_t, > NULL); > cpu_features->use_cached_memopt = (cached_memfunc > 0); > -#else > - cpu_features->use_cached_memopt = false; > -#endif > } > diff --git a/sysdeps/s390/cpu-features.c b/sysdeps/s390/cpu-features.c > index afeb9b5638..39f8c23a60 100644 > --- a/sysdeps/s390/cpu-features.c > +++ b/sysdeps/s390/cpu-features.c > @@ -18,15 +18,12 @@ > > #include > > -#if HAVE_TUNABLES > -# include > -# include > -# include > +#include > +#include > +#include > extern __typeof (memcmp) MEMCMP_DEFAULT; > -#endif > > -#if HAVE_TUNABLES > -# define S390_COPY_CPU_FEATURES(SRC_PTR, DEST_PTR) \ > +#define S390_COPY_CPU_FEATURES(SRC_PTR, DEST_PTR) \ > (DEST_PTR)->hwcap = (SRC_PTR)->hwcap; \ > (DEST_PTR)->stfle_bits[0] = (SRC_PTR)->stfle_bits[0]; > > @@ -205,7 +202,6 @@ TUNABLE_CALLBACK (set_hwcaps) (tunable_val_t *valp) > cpu_features->stfle_bits[0] = cpu_features_curr.stfle_bits[0] > & cpu_features_orig.stfle_bits[0]; > } > -#endif > > static inline void > init_cpu_features (struct cpu_features *cpu_features) > @@ -233,7 +229,5 @@ init_cpu_features (struct cpu_features *cpu_features) > cpu_features->stfle_bits[0] = 0ULL; > } > > -#if HAVE_TUNABLES > TUNABLE_GET (glibc, cpu, hwcaps, tunable_val_t *, TUNABLE_CALLBACK (set_hwcaps)); > -#endif > } > diff --git a/sysdeps/sparc/sparc32/dl-machine.h b/sysdeps/sparc/sparc32/dl-machine.h > index 835e68925b..9b57ae1a93 100644 > --- a/sysdeps/sparc/sparc32/dl-machine.h > +++ b/sysdeps/sparc/sparc32/dl-machine.h > @@ -39,8 +39,9 @@ elf_machine_matches_host (const Elf32_Ehdr *ehdr) > return 1; > else if (ehdr->e_machine == EM_SPARC32PLUS) > { > -#if HAVE_TUNABLES || defined SHARED > - uint64_t hwcap_mask = GET_HWCAP_MASK(); > +#if defined SHARED > + uint64_t hwcap_mask = TUNABLE_GET (glibc, cpu, hwcap_mask, uint64_t, > + NULL); > return GLRO(dl_hwcap) & hwcap_mask & HWCAP_SPARC_V9; > #else > return GLRO(dl_hwcap) & HWCAP_SPARC_V9; > diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile > index f8bd12d991..b4323b2604 100644 > --- a/sysdeps/unix/sysv/linux/Makefile > +++ b/sysdeps/unix/sysv/linux/Makefile > @@ -245,11 +245,9 @@ tests-internal += \ > tst-sigcontext-get_pc \ > # tests-internal > > -ifneq (no,$(have-tunables)) > tests-internal += \ > tst-rseq-disable \ > - # tests-internal $(have-tunables) > -endif > + # tests-internal > > tests-time64 += \ > tst-adjtimex-time64 \ > diff --git a/sysdeps/unix/sysv/linux/aarch64/cpu-features.c b/sysdeps/unix/sysv/linux/aarch64/cpu-features.c > index 0380f116de..dc09c1c827 100644 > --- a/sysdeps/unix/sysv/linux/aarch64/cpu-features.c > +++ b/sysdeps/unix/sysv/linux/aarch64/cpu-features.c > @@ -30,7 +30,6 @@ > to see when pointer have been correctly tagged. */ > #define MTE_ALLOWED_TAGS (0xfffe << PR_MTE_TAG_SHIFT) > > -#if HAVE_TUNABLES > struct cpu_list > { > const char *name; > @@ -59,19 +58,16 @@ get_midr_from_mcpu (const char *mcpu) > > return UINT64_MAX; > } > -#endif > > static inline void > init_cpu_features (struct cpu_features *cpu_features) > { > register uint64_t midr = UINT64_MAX; > > -#if HAVE_TUNABLES > /* Get the tunable override. */ > const char *mcpu = TUNABLE_GET (glibc, cpu, name, const char *, NULL); > if (mcpu != NULL) > midr = get_midr_from_mcpu (mcpu); > -#endif > > /* If there was no useful tunable override, query the MIDR if the kernel > allows it. */ > @@ -100,13 +96,11 @@ init_cpu_features (struct cpu_features *cpu_features) > cpu_features->mte_state = 0; > > #ifdef USE_MTAG > -# if HAVE_TUNABLES > int mte_state = TUNABLE_GET (glibc, mem, tagging, unsigned, 0); > cpu_features->mte_state = (GLRO (dl_hwcap2) & HWCAP2_MTE) ? mte_state : 0; > /* If we lack the MTE feature, disable the tunable, since it will > otherwise cause instructions that won't run on this CPU to be used. */ > TUNABLE_SET (glibc, mem, tagging, cpu_features->mte_state); > -# endif > > if (cpu_features->mte_state & 4) > /* Enable choosing system-preferred faulting mode. */ > diff --git a/sysdeps/unix/sysv/linux/powerpc/elision-conf.c b/sysdeps/unix/sysv/linux/powerpc/elision-conf.c > index 8cd945a6c5..7c98dbd0d0 100644 > --- a/sysdeps/unix/sysv/linux/powerpc/elision-conf.c > +++ b/sysdeps/unix/sysv/linux/powerpc/elision-conf.c > @@ -22,9 +22,7 @@ > #include > #include > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE elision > -#endif > +#define TUNABLE_NAMESPACE elision > #include > > /* Reasonable initial tuning values, may be revised in the future. > @@ -51,7 +49,6 @@ struct elision_config __elision_aconf = > .skip_trylock_internal_abort = 3, > }; > > -#if HAVE_TUNABLES > static inline void > __always_inline > do_set_elision_enable (int32_t elision_enable) > @@ -94,14 +91,12 @@ TUNABLE_CALLBACK_FNDECL (skip_lock_internal_abort, int32_t); > TUNABLE_CALLBACK_FNDECL (skip_lock_out_of_tbegin_retries, int32_t); > TUNABLE_CALLBACK_FNDECL (try_tbegin, int32_t); > TUNABLE_CALLBACK_FNDECL (skip_trylock_internal_abort, int32_t); > -#endif > > /* Initialize elision. */ > > void > __lll_elision_init (void) > { > -#if HAVE_TUNABLES > /* Elision depends on tunables and must be explicitly turned on by setting > the appropriate tunable on a supported platform. */ > > @@ -117,7 +112,6 @@ __lll_elision_init (void) > TUNABLE_CALLBACK (set_elision_try_tbegin)); > TUNABLE_GET (skip_trylock_internal_abort, int32_t, > TUNABLE_CALLBACK (set_elision_skip_trylock_internal_abort)); > -#endif > > /* Linux from 3.9 through 4.2 do not abort HTM transaction on syscalls, > instead it suspends the transaction and resumes it when returning to > diff --git a/sysdeps/unix/sysv/linux/s390/elision-conf.c b/sysdeps/unix/sysv/linux/s390/elision-conf.c > index 5911e51097..6a6d6814ee 100644 > --- a/sysdeps/unix/sysv/linux/s390/elision-conf.c > +++ b/sysdeps/unix/sysv/linux/s390/elision-conf.c > @@ -22,9 +22,7 @@ > #include > #include > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE elision > -#endif > +#define TUNABLE_NAMESPACE elision > #include > > /* Reasonable initial tuning values, may be revised in the future. > @@ -51,7 +49,6 @@ struct elision_config __elision_aconf = > .skip_trylock_internal_abort = 3, > }; > > -#if HAVE_TUNABLES > static inline void > __always_inline > do_set_elision_enable (int32_t elision_enable) > @@ -93,14 +90,12 @@ TUNABLE_CALLBACK_FNDECL (skip_lock_internal_abort, int32_t); > TUNABLE_CALLBACK_FNDECL (skip_lock_out_of_tbegin_retries, int32_t); > TUNABLE_CALLBACK_FNDECL (try_tbegin, int32_t); > TUNABLE_CALLBACK_FNDECL (skip_trylock_internal_abort, int32_t); > -#endif > > /* Initialize elison. */ > > void > __lll_elision_init (void) > { > -#if HAVE_TUNABLES > /* Elision depends on tunables and must be explicitly turned on by setting > the appropriate tunable on a supported platform. */ > > @@ -116,7 +111,6 @@ __lll_elision_init (void) > TUNABLE_CALLBACK (set_elision_try_tbegin)); > TUNABLE_GET (skip_trylock_internal_abort, int32_t, > TUNABLE_CALLBACK (set_elision_skip_trylock_internal_abort)); > -#endif > > if (!__pthread_force_elision) > __elision_aconf.try_tbegin = 0; /* Disable elision on rwlocks. */ > diff --git a/sysdeps/unix/sysv/linux/x86/elision-conf.c b/sysdeps/unix/sysv/linux/x86/elision-conf.c > index 6447bda96b..cfd9a71e31 100644 > --- a/sysdeps/unix/sysv/linux/x86/elision-conf.c > +++ b/sysdeps/unix/sysv/linux/x86/elision-conf.c > @@ -22,9 +22,7 @@ > #include > #include > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE elision > -#endif > +#define TUNABLE_NAMESPACE elision > #include > > /* Reasonable initial tuning values, may be revised in the future. > @@ -48,7 +46,6 @@ struct elision_config __elision_aconf = > .skip_trylock_internal_abort = 3, > }; > > -#if HAVE_TUNABLES > static __always_inline void > do_set_elision_enable (int32_t elision_enable) > { > @@ -87,14 +84,12 @@ TUNABLE_CALLBACK_FNDECL (skip_lock_busy, int32_t); > TUNABLE_CALLBACK_FNDECL (skip_lock_internal_abort, int32_t); > TUNABLE_CALLBACK_FNDECL (retry_try_xbegin, int32_t); > TUNABLE_CALLBACK_FNDECL (skip_trylock_internal_abort, int32_t); > -#endif > > /* Initialize elision. */ > > void > __lll_elision_init (void) > { > -#if HAVE_TUNABLES > /* Elision depends on tunables and must be explicitly turned on by setting > the appropriate tunable on a supported platform. */ > > @@ -108,7 +103,6 @@ __lll_elision_init (void) > TUNABLE_CALLBACK (set_elision_retry_try_xbegin)); > TUNABLE_GET (skip_trylock_internal_abort, int32_t, > TUNABLE_CALLBACK (set_elision_skip_trylock_internal_abort)); > -#endif > > if (!__pthread_force_elision) > __elision_aconf.retry_try_xbegin = 0; /* Disable elision on rwlocks. */ > diff --git a/sysdeps/unix/sysv/linux/x86_64/64/Makefile b/sysdeps/unix/sysv/linux/x86_64/64/Makefile > index 8ff4f27786..1bf7d528b2 100644 > --- a/sysdeps/unix/sysv/linux/x86_64/64/Makefile > +++ b/sysdeps/unix/sysv/linux/x86_64/64/Makefile > @@ -2,7 +2,6 @@ > default-abi := 64 > > ifeq ($(subdir),elf) > -ifneq ($(have-tunables),no) > > tests-map-32bit = \ > tst-map-32bit-1a \ > @@ -24,4 +23,3 @@ tst-map-32bit-1b-ENV = GLIBC_TUNABLES=glibc.cpu.prefer_map_32bit_exec=1 > $(objpfx)tst-map-32bit-1b: $(objpfx)tst-map-32bit-mod.so > > endif > -endif > diff --git a/sysdeps/x86/Makefile b/sysdeps/x86/Makefile > index 56fd5fc805..917c26f116 100644 > --- a/sysdeps/x86/Makefile > +++ b/sysdeps/x86/Makefile > @@ -25,7 +25,6 @@ tests-static += \ > tst-ifunc-isa-1-static > test-xfail-tst-ifunc-isa-1 = $(with-lld) > test-xfail-tst-ifunc-isa-1-static = $(with-lld) > -ifneq ($(have-tunables),no) > tests += \ > tst-ifunc-isa-2 \ > tst-ifunc-isa-2-static > @@ -35,7 +34,6 @@ test-xfail-tst-ifunc-isa-2 = $(with-lld) > test-xfail-tst-ifunc-isa-2-static = $(with-lld) > endif > endif > -endif > ifeq (yes,$(enable-x86-isa-level)) > tests += tst-isa-level-1 > modules-names += tst-isa-level-mod-1-baseline \ > @@ -65,11 +63,9 @@ $(objpfx)tst-isa-level-1.out: $(objpfx)tst-isa-level-mod-1-baseline.so \ > $(objpfx)tst-isa-level-mod-1-v3.so \ > $(objpfx)tst-isa-level-mod-1-v4.so > endif > -ifneq ($(have-tunables),no) > tst-ifunc-isa-2-ENV = GLIBC_TUNABLES=glibc.cpu.hwcaps=-SSE4_2,-AVX,-AVX2,-AVX512F > tst-ifunc-isa-2-static-ENV = $(tst-ifunc-isa-2-ENV) > endif > -endif > > ifeq ($(subdir),math) > tests += tst-ldbl-nonnormal-printf > @@ -79,12 +75,10 @@ ifeq ($(subdir),setjmp) > gen-as-const-headers += jmp_buf-ssp.sym > sysdep_routines += __longjmp_cancel > ifneq ($(enable-cet),no) > -ifneq ($(have-tunables),no) > tests += tst-setjmp-cet > tst-setjmp-cet-ENV = GLIBC_TUNABLES=glibc.cpu.x86_ibt=on:glibc.cpu.x86_shstk=on > endif > endif > -endif > > ifeq ($(subdir),string) > sysdep_routines += cacheinfo > @@ -135,10 +129,8 @@ tests += tst-cet-legacy-1 tst-cet-legacy-1a tst-cet-legacy-2 \ > tst-cet-legacy-10 tst-cet-legacy-10-static > tests-static += tst-cet-legacy-9-static tst-cet-legacy-10-static > tst-cet-legacy-1a-ARGS = -- $(host-test-program-cmd) > -ifneq (no,$(have-tunables)) > tests += tst-cet-legacy-4a tst-cet-legacy-4b tst-cet-legacy-4c \ > tst-cet-legacy-5b tst-cet-legacy-6b > -endif > modules-names += tst-cet-legacy-mod-1 tst-cet-legacy-mod-2 \ > tst-cet-legacy-mod-4 tst-cet-legacy-mod-5a \ > tst-cet-legacy-mod-5b tst-cet-legacy-mod-5c \ > @@ -195,7 +187,6 @@ $(objpfx)tst-cet-legacy-6a.out: $(objpfx)tst-cet-legacy-mod-6a.so \ > $(objpfx)tst-cet-legacy-mod-6a.so: $(objpfx)tst-cet-legacy-mod-6c.so > $(objpfx)tst-cet-legacy-mod-6b.so: $(objpfx)tst-cet-legacy-mod-6c.so > LDFLAGS-tst-cet-legacy-mod-6c.so = -Wl,--enable-new-dtags,-z,nodelete > -ifneq (no,$(have-tunables)) > $(objpfx)tst-cet-legacy-4a.out: $(objpfx)tst-cet-legacy-mod-4.so > tst-cet-legacy-4a-ENV = GLIBC_TUNABLES=glibc.cpu.x86_shstk=permissive > $(objpfx)tst-cet-legacy-4b.out: $(objpfx)tst-cet-legacy-mod-4.so > @@ -211,7 +202,6 @@ tst-cet-legacy-6b-ENV = GLIBC_TUNABLES=glibc.cpu.hwcaps=-IBT,-SHSTK > tst-cet-legacy-9-ENV = GLIBC_TUNABLES=glibc.cpu.hwcaps=-IBT,-SHSTK > tst-cet-legacy-9-static-ENV = GLIBC_TUNABLES=glibc.cpu.hwcaps=-IBT,-SHSTK > endif > -endif > > # Add -fcf-protection to CFLAGS when CET is enabled. > CFLAGS-.o += -fcf-protection > diff --git a/sysdeps/x86/cacheinfo.h b/sysdeps/x86/cacheinfo.h > index 7a99e04b73..ec1bc142c4 100644 > --- a/sysdeps/x86/cacheinfo.h > +++ b/sysdeps/x86/cacheinfo.h > @@ -21,11 +21,9 @@ > #include > #include > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE cpu > -# include /* Get STDOUT_FILENO for _dl_printf. */ > -# include > -#endif > +#define TUNABLE_NAMESPACE cpu > +#include /* Get STDOUT_FILENO for _dl_printf. */ > +#include > > #if IS_IN (libc) > /* Data cache size for use in memory and string routines, typically > diff --git a/sysdeps/x86/cpu-features.c b/sysdeps/x86/cpu-features.c > index 822688e21f..978eb29f72 100644 > --- a/sysdeps/x86/cpu-features.c > +++ b/sysdeps/x86/cpu-features.c > @@ -23,11 +23,10 @@ > #include > #include > > -#if HAVE_TUNABLES > extern void TUNABLE_CALLBACK (set_hwcaps) (tunable_val_t *) > attribute_hidden; > > -# ifdef __LP64__ > +#ifdef __LP64__ > static void > TUNABLE_CALLBACK (set_prefer_map_32bit_exec) (tunable_val_t *valp) > { > @@ -35,17 +34,14 @@ TUNABLE_CALLBACK (set_prefer_map_32bit_exec) (tunable_val_t *valp) > GLRO(dl_x86_cpu_features).preferred[index_arch_Prefer_MAP_32BIT_EXEC] > |= bit_arch_Prefer_MAP_32BIT_EXEC; > } > -# endif > +#endif > > -# if CET_ENABLED > +#if CET_ENABLED > extern void TUNABLE_CALLBACK (set_x86_ibt) (tunable_val_t *) > attribute_hidden; > extern void TUNABLE_CALLBACK (set_x86_shstk) (tunable_val_t *) > attribute_hidden; > -# endif > -#endif > > -#if CET_ENABLED > # include > #endif > > @@ -712,13 +708,12 @@ no_cpuid: > > dl_init_cacheinfo (cpu_features); > > -#if HAVE_TUNABLES > TUNABLE_GET (hwcaps, tunable_val_t *, TUNABLE_CALLBACK (set_hwcaps)); > > -# ifdef __LP64__ > +#ifdef __LP64__ > TUNABLE_GET (prefer_map_32bit_exec, tunable_val_t *, > TUNABLE_CALLBACK (set_prefer_map_32bit_exec)); > -# endif > +#endif > > bool disable_xsave_features = false; > > @@ -774,13 +769,6 @@ no_cpuid: > CPU_FEATURE_UNSET (cpu_features, FMA4); > } > > -#elif defined SHARED > - /* Reuse dl_platform, dl_hwcap and dl_hwcap_mask for x86. The > - glibc.cpu.hwcap_mask tunable is initialized already, so no > - need to do this. */ > - GLRO(dl_hwcap_mask) = HWCAP_IMPORTANT; > -#endif > - > #ifdef __x86_64__ > GLRO(dl_hwcap) = HWCAP_X86_64; > if (cpu_features->basic.kind == arch_kind_intel) > @@ -828,12 +816,10 @@ no_cpuid: > #endif > > #if CET_ENABLED > -# if HAVE_TUNABLES > TUNABLE_GET (x86_ibt, tunable_val_t *, > TUNABLE_CALLBACK (set_x86_ibt)); > TUNABLE_GET (x86_shstk, tunable_val_t *, > TUNABLE_CALLBACK (set_x86_shstk)); > -# endif > > /* Check CET status. */ > unsigned int cet_status = get_cet_status (); > diff --git a/sysdeps/x86/cpu-tunables.c b/sysdeps/x86/cpu-tunables.c > index 772fb0c4c2..0d4f328585 100644 > --- a/sysdeps/x86/cpu-tunables.c > +++ b/sysdeps/x86/cpu-tunables.c > @@ -16,31 +16,30 @@ > License along with the GNU C Library; if not, see > . */ > > -#if HAVE_TUNABLES > -# define TUNABLE_NAMESPACE cpu > -# include > -# include > -# include /* Get STDOUT_FILENO for _dl_printf. */ > -# include > -# include > -# include > -# include > +#define TUNABLE_NAMESPACE cpu > +#include > +#include > +#include /* Get STDOUT_FILENO for _dl_printf. */ > +#include > +#include > +#include > +#include > > /* We can't use IFUNC memcmp nor strlen in init_cpu_features from libc.a > since IFUNC must be set up by init_cpu_features. */ > -# if defined USE_MULTIARCH && !defined SHARED > -# ifdef __x86_64__ > +#if defined USE_MULTIARCH && !defined SHARED > +# ifdef __x86_64__ > /* DEFAULT_MEMCMP by sysdeps/x86_64/memcmp-isa-default-impl.h. */ > -# include > -# else > -# define DEFAULT_MEMCMP __memcmp_ia32 > -# endif > -extern __typeof (memcmp) DEFAULT_MEMCMP; > +# include > # else > -# define DEFAULT_MEMCMP memcmp > +# define DEFAULT_MEMCMP __memcmp_ia32 > # endif > +extern __typeof (memcmp) DEFAULT_MEMCMP; > +#else > +# define DEFAULT_MEMCMP memcmp > +#endif > > -# define CHECK_GLIBC_IFUNC_CPU_OFF(f, cpu_features, name, len) \ > +#define CHECK_GLIBC_IFUNC_CPU_OFF(f, cpu_features, name, len) \ > _Static_assert (sizeof (#name) - 1 == len, #name " != " #len); \ > if (!DEFAULT_MEMCMP (f, #name, len)) \ > { \ > @@ -50,7 +49,7 @@ extern __typeof (memcmp) DEFAULT_MEMCMP; > > /* Disable a preferred feature NAME. We don't enable a preferred feature > which isn't available. */ > -# define CHECK_GLIBC_IFUNC_PREFERRED_OFF(f, cpu_features, name, len) \ > +#define CHECK_GLIBC_IFUNC_PREFERRED_OFF(f, cpu_features, name, len) \ > _Static_assert (sizeof (#name) - 1 == len, #name " != " #len); \ > if (!DEFAULT_MEMCMP (f, #name, len)) \ > { \ > @@ -60,7 +59,7 @@ extern __typeof (memcmp) DEFAULT_MEMCMP; > } > > /* Enable/disable a preferred feature NAME. */ > -# define CHECK_GLIBC_IFUNC_PREFERRED_BOTH(f, cpu_features, name, \ > +#define CHECK_GLIBC_IFUNC_PREFERRED_BOTH(f, cpu_features, name, \ > disable, len) \ > _Static_assert (sizeof (#name) - 1 == len, #name " != " #len); \ > if (!DEFAULT_MEMCMP (f, #name, len)) \ > @@ -74,7 +73,7 @@ extern __typeof (memcmp) DEFAULT_MEMCMP; > > /* Enable/disable a preferred feature NAME. Enable a preferred feature > only if the feature NEED is usable. */ > -# define CHECK_GLIBC_IFUNC_PREFERRED_NEED_BOTH(f, cpu_features, name, \ > +#define CHECK_GLIBC_IFUNC_PREFERRED_NEED_BOTH(f, cpu_features, name, \ > need, disable, len) \ > _Static_assert (sizeof (#name) - 1 == len, #name " != " #len); \ > if (!DEFAULT_MEMCMP (f, #name, len)) \ > @@ -286,8 +285,7 @@ TUNABLE_CALLBACK (set_hwcaps) (tunable_val_t *valp) > while (*c != '\0'); > } > > -# if CET_ENABLED > - > +#if CET_ENABLED > attribute_hidden > void > TUNABLE_CALLBACK (set_x86_ibt) (tunable_val_t *valp) > @@ -313,5 +311,4 @@ TUNABLE_CALLBACK (set_x86_shstk) (tunable_val_t *valp) > sizeof ("permissive")) == 0) > GL(dl_x86_feature_control).shstk = cet_permissive; > } > -# endif > #endif > diff --git a/sysdeps/x86/dl-cacheinfo.h b/sysdeps/x86/dl-cacheinfo.h > index a0474af8d1..ab671487da 100644 > --- a/sysdeps/x86/dl-cacheinfo.h > +++ b/sysdeps/x86/dl-cacheinfo.h > @@ -760,10 +760,8 @@ dl_init_cacheinfo (struct cpu_features *cpu_features) > else if (non_temporal_threshold > maximum_non_temporal_threshold) > non_temporal_threshold = maximum_non_temporal_threshold; > > -#if HAVE_TUNABLES > /* NB: The REP MOVSB threshold must be greater than VEC_SIZE * 8. */ > unsigned int minimum_rep_movsb_threshold; > -#endif > /* NB: The default REP MOVSB threshold is 4096 * (VEC_SIZE / 16) for > VEC_SIZE == 64 or 32. For VEC_SIZE == 16, the default REP MOVSB > threshold is 2048 * (VEC_SIZE / 16). */ > @@ -772,24 +770,18 @@ dl_init_cacheinfo (struct cpu_features *cpu_features) > && !CPU_FEATURE_PREFERRED_P (cpu_features, Prefer_No_AVX512)) > { > rep_movsb_threshold = 4096 * (64 / 16); > -#if HAVE_TUNABLES > minimum_rep_movsb_threshold = 64 * 8; > -#endif > } > else if (CPU_FEATURE_PREFERRED_P (cpu_features, > AVX_Fast_Unaligned_Load)) > { > rep_movsb_threshold = 4096 * (32 / 16); > -#if HAVE_TUNABLES > minimum_rep_movsb_threshold = 32 * 8; > -#endif > } > else > { > rep_movsb_threshold = 2048 * (16 / 16); > -#if HAVE_TUNABLES > minimum_rep_movsb_threshold = 16 * 8; > -#endif > } > /* NB: The default REP MOVSB threshold is 2112 on processors with fast > short REP MOVSB (FSRM). */ > @@ -799,7 +791,6 @@ dl_init_cacheinfo (struct cpu_features *cpu_features) > /* The default threshold to use Enhanced REP STOSB. */ > unsigned long int rep_stosb_threshold = 2048; > > -#if HAVE_TUNABLES > long int tunable_size; > > tunable_size = TUNABLE_GET (x86_data_cache_size, long int, NULL); > @@ -836,7 +827,6 @@ dl_init_cacheinfo (struct cpu_features *cpu_features) > minimum_rep_movsb_threshold, SIZE_MAX); > TUNABLE_SET_WITH_BOUNDS (x86_rep_stosb_threshold, rep_stosb_threshold, 1, > SIZE_MAX); > -#endif > > unsigned long int rep_movsb_stop_threshold; > /* ERMS feature is implemented from AMD Zen3 architecture and it is > diff --git a/sysdeps/x86_64/Makefile b/sysdeps/x86_64/Makefile > index 688eb2d7c4..54396c3c3f 100644 > --- a/sysdeps/x86_64/Makefile > +++ b/sysdeps/x86_64/Makefile > @@ -57,7 +57,6 @@ $(objpfx)tst-quad2pie: $(objpfx)tst-quadmod2pie.o > CFLAGS-tst-quad1pie.c = $(PIE-ccflag) > CFLAGS-tst-quad2pie.c = $(PIE-ccflag) > > -ifneq (no,$(have-tunables)) > tests += tst-platform-1 > modules-names += tst-platformmod-1 x86_64/tst-platformmod-2 > extra-test-objs += tst-platformmod-2.o > @@ -71,7 +70,6 @@ $(objpfx)tst-platform-1.out: $(objpfx)x86_64/tst-platformmod-2.so > # always set to x86_64. > tst-platform-1-ENV = LD_PRELOAD=$(objpfx)\$$PLATFORM/tst-platformmod-2.so \ > GLIBC_TUNABLES=glibc.cpu.hwcaps=-AVX512F,-AVX2 > -endif > > tests += tst-audit3 tst-audit4 tst-audit5 tst-audit6 tst-audit7 \ > tst-audit10 tst-sse tst-avx tst-avx512