[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH] xen/init: Annotate all command line parameter infrastructure as const
There is no reason for any of it to be modified. Additionally, link .init.setup beside the other constant .init data. While editing this area, correct the types used in the extern declarations for __setup_start and __setup_end to match the types the linker actually produces. No functional change. Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> --- CC: Jan Beulich <JBeulich@xxxxxxxx> CC: Stefano Stabellini <stefano.stabellini@xxxxxxxxxx> CC: Julien Grall <julien.grall@xxxxxxx> --- xen/arch/arm/xen.lds.S | 11 ++++++----- xen/arch/x86/xen.lds.S | 11 ++++++----- xen/common/kernel.c | 2 +- xen/include/xen/init.h | 7 ++++--- 4 files changed, 17 insertions(+), 14 deletions(-) diff --git a/xen/arch/arm/xen.lds.S b/xen/arch/arm/xen.lds.S index 76982b2..0ad2ad9 100644 --- a/xen/arch/arm/xen.lds.S +++ b/xen/arch/arm/xen.lds.S @@ -135,6 +135,12 @@ SECTIONS *(.init.rodata) *(.init.rodata.rel) *(.init.rodata.str*) + + . = ALIGN(POINTER_ALIGN); + __setup_start = .; + *(.init.setup) + __setup_end = .; + *(.init.data) *(.init.data.rel) *(.init.data.rel.*) @@ -145,11 +151,6 @@ SECTIONS __ctors_end = .; } :text . = ALIGN(32); - .init.setup : { - __setup_start = .; - *(.init.setup) - __setup_end = .; - } :text .init.proc.info : { __proc_info_start = .; *(.init.proc.info) diff --git a/xen/arch/x86/xen.lds.S b/xen/arch/x86/xen.lds.S index a43b29d..e506714 100644 --- a/xen/arch/x86/xen.lds.S +++ b/xen/arch/x86/xen.lds.S @@ -152,6 +152,12 @@ SECTIONS *(.init.rodata) *(.init.rodata.rel) *(.init.rodata.str*) + + . = ALIGN(POINTER_ALIGN); + __setup_start = .; + *(.init.setup) + __setup_end = .; + *(.init.data) *(.init.data.rel) *(.init.data.rel.*) @@ -178,11 +184,6 @@ SECTIONS __ctors_end = .; } :text . = ALIGN(32); - .init.setup : { - __setup_start = .; - *(.init.setup) - __setup_end = .; - } :text .initcall.init : { __initcall_start = .; *(.initcallpresmp.init) diff --git a/xen/common/kernel.c b/xen/common/kernel.c index 1a6823a..6f785bb 100644 --- a/xen/common/kernel.c +++ b/xen/common/kernel.c @@ -96,7 +96,7 @@ void __init cmdline_parse(const char *cmdline) if ( !bool_assert ) optkey += 3; - for ( param = &__setup_start; param < &__setup_end; param++ ) + for ( param = __setup_start; param < __setup_end; param++ ) { if ( strcmp(param->name, optkey) ) { diff --git a/xen/include/xen/init.h b/xen/include/xen/init.h index 671ac81..9d7a080 100644 --- a/xen/include/xen/init.h +++ b/xen/include/xen/init.h @@ -86,10 +86,11 @@ struct kernel_param { void *var; }; -extern struct kernel_param __setup_start, __setup_end; +extern struct kernel_param __setup_start[], __setup_end[]; -#define __setup_str static __initdata __attribute__((__aligned__(1))) char -#define __kparam static __initsetup \ +#define __setup_str static const __initconstrel \ + __attribute__((__aligned__(1))) char +#define __kparam static const __initsetup \ __attribute__((__aligned__(sizeof(void *)))) struct kernel_param #define custom_param(_name, _var) \ -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |