public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <ak@suse.de>
To: virtualization@lists.osdl.org
Cc: Jeremy Fitzhardinge <jeremy@xensource.com>,
	Andrew Morton <akpm@osdl.org>,
	Xen-devel <xen-devel@lists.xensource.com>,
	Ian Pratt <ian.pratt@xensource.com>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	Chris Wright <chrisw@sous-sol.org>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Christoph Lameter <clameter@sgi.com>
Subject: Re: [PATCH 8 of 13] Add a bootparameter to reserve high linear address space for hypervisors
Date: Tue, 1 Aug 2006 23:47:20 +0200	[thread overview]
Message-ID: <200608012347.20556.ak@suse.de> (raw)
In-Reply-To: <0adfc39039c79e4f4121.1154462446@ezr>

 
> +		/*
> +		 * reservedtop=size reserves a hole at the top of the kernel
> +		 * address space which a hypervisor can load into later.
> +		 * Needed for dynamically loaded hypervisors, so relocating
> +		 * the fixmap can be done before paging initialization.
> +		 * This hole must be a multiple of 4M.
> +		 */
> +		else if (!memcmp(from, "reservedtop=", 12)) {
> +			unsigned long reserved = memparse(from+12, &from);
> +			reserved &= ~0x3fffff;
> +			set_fixaddr_top(-reserved);
> +		}

You need to add a dummy __setup for it, otherwise it will end up in
init's environments or be warned about.

Also it should be documented somewhere in Documentation

-Andi


       reply	other threads:[~2006-08-01 21:48 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <0adfc39039c79e4f4121.1154462446@ezr>
2006-08-01 21:47 ` Andi Kleen [this message]
2006-08-02  0:48   ` [PATCH 8 of 13] Add a bootparameter to reserve high linear address space for hypervisors Rusty Russell
2006-08-02  2:59     ` Andi Kleen
2006-08-02  3:54       ` [Xen-devel] " Rusty Russell
2006-08-02  4:21         ` Andi Kleen
2006-08-02  4:33           ` Rusty Russell
2006-08-02  4:36             ` Andi Kleen
2006-08-02  5:20               ` Rusty Russell
2006-08-02  5:24                 ` Andi Kleen
2006-08-02  9:06                   ` [PATCH 1/2] Allow early_param and identical __setup to exist Rusty Russell
2006-08-02  9:08                   ` [PATCH 2/2] Replace i386 open-coded cmdline parsing with early_param/parse_early_param Rusty Russell
2006-08-02  9:24                     ` Andi Kleen

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=200608012347.20556.ak@suse.de \
    --to=ak@suse.de \
    --cc=akpm@osdl.org \
    --cc=chrisw@sous-sol.org \
    --cc=clameter@sgi.com \
    --cc=ebiederm@xmission.com \
    --cc=ian.pratt@xensource.com \
    --cc=jeremy@xensource.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=virtualization@lists.osdl.org \
    --cc=xen-devel@lists.xensource.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox