xen-ia64-devel

[Top] [All Lists]

Re: [Xen-ia64-devel] [PATCH 5/5] kexec: remove unnecessary CONFIG_XEN

from [Simon Horman]

[Permanent Link][Original]

To:	Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Subject:	Re: [Xen-ia64-devel] [PATCH 5/5] kexec: remove unnecessary CONFIG_XEN
From:	Simon Horman <horms@xxxxxxxxxxxx>
Date:	Thu, 24 Jul 2008 17:51:28 +1000
Cc:	xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date:	Thu, 24 Jul 2008 00:51:32 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to:	<20080724055959.GA14470@xxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help:	<mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id:	Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post:	<mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References:	<20080724055959.GA14470@xxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender:	xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent:	Mutt/1.5.18 (2008-05-17)

On Thu, Jul 24, 2008 at 02:59:59PM +0900, Isaku Yamahata wrote:
> [IA64] kexec: remove unnecessary #ifdef CONFIG_XEN.
> 
> There is no specific reason to reduce memory and stack registers
> for kexec. So remove those #ifdef CONFIG_XEN.

The reason for this was that I wanted to make the entire
relocate_kernel code fit into a single 16k page because
I was worried that if more than one page was used
it might not be machine contiguous.

As you pointed out offline:

   "I understood.  The pages is allocated by kimage_alloc_pages() which
    calls xen_limit_pages_to_max_mfn() so that pages are guaranteed to
    be machine contiguous."

So I am fine with this change.

> Signed-off-by: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>

Acked-by: Simon Horman <horms@xxxxxxxxxxxx>

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-ia64-devel] [PATCH 5/5] kexec: remove unnecessary CONFIG_XEN, Isaku Yamahata Re: [Xen-ia64-devel] [PATCH 5/5] kexec: remove unnecessary CONFIG_XEN, Simon Horman <=

Previous by Date:	Re: [Xen-ia64-devel] [PATCH 3/5] kexec: remove warning., Simon Horman
Next by Date:	Re: [Xen-ia64-devel] [PATCH 4/5] kexec: fix /proc/iomem_machine, Simon Horman
Previous by Thread:	[Xen-ia64-devel] [PATCH 5/5] kexec: remove unnecessary CONFIG_XEN, Isaku Yamahata
Next by Thread:	[Xen-ia64-devel] [PATCH] xencomm: support PHYSDEVOP_manage_pci_add/remove., Isaku Yamahata
Indexes:	[Date] [Thread] [Top] [All Lists]