RE: [Xen-ia64-devel] crash while initializing balloon driver

["Zhang, Jingke" <jingke.zhang@xxxxxxxxx>]

Hi Alex,
    I have tested your patch on Cset#13122 (including 13117), it can
boot dom0. Great finding!

Thanks,
Zhangjingke

>-----Original Message-----
>From: Alex Williamson [mailto:alex.williamson@xxxxxx]
>Sent: Friday, January 05, 2007 1:23 PM
>To: Zhang, Jingke
>Cc: xen-ia64-devel
>Subject: RE: [Xen-ia64-devel] crash while initializing balloon driver
>
>On Fri, 2007-01-05 at 12:59 +0800, Zhang, Jingke wrote:
>> Hi Alex,
>>     As you said, I use " hg export 13117 | patch -p1 -R " in the
>> Cset#13122. Without 13117's patch, dom0 can boot up (Mem24M).
>
>  Great.  I think the proper fix is something like this:
>
>diff -r 0f571adbd700 xen/arch/ia64/xen/dom_fw.c
>--- a/xen/arch/ia64/xen/dom_fw.c       Thu Jan 04 16:25:14 2007 -0700
>+++ b/xen/arch/ia64/xen/dom_fw.c       Thu Jan 04 22:15:39 2007 -0700
>@@ -549,7 +549,8 @@ complete_dom0_memmap(struct domain *d,
>                       flags = ASSIGN_writable;        /* dummy - zero
*/
>                       if (md->attribute & EFI_MEMORY_WP)
>                               flags |= ASSIGN_readonly;
>-                      if (md->attribute & EFI_MEMORY_UC)
>+                      if (md->attribute & EFI_MEMORY_UC &&
>+                          !(md->attribute & EFI_MEMORY_WB))
>                               flags |= ASSIGN_nocache;
>
>                       assign_domain_mach_page(d, start, size, flags);
>
>Could you please test that this fixes the problem?  You'll need to need
>to undo reverting 13117 before applying this patch (hg revert).
Thanks,
>
>       Alex
>
>--
>Alex Williamson                             HP Open Source & Linux Org.

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