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

Re: [Xen-ia64-devel] [PATCH] print efi map descriptor infomation at boot

To: Alex Williamson <alex.williamson@xxxxxx>
Subject: Re: [Xen-ia64-devel] [PATCH] print efi map descriptor infomation at bootup
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Fri, 21 Apr 2006 14:29:14 +0900
Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 20 Apr 2006 22:29:26 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1145589275.22194.219.camel@localhost>
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: <20060421023938.GE7264%yamahata@xxxxxxxxxxxxx> <1145588468.22194.214.camel@localhost> <1145589275.22194.219.camel@localhost>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
I attached the updated patch with your comment.

On Thu, Apr 20, 2006 at 09:14:35PM -0600, Alex Williamson wrote:
> On Thu, 2006-04-20 at 21:01 -0600, Alex Williamson wrote:
> > On Fri, 2006-04-21 at 11:39 +0900, Isaku Yamahata wrote:
> > > print efi map descriptor infomation at bootup.
> > 
> > Hi Isaku,
> > 
> >    This is useful for debug, but is it really something we want to print
> > out to the console on every boot?  Maybe this could be off by default,
> > and enabled through a boot time option.  The linux-xen/efi.c changes
> > should probably also be enclosed in #ifdef XEN markers.  Thanks,
> 
>    On second thought, is there any reason efi_print() needs to be in
> linux-xen?  Seems this is may be fairly self-contained and could go in
> xenmisc (or somewhere similar) to avoid further modifying linux-xen
> files.  Thanks,
> 
>       Alex
> 
> -- 
> Alex Williamson                             HP Linux & Open Source Lab
> 
> 
> _______________________________________________
> Xen-ia64-devel mailing list
> Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-ia64-devel
> 

-- 
yamahata

Attachment: 9745:0b1a6af5a17f_efi_print.patch
Description: Text document

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