linux-kernel - Re: [PATCH] arch/cris/arch-v10/vmlinux.lds.S fix boot problem

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20071221151404.GM10888@axis.com>
Date:	Fri, 21 Dec 2007 16:14:04 +0100
From:	Jesper Nilsson <jesper.nilsson@...s.com>
To:	Yuusei KUWANA <kuwana@...nyan.to>
Cc:	Linux Kernel list <linux-kernel@...r.kernel.org>,
	mikael.starvik@...s.com
Subject: Re: [PATCH] arch/cris/arch-v10/vmlinux.lds.S fix boot problem


On Sat, Dec 15, 2007 at 02:59:33PM +0900, Yuusei KUWANA wrote:
> arch/cris/arch-v10/vmlinux.lds.S
>  fix boot problem
> * too old initcall style. replace INITCALLS macro
> * __init_begin, __init_end move for free_initmem()

Hi,

The conversion to INITCALLS is ok (I have the same change here),
but I'm curious what problem you are solving with moving __init_begin and
__init_end to only include the .init.data.

If there is use of initcalls/data outside init then that is a bug and
should be fixed, not worked around.

There might however be another problem that Mikael Starvik told me about,
the padding between initdata and the next section had gone AWOL,
which made the free_initmem() destroy useful data.

With the attached patch (and a lot of other patches) I am successful in
booting an Axis Devboard82+.
Could you please try the included patch and see if that solves your problem?

> Note: with this patch kernel boot and mount root,
> but after init done,  kernel panic at do_signal() ...

I have a patch in the pipe that should fix this, will send it in just
a small while.

> ryu

Thank you for your help!

Best regards,

/Jesper

> Signed-off-by: Yuusei KUWANA <ryu@...nyan.to>
> ---
>  arch/cris/arch-v10/vmlinux.lds.S |   13 ++++---------
>  1 files changed, 4 insertions(+), 9 deletions(-)
> 
> diff --git a/arch/cris/arch-v10/vmlinux.lds.S b/arch/cris/arch-v10/vmlinux.lds.S
> index 9859d49..c5bab0e 100644
> --- a/arch/cris/arch-v10/vmlinux.lds.S
> +++ b/arch/cris/arch-v10/vmlinux.lds.S
> @@ -53,26 +53,22 @@ SECTIONS
>    	.data.init_task : { *(.data.init_task) }
>  
>    	. = ALIGN(8192);              /* Init code and data */
> -  	__init_begin = .;
>  	.init.text : { 
>  		   _sinittext = .;
>  		   *(.init.text)
>  		   _einittext = .;
>  	}
> +  	__init_begin = .;
>    	.init.data : { *(.init.data) }
> +  	__init_end = .;
> +
>    	. = ALIGN(16);
>    	__setup_start = .;
>    	.init.setup : { *(.init.setup) }
>    	__setup_end = .;
>    	.initcall.init : {
>  		__initcall_start = .;
> -		*(.initcall1.init);
> -		*(.initcall2.init);
> -		*(.initcall3.init);
> -		*(.initcall4.init);
> -		*(.initcall5.init);
> -		*(.initcall6.init);
> -		*(.initcall7.init);
> +			INITCALLS
>  		__initcall_end = .;	
>  	}
>  
> @@ -97,7 +93,6 @@ SECTIONS
>  #endif
>  	
>  	__vmlinux_end = .;            /* last address of the physical file */
> -  	__init_end = .;
>  
>  	__data_end = . ;              /* Move to _edata ? */
>  	__bss_start = .;              /* BSS */
> -- 
> 1.5.3

diff --git a/arch/cris/arch-v10/vmlinux.lds.S b/arch/cris/arch-v10/vmlinux.lds.S
index 9859d49..30fa872 100644
--- a/arch/cris/arch-v10/vmlinux.lds.S
+++ b/arch/cris/arch-v10/vmlinux.lds.S
@@ -23,7 +23,7 @@ SECTIONS
 	_stext = .;
 	__stext = .;
 	.text : {
-		*(.text)
+		TEXT_TEXT
 		SCHED_TEXT
 		LOCK_TEXT
 		*(.fixup)
@@ -66,13 +66,7 @@ SECTIONS
   	__setup_end = .;
   	.initcall.init : {
 		__initcall_start = .;
-		*(.initcall1.init);
-		*(.initcall2.init);
-		*(.initcall3.init);
-		*(.initcall4.init);
-		*(.initcall5.init);
-		*(.initcall6.init);
-		*(.initcall7.init);
+		INIT_CALLS
 		__initcall_end = .;	
 	}
 
@@ -88,11 +82,15 @@ SECTIONS
 		__initramfs_start = .;
 		*(.init.ramfs)
 		__initramfs_end = .;
-		/* We fill to the next page, so we can discard all init
-		   pages without needing to consider what payload might be
-		   appended to the kernel image.  */
-		FILL (0); 
-		. = ALIGN (8192);
+		/*
+		 * We fill to the next page, so we can discard all init
+		 * pages without needing to consider what payload might be
+		 * appended to the kernel image.
+		 */
+		.init.fill : {
+			FILL (0);
+			. = ALIGN (8192);
+		}
 	}
 #endif
 	

/^JN - Jesper Nilsson
--
               Jesper Nilsson -- jesper.nilsson@...s.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/