[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Xen-ia64-devel] Community effort neededtocatch upwithxen-unstable


  • To: "Magenheimer, Dan \(HP Labs Fort Collins\)" <dan.magenheimer@xxxxxx>, "Byrne, John \(HP Labs\)" <john.l.byrne@xxxxxx>
  • From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
  • Date: Wed, 7 Sep 2005 08:59:06 +0800
  • Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
  • Delivery-date: Wed, 07 Sep 2005 00:56:57 +0000
  • List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
  • Thread-index: AcWvZqLq66NFDvT0SxG0t4WzIxQpTQAANxPQAPEyzMAABo/zoA==
  • Thread-topic: [Xen-ia64-devel] Community effort neededtocatch upwithxen-unstable

>From: Magenheimer, Dan (HP Labs Fort Collins)
>
>Hi Kevin --
>
>I hadn't yet applied this patch as I recall there was some discussion
>about doing it differently.  Should I go ahead and apply it and
>we will fix it later?

Go ahead since it's related to correctness of second domain. We can
enhance it later. BTW, I'll send out the tools change to xen mailing
list, since some common code is changed upon ia64.

Thanks,
Kevin
>
>Dan
>
>> Currently I just put the last page allocated to xenU as the
>> store page,
>> not in the middle. But yes, there's still one problem in dom_fw_init,
>> where all allocated pages are exposed to domain. Instead we should
>> reserve the last page and let store page out of domain's scope. Check
>> whether following changes solves your issue:
>>
>> diff -r 8799d14bef77 xen/arch/ia64/dom_fw.c
>> --- a/xen/arch/ia64/dom_fw.c    Thu Aug 25 22:53:20 2005
>> +++ b/xen/arch/ia64/dom_fw.c    Fri Sep  2 10:38:11 2005
>> @@ -512,6 +512,11 @@
>>                 return 0;
>>         }
>>  */
>> +
>> +       /* Last page is for xenstore, and not exported to domain */
>> +       if (d != dom0)
>> +               maxmem = (d->max_pages - 1) * PAGE_SIZE;
>> +
>>         memset(fw_mem, 0, fw_mem_size);
>>
>>  #ifdef XEN

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.