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] SMBios table support in XEN

To: "Alex Williamson" <alex.williamson@xxxxxx>
Subject: RE: [Xen-ia64-devel][PATCH] SMBios table support in XEN
From: "Zhang, Xing Z" <xing.z.zhang@xxxxxxxxx>
Date: Fri, 24 Aug 2007 13:11:58 +0800
Cc: xen-ia64-devel <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 23 Aug 2007 22:12:22 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1187877343.6810.200.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: <823A93EED437D048963A3697DB0E35DEA1C615@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <1187877343.6810.200.camel@bling>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcfljWLzAvJVb/2GSYWdeXbCn2J5bAAf11Zg
Thread-topic: [Xen-ia64-devel][PATCH] SMBios table support in XEN
This one is for your comments. I also clean up some redundant spaces. 
For XENVER_UUID_LEN, I temp define it in xc_ia64_hvm_build.c and it 
would be removed when IA32 standardize on the marco.

>-----Original Message-----
>From: Alex Williamson [mailto:alex.williamson@xxxxxx] 
>Sent: 2007年8月23日 21:56
>To: Zhang, Xing Z
>Cc: xen-ia64-devel
>Subject: Re: [Xen-ia64-devel][PATCH] SMBios table support in XEN
>
>On Thu, 2007-08-23 at 19:00 +0800, Zhang, Xing Z wrote:
>> Hi Alex:
>>      This patch used to get XEN version info and pass to GFW throuhg
>> Hob. I tested it with other pathes which for GFW, OSI interface works
>> fine with linux kernel 2.6.22.3.
>
>Hi,
>
>   Two comments.  First, check your indenting, it looks like you've got
>tab indenting additions to space indented files in a few places.
>Second, defining and using XENVER_UUID_LEN looks like a separate patch
>that should be sent to xen-devel.  I don't see why there would be any
>problem getting it in, but we can just as easily use 16 until we
>standardize on the macro.  Thanks,
>
>       Alex
>
>-- 
>Alex Williamson                             HP Open Source & Linux Org.
>

Attachment: smbios_for_xen_new.patch
Description: smbios_for_xen_new.patch

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