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]Creating VTI will reboot whole system inlate

To: "Alex Williamson" <alex.williamson@xxxxxx>, "You, Yongkang" <yongkang.you@xxxxxxxxx>
Subject: RE: [Xen-ia64-devel] [PATCH]Creating VTI will reboot whole system inlatest Cset.
From: "Xu, Anthony" <anthony.xu@xxxxxxxxx>
Date: Thu, 27 Apr 2006 10:10:08 +0800
Cc: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 26 Apr 2006 19:10:23 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
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>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcZpe6D8rT8zxEOYQpiqMEzBplEIpgAI6rhg
Thread-topic: [Xen-ia64-devel] [PATCH]Creating VTI will reboot whole system inlatest Cset.
Alex,

Thank you for providing the useful information.
This patch makes VTI-domain boot again.


Signed-off-by: Kevin Tian <kevin.tian@xxxxxxxxx>
Signed-off-by: Anthony Xu <anthony.tian@xxxxxxxxx>


diff -r 83f7dfe273a0 xen/arch/ia64/xen/domain.c
--- a/xen/arch/ia64/xen/domain.c        Wed Apr 26 05:38:09 2006
+++ b/xen/arch/ia64/xen/domain.c        Thu Apr 27 09:48:56 2006
@@ -680,7 +680,7 @@
 
        //XXX CONFIG_XEN_IA64_DOM0_VP
        //    TODO racy
-       if ((mpaddr & GPFN_IO_MASK) == GPFN_MEM)
+       if ((physaddr & GPFN_IO_MASK) == GPFN_MEM)
                set_gpfn_from_mfn(physaddr >> PAGE_SHIFT, mpaddr >> PAGE_SHIFT);
        }
        else printk("assign_domain_page: mpaddr %lx already mapped!\n",mpaddr);

Thanks,
-Anthony 


>-----Original Message-----
>From: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
>[mailto:xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Alex
>Williamson
>Sent: 2006?4?27? 5:52
>To: You, Yongkang
>Cc: Isaku Yamahata; xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
>Subject: Re: [Xen-ia64-devel] Creating VTI will reboot whole system inlatest
>Cset.
>
>On Wed, 2006-04-26 at 09:04 -0600, Alex Williamson wrote:
>> On Wed, 2006-04-26 at 18:21 +0800, You, Yongkang wrote:
>> > Hi all,
>> >
>> > I found latest Cset 9851 has problem to create VTI domain.
>> > When try to create VTI domain, the whole system will reset.
>>
>>    I see the same thing.  Looks like it started with cset 9756
>> "introduce P2M conversion".  I'll see if there's any obvious way to fix
>> it since reverting that patch may be difficult.  Thanks,
>
>Isaku,
>
>   AFAICT, we're dying in assign_domain_page() called from
>vmx_build_physmap_table() for the io_ranges.  Could you make sure I
>didn't break something when applying the patches.  The physaddr passed
>into assign_domain_page() from there seems suspicious.  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

Attachment: make_vti_boot.patch
Description: make_vti_boot.patch

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel
<Prev in Thread] Current Thread [Next in Thread>