All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch 20/51] mm: fix RODATA_TEST failure "rodata_test: test data was not read only"
@ 2017-10-03 23:15 akpm
  0 siblings, 0 replies; only message in thread
From: akpm @ 2017-10-03 23:15 UTC (permalink / raw)
  To: torvalds, mm-commits, akpm, christophe.leroy, David.Laight,
	jinb.park7, keescook, segher, stable

From: Christophe Leroy <christophe.leroy@c-s.fr>
Subject: mm: fix RODATA_TEST failure "rodata_test: test data was not read only"

On powerpc, RODATA_TEST fails with message the following messages:

[    6.199505] Freeing unused kernel memory: 528K
[    6.203935] rodata_test: test data was not read only

This is because GCC allocates it to .data section:

c0695034 g     O .data	00000004 rodata_test_data

Since 056b9d8a76924 ("mm: remove rodata_test_data export, add pr_fmt"),
rodata_test_data is used only inside rodata_test.c By declaring it static,
it gets properly allocated into .rodata section instead of .data:

c04df710 l     O .rodata	00000004 rodata_test_data

Fixes: 056b9d8a76924 ("mm: remove rodata_test_data export, add pr_fmt")
Link: http://lkml.kernel.org/r/20170921093729.1080368AC1@po15668-vm-win7.idsi0.si.c-s.fr
Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
Cc: Kees Cook <keescook@chromium.org>
Cc: Jinbum Park <jinb.park7@gmail.com>
Cc: Segher Boessenkool <segher@kernel.crashing.org>
Cc: David Laight <David.Laight@ACULAB.COM>
Cc: <stable@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 mm/rodata_test.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -puN mm/rodata_test.c~mm-fix-rodata_test-failure-rodata_test-test-data-was-not-read-only mm/rodata_test.c
--- a/mm/rodata_test.c~mm-fix-rodata_test-failure-rodata_test-test-data-was-not-read-only
+++ a/mm/rodata_test.c
@@ -14,7 +14,7 @@
 #include <linux/uaccess.h>
 #include <asm/sections.h>
 
-const int rodata_test_data = 0xC3;
+static const int rodata_test_data = 0xC3;
 
 void rodata_test(void)
 {
_

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-10-03 23:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-03 23:15 [patch 20/51] mm: fix RODATA_TEST failure "rodata_test: test data was not read only" akpm

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.