WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-ia64-devel

[Xen-ia64-devel] Re: [PATCH] [ia64] efi: allow efi memory regions to be

To: Alex Williamson <alex.williamson@xxxxxx>
Subject: [Xen-ia64-devel] Re: [PATCH] [ia64] efi: allow efi memory regions to be dumped at boot
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Wed, 27 Feb 2008 10:14:00 +0900
Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 26 Feb 2008 17:14:16 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1204049633.24352.38.camel@bling>
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/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20080226044607.A665F4F937@xxxxxxxxxxxxxxxxxxxxxxxxxxx> <1204049633.24352.38.camel@bling>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.17+20080114 (2008-01-14)
On Tue, Feb 26, 2008 at 11:13:53AM -0700, Alex Williamson wrote:
> 
> On Tue, 2008-02-26 at 13:42 +0900, Simon Horman wrote:
> > In Linux, when EFI_DEBUG in xen/arch/ia64/linux-xen/efi.c is set to
> > a non-zero value the EFI memory regions, as supplied by the
> > boot parameter are displayed. Unfortunately this doesn't work
> > on Xen as the console is not available at this time.
> 
> Hi Simon,
> 
>    Looks good, but there's already an efi_print in xensetup that should
> be incorporated.  We can currently dump the EFI memory map by passing
> the efi_print option on the xen boot line.  We should keep that
> functionality.  The split printk you're replacing is a bit dubious as
> we're relying on not printing another '\n' in the middle to retain the
> log level.  Thanks,

Hi Alex,

thanks for pointing that out, I wasn't aware of efi_print.
I'll look into consolidating my change into that code.

-- 
Horms


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