From 17284d650ebe5c736c9730ee16401008f26128c3 Mon Sep 17 00:00:00 2001 From: Siddhesh Poyarekar Date: Thu, 11 May 2017 20:13:49 +0530 Subject: [PATCH] tunables: Make tunable_list relro As of now we don't need tunables to be writable after they have been set initially, so there is no point in having them writable. Put tunable_list in .data.rel.ro so that it is set as read-only after relocation. This also allows us to move some of the dl_* variables that are tunables controlled into the tunables infrastructure instead of having two copies. In future if we ever need specific tunables to be writable at runtime, we can split the tunable_list into two. Regression tested on x86_64 to verify that tests continue to pass. * scripts/gen-tunables.awk: Add attribute_relro to tunable_list. --- ChangeLog | 5 +++++ scripts/gen-tunables.awk | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index d886161b20..f92e871198 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2017-05-11 Siddhesh Poyarekar + + * scripts/gen-tunables.awk: Add attribute_relro to + tunable_list. + 2017-05-11 Joseph Myers [BZ #21457] diff --git a/scripts/gen-tunables.awk b/scripts/gen-tunables.awk index 601240ac0d..defb3e7354 100644 --- a/scripts/gen-tunables.awk +++ b/scripts/gen-tunables.awk @@ -141,7 +141,7 @@ END { # Finally, the tunable list. print "\n#ifdef TUNABLES_INTERNAL" - print "static tunable_t tunable_list[] = {" + print "static tunable_t tunable_list[] attribute_relro = {" for (t in types) { for (n in types[t]) { for (m in types[t][n]) {