linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Include __param section in read-only data range
@ 2006-07-01 21:59 Marcelo Tosatti
  2006-07-01 22:05 ` Arjan van de Ven
  0 siblings, 1 reply; 2+ messages in thread
From: Marcelo Tosatti @ 2006-07-01 21:59 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: linux-kernel


Hi,

The param section is an array of "kernel_param" structures, storing only
constant data: pointer to name, permission of the variable pointed to by
(void *)arg and pointers to set/get methods.

Move end_rodata down to include __param section in the read-only range
used by CONFIG_DEBUG_RODATA.

Signed-off-by: Marcelo Tosatti <marcelo@kvack.org>

diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
index 9d11550..175b632 100644
--- a/include/asm-generic/vmlinux.lds.h
+++ b/include/asm-generic/vmlinux.lds.h
@@ -90,15 +90,15 @@ #define RODATA								\
         __ksymtab_strings : AT(ADDR(__ksymtab_strings) - LOAD_OFFSET) {	\
 		*(__ksymtab_strings)					\
 	}								\
-	__end_rodata = .;						\
-	. = ALIGN(4096);						\
 									\
 	/* Built-in module parameters. */				\
 	__param : AT(ADDR(__param) - LOAD_OFFSET) {			\
 		VMLINUX_SYMBOL(__start___param) = .;			\
 		*(__param)						\
 		VMLINUX_SYMBOL(__stop___param) = .;			\
-	}
+	}								\
+	__end_rodata = .;						\
+	. = ALIGN(4096);						
 
 #define SECURITY_INIT							\
 	.security_initcall.init : AT(ADDR(.security_initcall.init) - LOAD_OFFSET) { \

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] Include __param section in read-only data range
  2006-07-01 21:59 [PATCH] Include __param section in read-only data range Marcelo Tosatti
@ 2006-07-01 22:05 ` Arjan van de Ven
  0 siblings, 0 replies; 2+ messages in thread
From: Arjan van de Ven @ 2006-07-01 22:05 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: linux-kernel

On Sat, 2006-07-01 at 18:59 -0300, Marcelo Tosatti wrote:
> Hi,
> 
> The param section is an array of "kernel_param" structures, storing only
> constant data: pointer to name, permission of the variable pointed to by
> (void *)arg and pointers to set/get methods.
> 
> Move end_rodata down to include __param section in the read-only range
> used by CONFIG_DEBUG_RODATA.
> 
> Signed-off-by: Marcelo Tosatti <marcelo@kvack.org>

Acked-by: Arjan van de Ven <arjan@linux.intel.com>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-07-01 22:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-07-01 21:59 [PATCH] Include __param section in read-only data range Marcelo Tosatti
2006-07-01 22:05 ` Arjan van de Ven

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).