LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Haren Myneni <haren@us.ibm.com>
To: Michael Neuling <mikey@neuling.org>
Cc: linux-kernel@vger.kernel.org, H Peter Anvin <hpa@zytor.com>,
	Andrew Morton <akpm@osdl.org>, Al Viro <viro@ftp.linux.org.uk>,
	hpa@zytor.com, Fastboot mailing list <fastboot@lists.osdl.org>
Subject: Re: [PATCH] free initrds boot option
Date: Wed, 06 Dec 2006 19:56:27 -0800	[thread overview]
Message-ID: <457790EB.4060302@us.ibm.com> (raw)
In-Reply-To: <4410.1165450723@neuling.org>

Michael Neuling wrote:

>Add free_initrd= option to control freeing of initrd memory after
>extraction.  By default, free memory as previously.
>
>Signed-off-by: Michael Neuling <mikey@neuling.org>
>---
>Useful for kexec when you want to reuse the same initrd.  Testing on
>POWERPC with CPIOs
>  
>
   
 This option (free_initrd) will not work if the user loads the kdump 
kernel and does the normal kexec boot later on powerpc. The reason is 
initrd will be loaded by yaboot at 36MB and kdump image at 32M (Right 
now it is fixed). It could be possible that we will end up overwriting 
initrd.

> init/initramfs.c |   18 ++++++++++++++++--
> 1 file changed, 16 insertions(+), 2 deletions(-)
>
>Index: linux-2.6-ozlabs/init/initramfs.c
>===================================================================
>--- linux-2.6-ozlabs.orig/init/initramfs.c
>+++ linux-2.6-ozlabs/init/initramfs.c
>@@ -487,6 +487,17 @@ static char * __init unpack_to_rootfs(ch
> 	return message;
> }
>
>+static int do_free_initrd = 1;
>+
>+int __init free_initrd_param(char *p)
>+{
>+	if (p && strncmp(p, "0", 1) == 0)
>+		do_free_initrd = 0;
>+
>+	return 0;
>+}
>+early_param("free_initrd", free_initrd_param);
>+
> extern char __initramfs_start[], __initramfs_end[];
> #ifdef CONFIG_BLK_DEV_INITRD
> #include <linux/initrd.h>
>@@ -494,10 +505,13 @@ extern char __initramfs_start[], __initr
>
> static void __init free_initrd(void)
> {
>-#ifdef CONFIG_KEXEC
> 	unsigned long crashk_start = (unsigned long)__va(crashk_res.start);
> 	unsigned long crashk_end   = (unsigned long)__va(crashk_res.end);
>
>+	if (!do_free_initrd)
>+		goto skip;
>+
>+#ifdef CONFIG_KEXEC
> 	/*
> 	 * If the initrd region is overlapped with crashkernel reserved region,
> 	 * free only memory that is not part of crashkernel region.
>@@ -515,7 +529,7 @@ static void __init free_initrd(void)
> 	} else
> #endif
> 		free_initrd_mem(initrd_start, initrd_end);
>-
>+skip:
> 	initrd_start = 0;
> 	initrd_end = 0;
> }
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/
>  
>


  parent reply	other threads:[~2006-12-07  3:56 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-07  0:18 Michael Neuling
2006-12-07  0:25 ` Randy Dunlap
2006-12-07  0:30 ` Andrew Morton
2006-12-07  0:37   ` H. Peter Anvin
2006-12-07  3:36     ` Michael Neuling
2006-12-07 16:47       ` [Fastboot] " Vivek Goyal
2006-12-07 21:40         ` Haren Myneni
2006-12-07 23:32           ` Michael Neuling
2006-12-13  1:35             ` [Fastboot] " Horms
2007-02-08 12:35               ` Horms
2007-02-08 22:58                 ` Michael Neuling
2006-12-07  3:56 ` Haren Myneni [this message]
2006-12-07  3:42 Michael Neuling
2006-12-07  5:07 ` Randy Dunlap
2006-12-07  5:14 ` Randy Dunlap

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=457790EB.4060302@us.ibm.com \
    --to=haren@us.ibm.com \
    --cc=akpm@osdl.org \
    --cc=fastboot@lists.osdl.org \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mikey@neuling.org \
    --cc=viro@ftp.linux.org.uk \
    --subject='Re: [PATCH] free initrds boot option' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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).