Re: [PATCH] make all archs use early_param

[Heiko Carstens <heiko.carstens@de.ibm.com>]

On Fri, Aug 11, 2006 at 05:18:47PM +1000, Rusty Russell wrote:
> Gold star to PowerPC and s390 for calling parse_early_param(), *and*
> using it instead of open-coded early cmdline hacking.

Thanks! :)

> diff -urpN --exclude TAGS -X /home/rusty/devel/kernel/kernel-patches/dontdiff --minimal linux-2.6.18-rc3-mm2/arch/s390/kernel/setup.c working-2.6.18-rc3-mm2-early_param-for-all/arch/s390/kernel/setup.c
> --- linux-2.6.18-rc3-mm2/arch/s390/kernel/setup.c	2006-08-03 12:50:21.000000000 +1000
> +++ working-2.6.18-rc3-mm2-early_param-for-all/arch/s390/kernel/setup.c	2006-08-11 17:14:03.000000000 +1000
> @@ -600,6 +600,7 @@ setup_arch(char **cmdline_p)
>  
>  	/* Save unparsed command line copy for /proc/cmdline */
>  	strlcpy(saved_command_line, COMMAND_LINE, COMMAND_LINE_SIZE);
> +	parse_early_param();
>  
>  	*cmdline_p = COMMAND_LINE;
>  	*(*cmdline_p + COMMAND_LINE_SIZE - 1) = '\0';

This part of the patch should be removed.
That would be s390's second call to parse_early_param(). It needs to be done
a bit later _after_ the 'memory_end = memory_size' line, where it is already.
Otherwise passing 'mem=' to the kernel will have no effect anymory.